use switch for pagination

79f6e08c · Regis · 6bf0bd0f · 79f6e08c
Commit 79f6e08c authored Jan 06, 2017 by Regis
Hide whitespace changes
Inline Side-by-side

Showing with 18 additions and 12 deletions

app/assets/javascripts/vue_pagination/index.js.es6 app/assets/javascripts/vue_pagination/index.js.es6 +18 -12

No files found.
--- a/app/assets/javascripts/vue_pagination/index.js.es6
+++ b/app/assets/javascripts/vue_pagination/index.js.es6
@@ -66,18 +66,24 @@
        const text = e.target.innerText;
        const { totalPages, nextPage, previousPage } = this.pageInfo;
-        if (text === SPREAD) {
+        switch (text) {
-          return;
+          case SPREAD:
-        } else if (text === LAST) {
+            break;
-          pageNum = totalPages;
+          case LAST:
-        } else if (text === NEXT) {
+            pageNum = totalPages;
-          pageNum = nextPage;
+            break;
-        } else if (text === PREV) {
+          case NEXT:
-          pageNum = previousPage;
+            pageNum = nextPage;
-        } else if (text === FIRST) {
+            break;
-          pageNum = 1;
+          case PREV:
-        } else {
+            pageNum = previousPage;
-          pageNum = +text;
+            break;
+          case FIRST:
+            pageNum = 1;
+            break;
+          default:
+            pageNum = +text;
+            break;
        }
        this.change(pageNum, apiScope);