-
Notifications
You must be signed in to change notification settings - Fork 1
Apply auditing to entity Pedido and Renglon Pedido #465
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
k31kur0n0
wants to merge
36
commits into
master
Choose a base branch
from
issue442JaVers
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
20d8563
Cambia la forma de guardar un Pedido
k31kur0n0 3942843
First approach of Javer's use
k31kur0n0 faebe83
First approach to use switch
k31kur0n0 208b8a5
Implements the changeDTO and ValueChangeDTO models.
k31kur0n0 6a604d9
Change the ListChange treatment
k31kur0n0 08f769c
Change attributes names
k31kur0n0 519c552
Fixed the obtain data when a new instance is audited
k31kur0n0 337bb2d
The funtion to add or update a pedido its broken because the test DB …
k31kur0n0 59b2eb6
Adds the Audit Service Interface and a implementation
k31kur0n0 0dc1432
Changes the renglones commit endpoint to bring all the renglones comm…
k31kur0n0 aa2d5d3
Update dependencies and delete nombreDeClase from CommitDTO model. A …
k31kur0n0 ebbb4bc
Rename the AuditServiceImpl to JaversServiceImpl
k31kur0n0 4f627dc
Merge branch 'master' into issue442JaVers
k31kur0n0 a6bc436
Changed the audit service interface and the Javers service implementa…
k31kur0n0 bcd3418
Delete code and a non neccesary class. Adds some integration test.
k31kur0n0 795ad7b
Make the requested changes
k31kur0n0 774c3a1
Make the last requested changes
k31kur0n0 400cb6c
Fixed the integration test
k31kur0n0 2ff8eda
First approach to test the audited pedido details
k31kur0n0 8cbf759
Adds Test to prove the builders
k31kur0n0 5297c99
Deleted unnecesary test
k31kur0n0 bd9b83a
Resolve code smell
k31kur0n0 fb400e5
Fixed the raw use of parameterized class error
k31kur0n0 477e98b
Deleted unnesesary coment
k31kur0n0 2fe525b
Deleted unused code
k31kur0n0 a8e0a04
Remove unused import and apply format
belluccifranco 2765ad3
Renamed the JaversAuditServiceImpl
k31kur0n0 7bb8215
Merge branch 'master' into issue442JaVers
belluccifranco c17f36b
Remove unnecessary code
k31kur0n0 6d89624
Remove whitespaces
belluccifranco 404776a
Apply refactoring in javers impl
belluccifranco 740408d
Added the renglones special case to fix the missings details. Changed…
k31kur0n0 ebc6b7d
Fixed the integration test
k31kur0n0 0948fb8
Changed the instegration test to use list instead map
k31kur0n0 248d597
Merge pull request #470 from open-commercial/master
belluccifranco 2d88cab
Merge branch 'master' into issue442JaVers
belluccifranco File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| package sic.modelo.dto; | ||
|
|
||
|
|
||
| import com.fasterxml.jackson.annotation.JsonView; | ||
| import lombok.AllArgsConstructor; | ||
| import lombok.Builder; | ||
| import lombok.Data; | ||
| import lombok.NoArgsConstructor; | ||
| import sic.controller.Views; | ||
|
|
||
| @Data | ||
| @AllArgsConstructor | ||
| @NoArgsConstructor | ||
| @Builder | ||
| @JsonView(Views.Encargado.class) | ||
| public class CambioDTO { | ||
|
|
||
| private String atributo; | ||
| private String valorAnterior; | ||
| private String valorSiguiente; | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,27 @@ | ||
| package sic.modelo.dto; | ||
|
|
||
| import com.fasterxml.jackson.annotation.JsonView; | ||
| import lombok.AllArgsConstructor; | ||
| import lombok.Builder; | ||
| import lombok.Data; | ||
| import lombok.NoArgsConstructor; | ||
| import sic.controller.Views; | ||
|
|
||
| import java.time.LocalDateTime; | ||
| import java.util.List; | ||
|
|
||
| @Data | ||
| @AllArgsConstructor | ||
| @NoArgsConstructor | ||
| @Builder | ||
| @JsonView(Views.Encargado.class) | ||
| public class CommitDTO { | ||
|
|
||
| private String idCommit; | ||
| private String idCommitRelacionado; | ||
| private LocalDateTime fecha; | ||
| private String usuario; | ||
| private String tipoDeOperacion; | ||
| private List<CambioDTO> cambios; | ||
|
|
||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| package sic.service; | ||
|
|
||
| import sic.modelo.dto.CommitDTO; | ||
| import java.util.List; | ||
| import java.util.Map; | ||
|
|
||
| public interface IAuditService { | ||
|
|
||
| String auditar(String idUsuario, Object objeto, Map<String, String> propiedades); | ||
|
|
||
| <T> List<CommitDTO> getCambios(T objeto); | ||
|
|
||
| List<CommitDTO> getCambios(String idCommit); | ||
|
|
||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.