From 763bf4b903c6d96b6be62313e1b531aa1fd4788d Mon Sep 17 00:00:00 2001
From: Nicolas Giard <github@ngpixel.com>
Date: Sun, 11 Nov 2018 13:54:33 -0500
Subject: [PATCH] feat: sidebar rendering

---
 .../themes/default/components/nav-sidebar.vue   | 17 ++---------------
 client/themes/default/components/page.vue       |  1 +
 server/controllers/common.js                    |  3 ++-
 server/graph/resolvers/navigation.js            |  1 +
 server/models/navigation.js                     | 11 ++++++++++-
 server/setup.js                                 |  2 ++
 server/views/page.pug                           | 13 +++++++++++++
 7 files changed, 31 insertions(+), 17 deletions(-)

diff --git a/client/themes/default/components/nav-sidebar.vue b/client/themes/default/components/nav-sidebar.vue
index 8a4c311d..59651f6a 100644
--- a/client/themes/default/components/nav-sidebar.vue
+++ b/client/themes/default/components/nav-sidebar.vue
@@ -1,19 +1,6 @@
 <template lang="pug">
-  v-list(dense, :class='color', :dark='dark')
-    v-list-tile.pt-2(href='/')
-      v-list-tile-avatar: v-icon home
-      v-list-tile-title Home
-    v-divider.my-2
-    v-subheader.pl-4 Navigation
-    v-list-tile
-      v-list-tile-avatar: v-icon stars
-      v-list-tile-title The Universe
-    v-list-tile
-      v-list-tile-avatar: v-icon directions_boat
-      v-list-tile-title Ships
-    v-list-tile
-      v-list-tile-avatar: v-icon local_airport
-      v-list-tile-title Airports
+  v-list.py-2(dense, :class='color', :dark='dark')
+    slot
 </template>
 
 <script>
diff --git a/client/themes/default/components/page.vue b/client/themes/default/components/page.vue
index a6376c1c..a750f45d 100644
--- a/client/themes/default/components/page.vue
+++ b/client/themes/default/components/page.vue
@@ -12,6 +12,7 @@
       v-model='navShown'
       )
       nav-sidebar
+        slot(name='sidebar')
 
     v-content
       v-toolbar(color='grey lighten-3', flat, dense)
diff --git a/server/controllers/common.js b/server/controllers/common.js
index 976a64e9..835b81d1 100644
--- a/server/controllers/common.js
+++ b/server/controllers/common.js
@@ -75,7 +75,8 @@ router.get('/*', async (req, res, next) => {
     isPrivate: false
   })
   if (page) {
-    res.render('page', { page })
+    const sidebar = await WIKI.models.navigation.getTree({ cache: true })
+    res.render('page', { page, sidebar })
   } else if (pageArgs.path === 'home') {
     res.render('welcome')
   } else {
diff --git a/server/graph/resolvers/navigation.js b/server/graph/resolvers/navigation.js
index f56ab4d9..7034fa45 100644
--- a/server/graph/resolvers/navigation.js
+++ b/server/graph/resolvers/navigation.js
@@ -21,6 +21,7 @@ module.exports = {
         await WIKI.models.navigation.query().patch({
           config: args.tree
         }).where('key', 'site')
+        await WIKI.redis.set('nav:sidebar', JSON.stringify(args.tree), 'EX', 300)
 
         return {
           responseResult: graphHelper.generateSuccess('Navigation updated successfully')
diff --git a/server/models/navigation.js b/server/models/navigation.js
index a035709a..55ca4e17 100644
--- a/server/models/navigation.js
+++ b/server/models/navigation.js
@@ -21,9 +21,18 @@ module.exports = class Navigation extends Model {
     }
   }
 
-  static async getTree() {
+  static async getTree({ cache = false } = {}) {
+    if (cache) {
+      const navTreeCached = await WIKI.redis.get('nav:sidebar')
+      if (navTreeCached) {
+        return JSON.parse(navTreeCached)
+      }
+    }
     const navTree = await WIKI.models.navigation.query().findOne('key', 'site')
     if (navTree) {
+      if (cache) {
+        await WIKI.redis.set('nav:sidebar', JSON.stringify(navTree.config), 'EX', 300)
+      }
       return navTree.config
     } else {
       WIKI.logger.warn('Site Navigation is missing or corrupted.')
diff --git a/server/setup.js b/server/setup.js
index b1446c35..01da3084 100644
--- a/server/setup.js
+++ b/server/setup.js
@@ -1,4 +1,5 @@
 const path = require('path')
+const uuid = require('uuid/v4')
 
 /* global WIKI */
 
@@ -209,6 +210,7 @@ module.exports = () => {
         key: 'site',
         config: [
           {
+            id: uuid(),
             icon: 'home',
             kind: 'link',
             label: 'Home',
diff --git a/server/views/page.pug b/server/views/page.pug
index 6743b53f..d9b9318d 100644
--- a/server/views/page.pug
+++ b/server/views/page.pug
@@ -16,4 +16,17 @@ block body
       :author-id=page.authorId
       :is-published=page.isPublished
       )
+      template(slot='sidebar')
+        each navItem in sidebar
+          if navItem.kind === 'link'
+            v-list-tile(
+              href=navItem.target
+              )
+              v-list-tile-avatar
+                v-icon= navItem.icon
+              v-list-tile-title= navItem.label
+          else if navItem.kind === 'divider'
+            v-divider.my-2
+          else if navItem.kind === 'header'
+            v-subheader.pl-4= navItem.label
       template(slot='contents')!= page.render