feature/edit-address by Pablo-Correia · Pull Request #121 · utfpr-gp/servicebook

Pablo-Correia · 2023-06-22T03:47:36Z

Criado rota e formulário para editar o endereço e também criei um service para separar a lógica que vai salvar e comparar o endereço atual com o antigo para que a rota não fique poluída.

…a rota

…eco do usuario

… nessa nova classe

…do da rota

ronifabio

Não funciona!
Assistir o vídeo de como cadastrar uma entidade
É preciso validar os dados antes de persistir, fazendo uso do DTO.

ronifabio · 2023-06-22T20:06:09Z

src/main/java/br/edu/utfpr/servicebook/controller/MyAccountController.java


+//    teste rota endereço
+    /**
+     * Apresenta a tela de email do usuário.


Comentários não relacionados ao contexto

ronifabio · 2023-06-22T20:07:10Z

src/main/java/br/edu/utfpr/servicebook/controller/MyAccountController.java

+     */
+    @PostMapping("/salvar-endereco/{id}")
+    @RolesAllowed({RoleType.USER, RoleType.COMPANY})
+    public String saveAddress(


É preciso receber um DTO como parâmetro para persistência

ronifabio · 2023-06-22T20:08:46Z

src/main/java/br/edu/utfpr/servicebook/service/AddressService.java

+        try {
+            Optional<User> oUser = this.userService.findById(id);
+            User user = oUser.get();
+            Optional<City> cities = this.cityService.findById(user.getAddress().getCity().getId());


Usa-se plural quando é lista, neste caso, quando é Optional, usa-se: oCity

ronifabio · 2023-06-22T20:10:00Z

src/main/java/br/edu/utfpr/servicebook/service/AddressService.java

+        }
+    }
+
+    public void editAddress(Long id, HttpServletRequest request) {


Não se faz uso do HttpServletRequest para pegar os parâmetros do formulário. O formulário é mapeado a um DTO. Ver os vídeos.

ronifabio · 2023-06-22T20:10:18Z

src/main/java/br/edu/utfpr/servicebook/service/AddressService.java

+            City cityUser = cities.get();
+            Address addressEdit = new Address();
+            addressEdit.setNeighborhood(request.getParameter("neighborhood"));
+            addressEdit.setStreet(request.getParameter("street"));


Não se usa getParameter em Spring

…mudança do getParameter para capturar os valores do formulario usando o dto e mais alguns ajustes como remoção de comentarios etc.

ronifabio · 2023-06-27T01:52:53Z

src/main/java/br/edu/utfpr/servicebook/service/AddressService.java

+            City cityMidDTO = oCity.get();
+
+            Address addressFullDTO = new Address();
            Optional<User> oUser = this.userService.findById(id);


Faltou verificar se o usuário está logado.
E também se o id passado na URL corresponde ao id do usuário logado para ele não alterar dado de outro usuário

ronifabio · 2023-06-27T01:54:33Z

src/main/java/br/edu/utfpr/servicebook/service/AddressService.java

-                user.setAddress(addressEdit);
-                this.userService.save(user);
-            }
+            user.getAddress().setStreet(dto.getStreet().trim());


Usar o Mapper para copiar os dados do DTO para entidade.
Como é edição, o id do Address já existe

ronifabio · 2023-06-27T01:55:16Z

src/main/java/br/edu/utfpr/servicebook/controller/MyAccountController.java

            HttpServletRequest request,
-            RedirectAttributes redirectAttributes)
+            RedirectAttributes redirectAttributes,
+            @Validated(AddressDTO.RequestUserAddressInfoGroupValidation.class) AddressDTO dto,


Não precisa deste grupo: RequestUserAddressInfoGroupValidation

ronifabio · 2023-06-27T01:55:40Z

src/main/java/br/edu/utfpr/servicebook/controller/MyAccountController.java

