From a3fa59e226b45e67b3796ebb5b4a584c31fa38f7 Mon Sep 17 00:00:00 2001
From: Andrew Davidson <andrew@amdavidson.com>
Date: Sat, 16 Jan 2016 23:17:24 -0500
Subject: [PATCH] Working on replacing scattered post model with a cohesive one
 and mongodb support ( #6 ), some of this will also support issue #7 as well,
 but right now the admin page is broken and post generating is broken.

---
 app.js                     | 163 ++++++++++++++++---------------------
 db.js                      |  21 -----
 genPhotos.js               |   1 -
 genPosts.js                |   1 -
 helper.js                  |  61 --------------
 package.json               |   2 +
 schemas.js                 |  28 ++++++-
 user.js                    |   2 +-
 views/admin-post-edit.jade |  14 ++--
 views/admin-post-list.jade |  28 +++----
 views/admin-post-new.jade  |   6 +-
 views/admin-post-view.jade |  14 ++--
 12 files changed, 132 insertions(+), 209 deletions(-)
 delete mode 100644 helper.js

diff --git a/app.js b/app.js
index ea2a43c..f1d1272 100644
--- a/app.js
+++ b/app.js
@@ -8,15 +8,13 @@ var multer = require('multer');
 var fs = require('fs');
 var markdown = require( "markdown" ).markdown;
 
-
 // Get connected to our database
 var MongoClient = require('mongodb').MongoClient
 
 Post = require('./post.js');
 User = require('./user.js');
-var helper = require('./helper.js');
+Category = require('./category.js');
 var database = require('./db.js');
-var genPosts = require('./genPosts.js');
 var genStatic = require('./genStatic.js');
 var genPhotos = require('./genPhotos.js');
 
@@ -102,28 +100,21 @@ app.get('/admin/post/list/:start?',
         } else {
             var start = 0;
         }
-        database.listPosts(count, start, function(posts){
-            for (post in posts) {
-                var date = new Date(posts[post].postDate);
-                posts[post].dateString = date.getFullYear() + '-' +
-                    ("0" + (date.getMonth()+1)).slice(-2) + '-' +
-                    ("0" + date.getDate()).slice(-2);
-            }
+        Post.getPosts(count, start, function(err, posts){
             res.render('admin-post-list', {posts, user: req.user});
         });
 }
 );
 
-app.get('/admin/post/view/:id?',
+app.get('/admin/post/view/:uuid?',
     function (req, res, next) {
-        if (req.params.id) {
-            database.getPostById(req.params.id, function(row) {
+        if (req.params.uuid) {
+            Post.getByUUID(req.params.uuid, function(err, post) {
                 res.render('admin-post-view', {
                     successNotice: req.flash('successNotice'),
                     failureNotice: req.flash('failureNotice'),
-                    date: helper.epochToShortDateString(row.postDate),
-                    post: row,
-                    content: markdown.toHTML(row.markdown),
+                    post: post,
+                    content: markdown.toHTML(post.get("markdown")),
                     user: req.user
                 })
             })
@@ -136,104 +127,92 @@ app.get('/admin/post/view/:id?',
 
 app.get('/admin/post/new',
     function(req, res, next) {
-        database.listCategories(function(rows){
-            res.render('admin-post-new', { categories: rows, user: req.user });
+        Category.getCategories(null, null, function (err, categories) {
+            res.render('admin-post-new', { categories: categories, user: req.user });
         });
-}
+    }
 );
 
 app.post('/admin/post/new',
     function(req, res, next) {
-        var title = req.body.title;
+        var post = new Post();
+        post.set("title", req.body.title);
         if (req.body.slug != "") {
-            var slug = req.body.slug;
+            post.set("slug", req.body.slug);
         } else {
-            var slug = helper.makeSlug(title);
+            post.set("slug", post.makeSlug());
         }
-        var markdown = req.body.markdown;
         if (req.body.postDate != "") {
-            var postDate = helper.dateToEpoch(new Date(req.body.postDate));
-        } else {
-            var postDate = helper.dateToEpoch(new Date());
+            post.set("postDate", new Date(req.body.postDate));
+            post.set("updatedDate", new Date(req.body.postDate));
         }
-        var updatedDate = postDate;
-        var createDate = helper.dateToEpoch(new Date());
-        var categoryName = req.body.category;
-        var tags = helper.parseTags(req.body.tags);
+        post.tagPost(req.body.tags);
+        post.set("markdown", req.body.markdown);
 
-        database.createPost(title, slug, markdown, postDate, updatedDate, createDate, function(err, row) {
-            database.setPostCategory(row.id, categoryName, function(category) {
-                database.tagPost(row.id, tags);
-                req.flash('successNotice', 'Post created.');
-                res.redirect('/admin/post/edit/'+row.id);
+        Category.getByName(req.body.category, function (err, category) {
+            post.set("category", category.get("uuid"));
+            console.log(post);
+            post.save(function (err) {
+                if (!err) {
+                    req.flash('successNotice', 'Post created.');
+                    res.redirect('/admin/post/edit/'+post.get("uuid"));
+                }
+                else {
+                    req.flash('failureNotice', 'Post creation failed');
+                    res.redirect('/admin/post/new/');
+                }
             });
         });
-}
+    }
 );
 
-app.get('/admin/post/edit/:id',
+app.get('/admin/post/edit/:uuid',
     function(req, res, next) {
-        var id = req.params.id;
-        async.parallel({
-            categories: function(callback) {
-                database.listCategories(function(categories) {
-                    callback(null, categories);
-                })
-            },
-            postCategory: function(callback) {
-                database.getPostCategory(id, function(category) {
-                    callback(null, category);
-                })
-            },
-            postTags: function(callback) {
-                database.getPostTagsAsString(id, function(tagString) {
-                    callback(null, tagString);
-                })
-            },
-            post: function(callback) {
-                database.getPostById(id, function(post) {
-                    callback(null, post);
-                })
-            }
-        },
-        function(err, results) {
-            if (results.postCategory) {
-                var categoryName = results.postCategory.name;
-            } else {
-                var categoryName = null;
-            }
-            res.render('admin-post-edit', {
-                successNotice: req.flash('successNotice'),
-                failureNotice: req.flash('failureNotice'),
-                categories: results.categories,
-                postCategory: categoryName,
-                post: results.post,
-                postTags: results.postTags,
-                formattedDate: helper.epochToDateString(results.post.postDate),
-                user: req.user
+        Post.getByUUID(req.params.uuid, function(err, post) {
+            Category.getByUUID(post.get("category"), function(err, category) {
+                Category.getCategories(null, null, function(err, categories) {
+                    var tags = ""
+                    for (i=0; i<post.get("tags").length; i++) {
+                        tags += post.get("tags")[i].name + " ";
+                    }
+                    res.render('admin-post-edit', {
+                        successNotice: req.flash('successNotice'),
+                        failureNotice: req.flash('failureNotice'),
+                        categories: categories,
+                        category: category,
+                        post: post,
+                        postTags: tags,
+                        user: req.user
+                    });
+                });
             });
         });
-}
+    }
 );
 
-app.post('/admin/post/edit/:id',
+app.post('/admin/post/edit/:uuid',
     function(req, res, next) {
-        var id = req.params.id;
-        var title = req.body.title;
-        var slug = req.body.slug;
-        var markdown = req.body.markdown;
-        var postDate = helper.dateToEpoch(new Date(req.body.postDate));
-        var categoryName = req.body.category;
-        var tags = helper.parseTags(req.body.tags);
-        console.log('Post '+id+' update request received');
-        database.tagPost(id, tags);
-        database.updatePost(id, title, slug, markdown, postDate, function(err, row) {
-            database.setPostCategory(id, categoryName, function(err) {
-                req.flash('successNotice', 'Post updated.');
-                res.redirect('/admin/post/edit/'+id);
+        Post.getByUUID(req.params.uuid, function(err, post) {
+            post.set("title", req.body.title);
+            post.set("slug", req.body.slug);
+            post.set("markdown", req.body.markdown);
+            post.set("postDate", new Date(req.body.postDate));
+            post.tagPost(req.body.tags);
+            Category.getByName(req.body.category, function(err, category) {
+                post.set("category", category.get("uuid"));
+                post.save(function(err) {
+                    if (!err) {
+                        req.flash("successNotice", "Post updated.");
+                        res.redirect("/admin/post/edit/"+post.get("uuid"));
+                    }
+                    else {
+                        req.flash("failureNotice", "Post failed to update.");
+                        res.redirect("/admin/post/edit/"+post.get("uuid"));
+                    }
+                });
             });
         });
-}
+    }
 );
 
 app.get('/admin/post/regenerate/:id?',
@@ -430,7 +409,7 @@ app.get('/admin',
                     });
                 },
                 posts: function(callback) {
-                    database.countPosts(function(count) {
+                    Post.countPosts(function(err, count) {
                         callback(null, count);
                     });
                 },
diff --git a/db.js b/db.js
index 7d920bb..cac3b04 100644
--- a/db.js
+++ b/db.js
@@ -2,7 +2,6 @@ var sqlite = require('sqlite3').verbose();
 var async = require('async');
 
 var config = require('./config.js').config;
-var helper = require('./helper.js');
 
 var db = new sqlite.Database(config.dbPath);
 
@@ -34,14 +33,6 @@ exports.getLastUploadDate = function(cb) {
     });
 }
 
-// Function to get a count of current posts
-// Returns count of posts
-exports.countPosts = function(cb) {
-    db.get('SELECT count(id) as count FROM posts;', function(err, count) {
-        cb(count.count);
-    });
-}
-
 // Function to get a count of current galleries
 // Returns count of galleries
 exports.countGalleries = function(cb) {
@@ -82,18 +73,6 @@ exports.listCategories = function(cb) {
     });
 }
 
-// Function to get a list of posts of a certain count and starting at an offset
-// Returns a list of post objects
-exports.listPosts = function(count, start, cb) {
-    db.all('SELECT * FROM posts WHERE deleted = 0 \
-        ORDER BY postDate DESC, title ASC \
-        LIMIT '+count+' OFFSET '+start+';',
-        function(err, posts) {
-            console.log(err);
-            cb(posts);
-        });
-}
-
 // Function to get a list of photos of a certain count and starting at an offset
 // Returns a list of photo objects
 exports.listPhotos = function(count, start, cb) {
diff --git a/genPhotos.js b/genPhotos.js
index 8d667a7..de8fe01 100644
--- a/genPhotos.js
+++ b/genPhotos.js
@@ -7,7 +7,6 @@ var jade = require('jade');
 
 var config = require('./config.js').config;
 var db = require('./db.js');
-var helper = require('./helper.js');
 
 var generatePhotoSizesById = function generatePhotoSizesById(id, cb) {
     db.getPhotoById(id, function (photo) {
diff --git a/genPosts.js b/genPosts.js
index c458269..7e52cca 100644
--- a/genPosts.js
+++ b/genPosts.js
@@ -4,7 +4,6 @@ var jade = require('jade');
 var markdown = require( "markdown" ).markdown;
 var async = require('async');
 
-var helper = require('./helper.js');
 var db = require('./db.js');
 
 // Generate all archive pages associated with a particular post ID
diff --git a/helper.js b/helper.js
deleted file mode 100644
index ed959af..0000000
--- a/helper.js
+++ /dev/null
@@ -1,61 +0,0 @@
-// Helper functions
-var crypto = require('crypto');
-
-// sha256 hex-hashes password with supplied salt.
-// Returns hex-hash string
-exports.hashPassword = function(password, salt) {
-  var hash = crypto.createHash('sha256');
-  hash.update(password);
-  hash.update(salt);
-  return hash.digest('hex');
-}
-
-// Convert date() to epoch
-exports.dateToEpoch = function(date) {
-    return Math.floor(date);
-};
-
-// Convert epoch (in seconds) to date string
-exports.epochToDateString = function(epoch) {
-    var date = new Date(epoch);
-    return date.toLocaleString();
-};
-
-// Convert epoch to an ISO formatted date.
-exports.epochToShortDateString = function(epoch) {
-    var date = new Date(epoch);
-    return date.toISOString().slice(0,10);
-}
-
-// Build a slug from a title string
-// Returns the slug as a string
-exports.makeSlug = function(str) {
-      str = str.replace(/^\s+|\s+$/g, ''); // trim
-      str = str.toLowerCase();
-      str = str.trim();
-
-      // remove accents, swap ñ for n, etc
-      var from = "àáäâèéëêìíïîòóöôùúüûñç·/_,:;&";
-      var to   = "aaaaeeeeiiiioooouuuunc-------";
-      for (var i=0, l=from.length ; i<l ; i++) {
-        str = str.replace(new RegExp(from.charAt(i), 'g'), to.charAt(i));
-      }
-
-      str = str.replace(/[^a-z0-9 -]/g, '') // remove invalid chars
-        .replace(/\s+/g, '-') // collapse whitespace and replace by -
-        .replace(/-+/g, '-'); // collapse dashes
-
-      return str;
-};
-
-// Parse a string of tags
-// Returns a list of tags
-exports.parseTags = function(str) {
-    str = str.replace(/,/g,'');
-    var list = str.split(" ");
-    for (var i = 0, size = list.length; i < size; i++) {
-        list[i] = list[i].trim();
-    }
-    console.log('tag parse '+list);
-    return list
-}
diff --git a/package.json b/package.json
index 8b7314e..f259efd 100644
--- a/package.json
+++ b/package.json
@@ -15,10 +15,12 @@
     "jade": "^1.11.0",
     "lodash": "^4.0.0",
     "markdown": "^0.5.0",
+    "moment": "^2.11.1",
     "mongodb": "^2.1.4",
     "morgan": "^1.6.1",
     "multer": "^1.1.0",
     "ncp": "^2.0.0",
+    "node-uuid": "^1.4.7",
     "passport": "^0.3.0",
     "passport-local": "^1.0.0",
     "serve-favicon": "^2.3.0",
diff --git a/schemas.js b/schemas.js
index a8092ea..61eab4d 100644
--- a/schemas.js
+++ b/schemas.js
@@ -1,9 +1,35 @@
+var uuid = require('node-uuid');
+
 schemas  = {
     user: {
         username: null,
         password: null,
         hash: null,
-        email: null
+        email: null,
+        displayName: null
+    },
+    post: {
+        uuid: uuid.v1(),
+        title: null,
+        slug: null,
+        url: null,
+        markdown: null,
+        createdDate: new Date(),
+        postDate: new Date(),
+        updatedDate: new Date(),
+        deleted: false,
+        published: false,
+        category: null,
+        tags: [],
+        lastGenerateDate: 0,
+        lastUpload: 0
+    },
+    category: {
+        uuid: uuid.v1(),
+        name: null,
+        slug: null,
+        description: null,
+        createdDate: new Date()
     }
 }
 
diff --git a/user.js b/user.js
index 11eee60..fb5e062 100644
--- a/user.js
+++ b/user.js
@@ -50,7 +50,7 @@ User.verify = function (username, password, callback) {
             console.log("Username "+username+" exists");
             console.log(username + "'s salt is " + doc.salt);
             var hash = hashPassword(password, doc.salt);
-            console.log(username+"'s hashed password is "+hash);
+            console.log(username+" is loging in with a hashed password of "+hash);
             db.collection("users").findOne({username: username, password: hash},
                 function(err, doc) {
                     callback(err, doc);
diff --git a/views/admin-post-edit.jade b/views/admin-post-edit.jade
index 7949b26..452fb8b 100644
--- a/views/admin-post-edit.jade
+++ b/views/admin-post-edit.jade
@@ -8,29 +8,29 @@ block content
             h1(class="page-header") Edit Post
 
             include ./admin-messages.jade
-            
+
             form(method="post", action="/admin/post/edit/#{post.id}")
                 div(class="row page-header")
                     div(class="input-group input-group-lg col-xs-10 col-xs-offset-1 col-md-10 col-md-offset-1")
-                        input(class="form-control", type="text", name="title", placeholder="Enter title", value="#{post.title}")
+                        input(class="form-control", type="text", name="title", placeholder="Enter title", value="#{post.get('title')}")
                     div(class="input-group col-xs-10 col-xs-offset-1 col-md-10 col-md-offset-1")
-                        input(class="form-control", type="text", name="slug", placeholder="Enter slug (optional)", value="#{post.slug}")
+                        input(class="form-control", type="text", name="slug", placeholder="Enter slug (optional)", value="#{post.get('slug')}")
                 div(class="row page-header")
                     div(class="input-group col-xs-10 col-xs-offset-1 col-md-10 col-md-offset-1")
                         div(class="input-group-btn")
                             button(type="button", class="btn btn-primary dropdown-toggle", data-toggle="dropdown", aria-haspopup="true", aria-expanded="false")  Category
                             ul(class="dropdown-menu")
                                 each row in categories
-                                    li: a(href="#", onClick="document.getElementById('category').value = '#{row.name}'") #{row.name}
-                        input(type="text", class="form-control", aria-label="Post category", name="category", id="category" value="#{categoryName}")
+                                    li: a(href="#", onClick="document.getElementById('category').value = '#{row.data.name}'") #{row.get("name")}
+                        input(type="text", class="form-control", aria-label="Post category", name="category", id="category" value="#{category.get('name')}")
                     div(class="input-group col-xs-10 col-xs-offset-1 col-md-10 col-md-offset-1")
-                        input(type="text", class="form-control", placeholder="Publish Date", name="postDate", value="#{formattedDate}")
+                        input(type="text", class="form-control", placeholder="Publish Date", name="postDate", value="#{post.get('postDate')}")
                     div(class="input-group col-xs-10 col-xs-offset-1 col-md-10 col-md-offset-1")
                         input(type="text", class="form-control", placeholder="Tags", name="tags" value="#{postTags}")
                 div(class="row page-header")
                     div(class="input-group col-xs-10 col-xs-offset-1 col-md-10 col-md-offset-1")
                         textarea(class="form-control", rows="12", name="markdown")
-                            | #{post.markdown}
+                            | #{post.get("markdown")}
                 div(class="row")
                     div(class="input-group col-xs-10 col-xs-offset-1 col-md-10 col-md-offset-1")
                         div(class="pull-left")
diff --git a/views/admin-post-list.jade b/views/admin-post-list.jade
index 912eb79..032156f 100644
--- a/views/admin-post-list.jade
+++ b/views/admin-post-list.jade
@@ -6,22 +6,22 @@ block content
 
         div(class="col-sm-10 col-sm-offset-2 main")
             h1(class="page-header") Posts
-            
+
             include ./admin-messages.jade
-            
+
             table(class="table table-striped")
                 each post in posts
                     tr
-                        td #{post.id}
-                        td: a(href="/admin/post/view/#{post.id}") #{post.title}
-                        td #{post.dateString}
+                        td #{post.get("uuid")}
+                        td: a(href="/admin/post/view/#{post.get('uuid')}") #{post.get("title")}
+                        td #{post.getShortDate()}
                         td
-                            a(href="/admin/post/edit/#{post.id}") Edit
-                            |  - 
-                            a(href="/admin/post/delete/#{post.id}") Delete
-                            |  - 
-                            a(href="/admin/post/regenerate/#{post.id}") Regenerate
-                            |  - 
-                            a(href="/admin/post/publish/#{post.id}") Publish
-                        
-                            
+                            a(href="/admin/post/edit/#{post.get('uuid')}")  Edit
+                            |  -
+                            a(href="/admin/post/delete/#{post.get('uuid')}")  Delete
+                            |  -
+                            a(href="/admin/post/regenerate/#{post.get('uuid')}")  Regenerate
+                            |  -
+                            a(href="/admin/post/publish/#{post.get('uuid')}")  Publish
+
+
diff --git a/views/admin-post-new.jade b/views/admin-post-new.jade
index 31630e4..5a54d6e 100644
--- a/views/admin-post-new.jade
+++ b/views/admin-post-new.jade
@@ -3,7 +3,7 @@ extends admin-layout
 block content
     div(class="row")
         include ./admin-sidebar.jade
-        
+
         div(class="col-sm-10 col-sm-offset-2 main")
             h1(class="page-header") New Post
 
@@ -20,8 +20,8 @@ block content
                         div(class="input-group-btn")
                             button(type="button", class="btn btn-primary dropdown-toggle", data-toggle="dropdown", aria-haspopup="true", aria-expanded="false")  Category
                             ul(class="dropdown-menu")
-                                each row in categories
-                                    li: a(href="#", onClick="document.getElementById('category').value = '#{row.name}'") #{row.name}
+                                each doc in categories
+                                    li: a(href="#", onClick="document.getElementById('category').value = '#{doc.data.name}'") #{doc.data.name}
                         input(type="text", class="form-control", aria-label="Post category", name="category", id="category", value="Uncategorized")
                     div(class="input-group col-xs-10 col-xs-offset-1 col-md-10 col-md-offset-1")
                         input(type="text", class="form-control", placeholder="Publish Date", name="postDate")
diff --git a/views/admin-post-view.jade b/views/admin-post-view.jade
index b953622..3a89970 100644
--- a/views/admin-post-view.jade
+++ b/views/admin-post-view.jade
@@ -3,18 +3,18 @@ extends admin-layout
 block content
     div(class="row")
         include ./admin-sidebar.jade
-        
+
         div(class="col-sm-10 col-sm-offset-2 main")
 
             include ./admin-messages.jade
 
             div(class='page-header row')
-                h3 #{post.title}
-                
+                h3 #{post.get('title')}
+
             div(class='row')
-                | #{date} -- 
-                a(href="/admin/post/edit/#{post.id}") Edit
-                |  - 
-                a(href="/admin/post/regenerate/#{post.id}") Regenerate
+                | #{post.getShortDate()} --
+                a(href="/admin/post/edit/#{post.get('uuid')}") Edit
+                |  -
+                a(href="/admin/post/regenerate/#{post.get('uuid')}") Regenerate
 
             div(class='row') !{content}