delete get에서 post로 변경

Author: wonwoo

web/src/main/kotlin/com/example/message/MessageController.kt

@@ -36,8 +36,7 @@ class MessageController(private val messageService: MessageService) {
 
     }
 
-    //TODO GET -> POST
-    @GetMapping("/delete/{id}")
+    @PostMapping("/delete/{id}")
     fun delete(@PathVariable id: Long): String {
 
         messageService.delete(id)

web/src/main/resources/templates/message.html

@@ -47,10 +47,23 @@ <h2>Message</h2>
             <th th:text="${message.message}"></th>
             <td th:text="${message.account.name}">Mark</td>
             <td th:text="${message.regDate}"></td>
-            <td><a class="btn btn-default glyphicon" th:href="@{'/message/delete/' + ${message.id}}">Delete</a></td>
+            <td><a class="btn btn-default glyphicon" th:attr="onclick=|delete_message('${message.id}')|">Delete</a></td>
         </tr>
         </tbody>
     </table>
+
+    <form role="form" th:name="delete-form" th:id="delete-form" th:action="@{/}" th:method="post">
+    </form>
+    <script>
+
+      function delete_message(id) {
+        var form = document.getElementById("delete-form");
+        form.action = "/message/delete/" + id;
+        form.method = "post";
+        form.submit();
+      }
+
+    </script>
 </div>
 </body>
 </html>

web/src/test/kotlin/com/example/message/MessageControllerTests.kt

@@ -82,7 +82,7 @@ internal class MessageControllerTests(@Autowired private val mockMvc: MockMvc) {
 
         doNothing().`when`(messageService).delete(1L)
 
-        mockMvc.get("/message/delete/{id}", 1) {
+        mockMvc.post("/message/delete/{id}", 1) {
 
             accept = MediaType.TEXT_HTML
             with(csrf())

webflux/src/main/kotlin/com/example/message/MessageController.kt

@@ -41,8 +41,7 @@ class MessageController(private val messageService: MessageService) {
 
     }
 
-    //TODO GET -> POST
-    @GetMapping("/delete/{id}")
+    @PostMapping("/delete/{id}")
     fun delete(@PathVariable id: String): Rendering {
 
         return Rendering.redirectTo("/message")

webflux/src/main/resources/templates/message.html

@@ -47,10 +47,24 @@ <h2>Message</h2>
             <th th:text="${message.message}"></th>
             <td th:text="${message.account.name}">Mark</td>
             <td th:text="${message.regDate}"></td>
-            <td><a class="btn btn-default glyphicon" th:href="@{'/message/delete/' + ${message.id}}">Delete</a></td>
+            <td><a class="btn btn-default glyphicon" th:attr="onclick=|delete_message('${message.id}')|">Delete</a></td>
         </tr>
         </tbody>
     </table>
+
+    <form role="form" th:name="delete-form" th:id="delete-form" th:action="@{/}" th:method="post">
+    </form>
+
+    <script>
+
+        function delete_message(id) {
+          var form = document.getElementById("delete-form");
+          form.action = "/message/delete/" + id;
+          form.method = "post";
+          form.submit();
+        }
+
+    </script>
 </div>
 </body>
 </html>

webflux/src/test/kotlin/com/example/message/MessageControllerTests.kt

@@ -88,7 +88,7 @@ internal class MessageControllerTests(@Autowired private val webTestClient: WebT
         given(messageService.findAll()).willReturn(Flux.just(Message(id = "foo", message = "test message", account = Account("wonwoo", "foo"))))
 
         webTestClient.mutateWith(csrf())
-            .get()
+            .post()
             .uri("/message/delete/{id}", "foo")
 
             .accept(MediaType.TEXT_HTML)