adds slugify feature for #522

Author: catarak

package-lock.json

@@ -125,6 +125,7 @@
     "s3": "^4.4.0",
     "s3-policy": "^0.2.0",
     "shortid": "^2.2.6",
+    "slugify": "^1.2.9",
     "srcdoc-polyfill": "^0.2.0",
     "url": "^0.11.0",
     "webpack": "^2.6.1",

package.json

@@ -77,13 +77,24 @@ export function updateProject(req, res) {
 }
 
 export function getProject(req, res) {
-  Project.findById(req.params.project_id)
+  const projectId = req.params.project_id;
+  Project.findById(projectId)
     .populate('user', 'username')
-    .exec((err, project) => {
+    .exec((err, project) => { // eslint-disable-line
       if (err) {
         return res.status(404).send({ message: 'Project with that id does not exist' });
+      } else if (!project) {
+        Project.findOne({ slug: projectId })
+        .populate('user', 'username')
+        .exec((innerErr, projectBySlug) => {
+          if (innerErr || !projectBySlug) {
+            return res.status(404).send({ message: 'Project with that id does not exist' });
+          }
+          return res.json(projectBySlug);
+        });
+      } else {
+        return res.json(project);
       }
-      return res.json(project);
     });
 }
 

server/controllers/project.controller.js

@@ -1,5 +1,6 @@
 import mongoose from 'mongoose';
 import shortid from 'shortid';
+import slugify from 'slugify';
 
 const Schema = mongoose.Schema;
 
@@ -25,7 +26,8 @@ const projectSchema = new Schema({
   user: { type: Schema.Types.ObjectId, ref: 'User' },
   serveSecure: { type: Boolean, default: false },
   files: { type: [fileSchema] },
-  _id: { type: String, default: shortid.generate }
+  _id: { type: String, default: shortid.generate },
+  slug: { type: String }
 }, { timestamps: true });
 
 projectSchema.virtual('id').get(function getProjectId() {
@@ -36,4 +38,10 @@ projectSchema.set('toJSON', {
   virtuals: true
 });
 
+projectSchema.pre('save', function generateSlug(next) {
+  const project = this;
+  project.slug = slugify(project.name, '_');
+  return next();
+});
+
 export default mongoose.model('Project', projectSchema);

server/models/project.js

@@ -4,10 +4,6 @@
   header does not match `type`
 */
 const requestsOfType = type => (req, res, next) => {
-  if (process.env.NODE_ENV === 'development') {
-    console.log(req);
-    console.log(req.get('content-type'));
-  }
   if (req.get('content-type') != null && !req.is(type)) {
     if (process.env.NODE_ENV === 'development') {
       console.log('in requests of type error');