+            BindingResult errors)
            throws IOException {
-        this.addressService.editAddress(id, request);
+


Faltou usar o errors para mandar os erros de validação ao cliente, quando houver

ronifabio · 2023-06-27T01:55:58Z

src/main/java/br/edu/utfpr/servicebook/controller/MyAccountController.java

+
+        this.addressService.editAddress(id, dto, errors, httpSession);

        redirectAttributes.addFlashAttribute("msg", "Endereço editado com sucesso");


Sempre vai mandar mensagem de sucesso, não está certo

ronifabio · 2023-06-27T01:57:08Z

src/main/java/br/edu/utfpr/servicebook/controller/MyAccountController.java

            throws IOException {
-        this.addressService.editAddress(id, request);
+
+        this.addressService.editAddress(id, dto, errors, httpSession);


Poderia deixar no próprio controller, principalmente as validações e a emissão de exception ou encaminhamento em caso de erro

…eixei comentado pois esta com um erro que deixei um comentario no codigo

ronifabio

Colocar todos os campos input do formulário com disabled.
Ao clicar no botão Editar, remover o modo disabled. Ou seja, o usuário poderá editar os valores. Também esconder o botão Editar e mostrar o botão SALVAR.
O editar é type button e o salvar é type submit, que será usado para submter o formulário.

ronifabio · 2023-06-30T17:42:23Z

src/main/java/br/edu/utfpr/servicebook/controller/MyAccountController.java

        System.out.println(testeNovo);
-        CityDTO testeCity = new CityDTO();
-//        System.out.println(userDTO.getAddress().getCity());
+        Optional<City> cities = this.cityService.findById(testeNovo.getId());


Quando é Optional, usa o prefixo o antes do nome da variável.

Ficaria oCity

Não se usa plural quando e apenas uma unidade, então, é oCity e não cities.

ronifabio · 2023-06-30T17:42:53Z

src/main/java/br/edu/utfpr/servicebook/controller/MyAccountController.java

@@ -209,13 +206,51 @@ public ModelAndView showMyAddress(@PathVariable Long id) throws IOException {

        CityMidDTO testeNovo = userDTO.getAddress().getCity();
        System.out.println(testeNovo);


Não usar português

ronifabio · 2023-06-30T17:44:15Z

src/main/java/br/edu/utfpr/servicebook/controller/MyAccountController.java

+            RedirectAttributes redirectAttributes)
+            throws IOException {
+        System.out.println("realmente um teste aqui");
+        Optional<User> oUser = this.userService.findById(id);


Verificar se o id recebido por parâmetro é o mesmo do usuário logado.

ronifabio · 2023-06-30T17:47:00Z

src/main/java/br/edu/utfpr/servicebook/controller/MyAccountController.java

+     * @return
+     * @throws IOException
+     */
+    @PostMapping("/salvar-endereco/{id}")


Usar PATCH ao invés de POST

Está atualizando parte do usuário, sendo que o id do usuário permanece, só muda o endereço.

ronifabio · 2023-06-30T17:47:29Z

src/main/java/br/edu/utfpr/servicebook/controller/MyAccountController.java

+        System.out.println("realmente um teste aqui");
+        Optional<User> oUser = this.userService.findById(id);
+        User user = oUser.get();
+        Optional<City> cities = this.cityService.findById(user.getAddress().getCity().getId());


ronifabio · 2023-06-30T17:48:44Z

src/main/java/br/edu/utfpr/servicebook/controller/MyAccountController.java

+        System.out.println("realmente um teste aqui");
+        Optional<User> oUser = this.userService.findById(id);
+        User user = oUser.get();
+        Optional<City> cities = this.cityService.findById(user.getAddress().getCity().getId());


Não pode pegar a mesma cidade que já está no usuário e colocar no novo endereço.
Vai que o usuário mudou de cidade.
Você precisa pegar a cidade que ele colocou no campo cidade no formulário.

ronifabio · 2023-06-30T17:50:13Z

src/main/webapp/WEB-INF/view/professional/account/my-address.jsp

                                <div class="row">
                                    <div class="input-field col s8 offset-s2">
-                                        <input id="city" name="city" type="text" placeholder="Cidade" class="validate">
+                                        <input id="city" name="city" value="${city.name}" type="text" placeholder="Cidade" class="validate">


Colocar as cidades em um campo select, não deixar em um input.

Apenas listar as cidades cadastradas no sistema, ou seja, fazer um findAll em City e lista no select.

ronifabio · 2023-06-30T17:51:06Z

src/main/webapp/WEB-INF/view/professional/account/my-address.jsp

-                                        <input id="state" name="state" type="text" placeholder="Estado"
+                                        <input id="state" name="state" value="${city.state.name}" type="text" placeholder="Estado"
                                               class="validate">
                                        <label for="state">Estado</label>


Só listas os estados cadastrados no sistema.
Ver exemplo no cadastro de especialidades que usa um select de categorias
Está em ExpertiseController

… editar e o disabled nos campos, também arrumei algumas coisas refatorações no codigo do controller envolvendo verificações e também mantive o comentario e o mapper já que continuam sem funcionar mesmo usando o PatchMapping

ronifabio

Faltou ajustar o CSS do formulário para deixar padronizado com a edição do telefone.
Ao buscar o endereço por CEP, verificar se a cidade e estado buscados existem no banco de dados. Fazer uma nova requisição ao nosso servidor. Caso a cidade e estado não sejam suportadas, mostrar uma mensagem de erro.
Nesta requisição, usar @responsebody para injetar o json diretamente na resposta.
Olhar o branch dynamic-statistics para exemplo de requisição ajax para o controller e obtenção da resposta.

ronifabio · 2023-07-05T15:52:27Z

Não esquecer de atualizar o seu branch com as minhas modificações.
Com isso, basta fazer este tratamento com JS citado acima.

…, criei uma nova rotapost que retorna ResponseEntity

Pablo-Correia added 5 commits June 20, 2023 22:00

feat: adicionado a nova pagina de edição de endereço e tambem criado …

8f36e58

…a rota

feat: feito mais alguns ajustes no metodo da rota para editar o ender…

5c2e503

…eco do usuario

feat: adicionado novo arquivo addressService e colocado a logica todo…

16e06a3

… nessa nova classe

feat: adicionado a mensagem flash ao html e também reformulado o meto…

e62bfad

…do da rota

refactor: ajustes

665a2a7

Pablo-Correia requested a review from ronifabio June 22, 2023 03:47

ajustes

deba64b

ronifabio requested changes Jun 22, 2023

View reviewed changes

refactor: concluido os ajustes pedidos no review, entre eles estão a …

2509680

…mudança do getParameter para capturar os valores do formulario usando o dto e mais alguns ajustes como remoção de comentarios etc.

ronifabio changed the title ~~Feature/edit address~~ feature/edit-address Jun 27, 2023

ronifabio requested changes Jun 27, 2023

View reviewed changes

refactor: fiz os ajuste que foram pedidos menos o uso do mapper que d…

f418f2f

…eixei comentado pois esta com um erro que deixei um comentario no codigo

ronifabio requested changes Jun 30, 2023

View reviewed changes

Pablo-Correia and others added 2 commits July 2, 2023 21:27

refactor: ajustes gerais

9fb1e03

ronifabio self-requested a review July 4, 2023 16:39

ronifabio requested changes Jul 4, 2023

View reviewed changes

refactor: implementei os ajustes que foram pedidos, a validação do js…

92cdbb1

…, criei uma nova rotapost que retorna ResponseEntity


		this.addressService.editAddress(id, dto, errors, httpSession);

		redirectAttributes.addFlashAttribute("msg", "Endereço editado com sucesso");

		@@ -209,13 +206,51 @@ public ModelAndView showMyAddress(@PathVariable Long id) throws IOException {

		CityMidDTO testeNovo = userDTO.getAddress().getCity();
		System.out.println(testeNovo);

Conversation

Pablo-Correia commented Jun 22, 2023

Uh oh!

ronifabio left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ronifabio left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ronifabio left a comment

Choose a reason for hiding this comment

Uh oh!

ronifabio commented Jul 5, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants