Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ItemPedido #11

Open
Allanfs opened this issue Jun 14, 2019 · 2 comments
Open

ItemPedido #11

Allanfs opened this issue Jun 14, 2019 · 2 comments
Assignees
@Allanfs
Milestone
Manter Pedido

Comments

@Allanfs
Copy link
Owner

Allanfs commented Jun 14, 2019

Um item de pedido não faz sentido sem um Pedido. Por isso não é necessário haver um serviço para eles. Basta que na classe Pedido seja marcado os CascadeType, e orphanRemoval.

Os ItemPedido possuem referencia a um item de pedido, esta referencia não pode sofrer modificação após o pedido ser persistido.
@Allanfs Allanfs added this to the Manter Pedido milestone Jun 14, 2019
@Allanfs Allanfs self-assigned this Jun 14, 2019
@Allanfs
Copy link
Owner Author

Allanfs commented Jun 20, 2019

commit 911b770 troca o mapeamento de Set<ItemPedido> para Map<Produto, Integer>

Allanfs pushed a commit that referenced this issue Jun 21, 2019
Refatorado @getter @Setter das classes
592de85 
#2 #3 #4 #6 #7 #8 #10 #11 #12
removido anotações dos atributos, e adicionado na classe, dessa forma,
todos os atributos possuirão os métodos getters e setters
@Allanfs
Copy link
Owner Author

Allanfs commented Jul 8, 2019

É necessário voltar para Set, devido à conversão de JSON para Map em uma requisição via REST. Devido ao limitado conhecimento na biblioteca, torna-se necessário voltar ao uso do Set<ItemPedido>, e para evitar consumir mais tempo desenvolvendo esta funcionalidade.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Allanfs Allanfs

Labels
None yet
Projects
None yet
Milestone
Manter Pedido
Development

No branches or pull requests
1 participant
@Allanfs