From 8458cbcb9dad13ef7e0c594cb9659b86bdf51916 Mon Sep 17 00:00:00 2001
From: Emre Deger <ben@emre.dev>
Date: Sun, 11 Dec 2022 19:02:35 +0300
Subject: [PATCH] fix: extract on view method to novel service

---
 app/Controllers/Http/NovelController.ts | 68 ++++----------------
 app/Services/NovelService.ts            | 82 +++++++++++++++++++++++++
 2 files changed, 93 insertions(+), 57 deletions(-)
 create mode 100644 app/Services/NovelService.ts

diff --git a/app/Controllers/Http/NovelController.ts b/app/Controllers/Http/NovelController.ts
index bdba02b..6f28c91 100644
--- a/app/Controllers/Http/NovelController.ts
+++ b/app/Controllers/Http/NovelController.ts
@@ -3,75 +3,29 @@ import Database from '@ioc:Adonis/Lucid/Database'
 import NovelPublishStatus from 'App/Enums/NovelPublishStatus'
 import UserRole from 'App/Enums/UserRole'
 import Novel from 'App/Models/Novel'
+import NovelService from 'App/Services/NovelService'
 import NovelRequestValidator from 'App/Validators/NovelRequestValidator'
 import showdown from 'showdown'
 import { isNumeric } from '../../../utils'
 
 export default class NovelController {
   async index({ auth, response, request }: HttpContextContract) {
-    const novelsQuery = Novel.query()
-
     const user = await auth.authenticate()
-
     const isAdmin = user?.role === UserRole.ADMIN
 
-    if (!isAdmin) {
-      novelsQuery.where('publish_status', NovelPublishStatus.PUBLISHED)
-    } else {
-      novelsQuery.preload('user')
-
-      if (request.input('publish_status')) {
-        novelsQuery.where('novels.publish_status', request.input('publish_status'))
-      }
-    }
-
-    if (request.input('filter')) {
-      novelsQuery
-        .where('name', 'ilike', `%${request.input('filter')}%`)
-        .orWhere('other_names', 'ilike', `%${request.input('filter')}%`)
-        .orWhere('shorthand', 'ilike', `%${request.input('filter')}%`)
+    const requestInputs = {
+      page: request.input('page'),
+      take: request.input('take', 10),
+      tags: request.input('tags'),
+      filter: request.input('filter'),
+      fields: request.input('fields'),
+      publish_status: request.input('publish_status'),
     }
 
-    if (request.input('tags')) {
-      const tags = request.input('tags').split(',')
-
-      for (const tag of tags) {
-        novelsQuery.whereExists((query) => {
-          query
-            .select('*')
-            .from('novel_tag')
-            .whereColumn('novel_tag.novel_id', 'novels.id')
-            .where('novel_tag.tag_id', tag)
-        })
-      }
-    }
+    const { have_fields, novels } = await NovelService.onList(user, isAdmin, requestInputs)
 
-    const novels = await novelsQuery
-      .preload('country')
-      .preload('latest_chapter', (query) => {
-        query.preload('volume')
-      })
-      .orderBy('id', 'desc')
-      .paginate(request.input('page', 1), request.input('take', 10))
-
-    const fields = request
-      .input('fields')
-      ?.split(',')
-      ?.filter((x) => x !== 'context')
-
-    if (fields) {
-      const novelsJSON = novels.toJSON()
-
-      return response.send({
-        ...novelsJSON,
-        data: novelsJSON.data.map((novel) => {
-          const result = {}
-          for (const field of fields) {
-            result[field] = novel[field]
-          }
-          return result
-        }),
-      })
+    if (have_fields) {
+      return response.send(novels)
     }
 
     return response.send(novels)
diff --git a/app/Services/NovelService.ts b/app/Services/NovelService.ts
new file mode 100644
index 0000000..12f710a
--- /dev/null
+++ b/app/Services/NovelService.ts
@@ -0,0 +1,82 @@
+import NovelPublishStatus from 'App/Enums/NovelPublishStatus'
+import Novel from 'App/Models/Novel'
+import User from 'App/Models/User'
+
+type NovelOnViewRequestInputs = {
+  page: number
+  take: number
+  tags: number[]
+  fields: string[]
+  filter: string
+  publish_status: NovelPublishStatus
+}
+
+export default class NovelService {
+  public static async onList(user: User, isAdmin: boolean, inputs: NovelOnViewRequestInputs) {
+    const novelsQuery = Novel.query()
+
+    if (!isAdmin) {
+      novelsQuery.where('publish_status', NovelPublishStatus.PUBLISHED)
+    } else {
+      novelsQuery.preload('user')
+
+      if (inputs.publish_status) {
+        novelsQuery.where('novels.publish_status', inputs.publish_status)
+      }
+    }
+
+    if (inputs.filter) {
+      novelsQuery
+        .where('name', 'ilike', `%${inputs.filter}%`)
+        .orWhere('other_names', 'ilike', `%${inputs.filter}%`)
+        .orWhere('shorthand', 'ilike', `%${inputs.filter}%`)
+    }
+
+    if (inputs.tags) {
+      const tags = String(inputs.tags).split(',')
+
+      for (const tag of tags) {
+        novelsQuery.whereExists((query) => {
+          query
+            .select('*')
+            .from('novel_tag')
+            .whereColumn('novel_tag.novel_id', 'novels.id')
+            .where('novel_tag.tag_id', tag)
+        })
+      }
+
+      const novels = await novelsQuery
+        .preload('country')
+        .preload('latest_chapter', (query) => {
+          query.preload('volume')
+        })
+        .orderBy('id', 'desc')
+        .paginate(inputs.page, inputs.take)
+
+      const fields = String(inputs.fields)
+        .split(',')
+        ?.filter((x) => x !== 'context')
+
+      if (fields) {
+        const novelsJSON = novels.toJSON()
+
+        return {
+          have_fields: true,
+
+          novels: {
+            ...novelsJSON,
+            data: novelsJSON.data.map((novel) => {
+              const result = {}
+              for (const field of fields) {
+                result[field] = novel[field]
+              }
+              return result
+            }),
+          },
+        }
+      }
+
+      return { novels, have_fields: false }
+    }
+  }
+}