fix(posts): handle start param

used legacy code method of checking if start is a parameter, to conditionally set start and page for posts api call
epochtalk · Jul 21, 2021 · f672852 · f672852
1 parent 2ba84e4
commit f672852
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/src/views/Posts.vue b/src/views/Posts.vue
@@ -447,8 +447,8 @@ export default {
   beforeRouteEnter(to, from, next) {
     const params = {
       limit: localStoragePrefs().data.posts_per_page,
-      page: to.query.page || 1,
-      start: to.query.start
+      page: isNaN(to.query.start) ? to.query.page || 1 : undefined,
+      start: isNaN(to.query.start) ? undefined : Number(to.query.start)
     }
     threadsApi.slugToThreadId(to.params.threadSlug).then(t => t.id)
       .then(threadId => {
@@ -465,8 +465,8 @@ export default {
   beforeRouteUpdate(to, from, next) {
     const params = {
       limit: localStoragePrefs().data.posts_per_page,
-      page: to.query.page || 1,
-      start: to.query.start
+      page: isNaN(to.query.start) ? to.query.page || 1 : undefined,
+      start: isNaN(to.query.start) ? undefined : Number(to.query.start)
     }
     threadsApi.slugToThreadId(to.params.threadSlug).then(t => t.id)
       .then(threadId => {
@@ -492,8 +492,8 @@ export default {
         const params = {
           thread_id: threadId,
           limit: v.prefs.posts_per_page,
-          page: $route.query.page || 1,
-          start: $route.query.start
+          page: isNaN($route.query.start) ? $route.query.page || 1 : undefined,
+          start: isNaN($route.query.start) ? undefined : Number($route.query.start)
         }
         return postsApi.byThread(params)
       })