Add feedArticles endpoint and fix method naming consistency

gunkim · Feb 19, 2025 · a447cb2 · a447cb2
1 parent d23cd23
commit a447cb2
Showing 1 changed file with 12 additions and 12 deletions.
diff --git a/src/main/kotlin/io/github/gunkim/realworld/web/api/article/ArticlesController.kt b/src/main/kotlin/io/github/gunkim/realworld/web/api/article/ArticlesController.kt
@@ -34,6 +34,12 @@ interface ArticleResource {
         @AuthenticationPrincipal authenticatedUser: AuthenticatedUser?,
     ): ArticlesResponse
 
+    @GetMapping("/feed")
+    fun feedArticles(
+        @ModelAttribute request: PagingRequest,
+        @AuthenticationPrincipal authenticatedUser: AuthenticatedUser,
+    ): ArticlesResponse
+
     @GetMapping("/{slug}")
     fun getArticle(@PathVariable slug: String): ArticleResponseWrapper
 
@@ -45,20 +51,20 @@ interface ArticleResource {
     ): ArticleResponseWrapper
 
     @PutMapping("/{slug}")
-    fun updateArticles(
+    fun updateArticle(
         @PathVariable slug: String,
         @JsonRequest("article") request: UpdateArticleRequest,
         @AuthenticationPrincipal authenticatedUser: AuthenticatedUser,
     ): ArticleResponseWrapper
 
     @DeleteMapping("/{slug}")
-    fun deleteArticles(
+    fun deleteArticle(
         @PathVariable slug: String,
         @AuthenticationPrincipal authenticatedUser: AuthenticatedUser,
     )
 
     @PostMapping("/{slug}/favorite")
-    fun favoriteArticles(
+    fun favoriteArticle(
         @PathVariable slug: String,
         @AuthenticationPrincipal authenticatedUser: AuthenticatedUser,
     ): ArticleResponseWrapper
@@ -68,12 +74,6 @@ interface ArticleResource {
         @PathVariable slug: String,
         @AuthenticationPrincipal authenticatedUser: AuthenticatedUser,
     ): ArticleResponseWrapper
-
-    @GetMapping("/feed")
-    fun feedArticles(
-        @ModelAttribute request: PagingRequest,
-        @AuthenticationPrincipal authenticatedUser: AuthenticatedUser,
-    ): ArticlesResponse
 }
 
 @RestController
@@ -119,7 +119,7 @@ class ArticlesController(
             .let(::ArticleResponseWrapper)
     }
 
-    override fun updateArticles(
+    override fun updateArticle(
         slug: String,
         request: UpdateArticleRequest,
         authenticatedUser: AuthenticatedUser,
@@ -134,14 +134,14 @@ class ArticlesController(
             .let(::ArticleResponseWrapper)
     }
 
-    override fun deleteArticles(
+    override fun deleteArticle(
         slug: String,
         authenticatedUser: AuthenticatedUser,
     ) {
         deleteArticleUseCase.deleteArticle(slug, authenticatedUser.uuid)
     }
 
-    override fun favoriteArticles(
+    override fun favoriteArticle(
         slug: String,
         authenticatedUser: AuthenticatedUser,
     ): ArticleResponseWrapper {