fix(post): solve problems with update partials

Author: sergio11

src/main/java/controllers/admin/PostController.java

@@ -21,13 +21,15 @@
 import org.springframework.context.support.ReloadableResourceBundleMessageSource;
 import org.springframework.stereotype.Controller;
 import org.springframework.ui.Model;
+import org.springframework.validation.BindingResult;
 import org.springframework.validation.Errors;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.ModelAttribute;
 import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.PostMapping;
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RequestPart;
+import org.springframework.web.bind.annotation.SessionAttributes;
 import org.springframework.web.multipart.MultipartFile;
 import org.springframework.web.servlet.mvc.support.RedirectAttributes;
 import services.PostService;
@@ -40,10 +42,14 @@
  */
 @Controller("AdminPostController")
 @RequestMapping("/admin/posts")
+@SessionAttributes({PostController.ATTRIBUTE_NAME})
 public class PostController {
 
     private static Logger logger = LoggerFactory.getLogger(PostController.class);
 
+    public static final String ATTRIBUTE_NAME = "post";
+    public static final String BINDING_RESULT_NAME = "org.springframework.validation.BindingResult." + ATTRIBUTE_NAME;
+    
     @Autowired
     private PostService postService;
     @Autowired
@@ -59,11 +65,14 @@ public String all(@CurrentUser User activeUser, Model model){
 
     @GetMapping("/edit/{postId}")
     public String showUpdatePostForm(@PathVariable Long postId, Model model) {
-        Post post = postService.findById(postId);
-        if (post == null) {
-            throw new PostNotFoundException();
+        /* if "fresh" GET (ie, not redirect w validation errors): */
+        if(!model.containsAttribute(BINDING_RESULT_NAME)) {
+            Post post = postService.findById(postId);
+            if (post == null) {
+                throw new PostNotFoundException();
+            }
+            model.addAttribute(ATTRIBUTE_NAME, post);
         }
-        model.addAttribute("post", post);
         return "admin/post/edit";
     }
 
@@ -88,19 +97,26 @@ public String processDelete(@ModelAttribute Post post, RedirectAttributes model)
 
     @GetMapping("/create")
     public String showCreatePostForm(Model model){
-        model.addAttribute("post", new Post());
+        model.addAttribute(ATTRIBUTE_NAME,  new Post());
         return "admin/post/create";
     }
 
     @PostMapping("/save")
-    public String processPost(@RequestPart("postImage") MultipartFile postImage, @ModelAttribute @Valid Post post, Errors errors, 
-            @CurrentUserAttached User activeUser, RedirectAttributes model) throws IOException, SQLException{
+    public String processPost(
+            @RequestPart("postImage") MultipartFile postImage, 
+            @ModelAttribute(ATTRIBUTE_NAME) @Valid Post post, 
+            BindingResult bindingResult, 
+            @CurrentUserAttached User activeUser, 
+            RedirectAttributes model) throws IOException, SQLException{
 
-        if(errors.hasErrors()){
-            return "admin/post/create";
+        String url = "redirect:/admin/posts/all";
+
+        if (bindingResult.hasErrors()) {
+            model.addFlashAttribute(BINDING_RESULT_NAME, bindingResult);
+            return url;
         }
-        
-        if(postImage != null && !postImage.isEmpty()){
+
+        if (postImage != null && !postImage.isEmpty()) {
             logger.info("Añadiendo información de la imagen");
             FileImage image = new FileImage();
             image.setName(postImage.getName());
@@ -109,19 +125,17 @@ public String processPost(@RequestPart("postImage") MultipartFile postImage, @Mo
             image.setContent(postImage.getBytes());
             post.setImage(image);
         }
-        
+
         post.setAuthor(activeUser);
-        if(post.getId() == null){
-            logger.info("Creando Nuevo Post");
+        if (post.getId() == null) {
             postService.create(post);
-        }else{
-            logger.info("Editando Post");
+        } else {
             postService.edit(post);
         }
-        
+
         List<String> successMessages = new ArrayList();
-        successMessages.add(messageSource.getMessage("message.post.save.success", new Object[] {post.getId()}, Locale.getDefault()));
-        model.addFlashAttribute("successFlashMessages", successMessages);        
-        return "redirect:/admin/posts/all";
+        successMessages.add(messageSource.getMessage("message.post.save.success", new Object[]{post.getId()}, Locale.getDefault()));
+        model.addFlashAttribute("successFlashMessages", successMessages);
+        return url;
     }
 }

src/main/webapp/WEB-INF/templates/admin/fragment/alerts.html

@@ -8,7 +8,7 @@
     <body>
         <div th:fragment="form-errors" class="alert alert-danger" th:if="${#fields.hasErrors('*')}">
             <ul>
-                <li th:each="err: ${#fields.errors('*')}" th:text="${err}">Input is incorrect</li>
+                <li class="text-left" th:each="err: ${#fields.errors('*')}" th:text="${err}">Input is incorrect</li>
             </ul>
         </div>
         <div th:fragment="flash-message" th:if="${not #lists.isEmpty(successFlashMessages)}" class="alert alert-success">

src/main/webapp/WEB-INF/templates/admin/fragment/post.html

@@ -59,7 +59,6 @@
                     <textarea id="bootstrap-editor" th:readonly="${delete}" th:field="*{body}" placeholder="Enter text ..." style="width:98%;height:200px;"></textarea>
                 </div>
             </div>
-            <input type="hidden" th:field="*{id}"/>
             <br />
             <div class="row">
                 <div class="form-group col-xs-12">