playing with eslint CLI, removed more garbage assets

Author: Jrschellenberg

.eslintrc

@@ -9,5 +9,8 @@
   "parserOptions": {
     "sourceType": "module",
     "allowImportExportEverywhere": true
+  },
+  "rules":{
+    "no-console": 0
   }
 }

src/app.js

@@ -1,15 +1,11 @@
 const express = require('express');
 const path = require('path');
-const favicon = require('serve-favicon');
 const logger = require('morgan');
 const cookieParser = require('cookie-parser');
 const bodyParser = require('body-parser');
 const mongoose = require('mongoose');
 const seeder = require('mongoose-seed');
 const data = require('./data/data.json');
-const User = require('./models/user');
-const Course = require('./models/course');
-const Review = require('./models/review');
 const session = require('express-session');
 const MongoStore = require('connect-mongo')(session);
 
@@ -19,32 +15,33 @@ const dbConfig = config.get('DBHost');
 const course = require('./routes/course');
 const user = require('./routes/user');
 const app = express();
-//process.setMaxListeners(11);
+// process.setMaxListeners(11);
 
-//const env = process.env.NODE_ENV || 'dev';
+// const env = process.env.NODE_ENV || 'dev';
 
-//mongoDb Connection
+// mongoDb Connection
 mongoose.connect(dbConfig);
 
 const db = mongoose.connection;
+
 db.on('error', console.error.bind(console, 'connection error:'));
 
-db.on('connected', function() {
-	seeder.connect(dbConfig, function(){
-		console.log("seeder connected to Database "+dbConfig);
+db.on('connected', function () {
+	seeder.connect(dbConfig, function () {
+		console.log('seeder connected to Database ' + dbConfig);
 		// Load Mongoose models
 		seeder.loadModels([
-			__dirname+'/models/user',
-			__dirname+'/models/course',
-			__dirname+'/models/review'
+			path.join(__dirname, '/models/user'),
+			path.join(__dirname, '/models/course'),
+			path.join(__dirname, '/models/review')
 		]);
 		// Clear specified collections
-		seeder.clearModels(['User', 'Course', 'Review'], function() {
+		seeder.clearModels(['User', 'Course', 'Review'], function () {
 			// Callback to populate DB once collections have been cleared
-			seeder.populateModels(data, function() {
-				console.log("Finished seeding Database!");
-
-				app.emit("appStarted"); //Emits an event to tell our tests it is ok to now test.
+			seeder.populateModels(data, function () {
+				console.log('Finished seeding Database!');
+				
+				app.emit('appStarted'); // Emits an event to tell our tests it is ok to now test.
 			});
 		});
 	});
@@ -55,46 +52,45 @@ app.set('views', path.join(__dirname, 'views'));
 app.set('view engine', 'pug');
 
 // uncomment after placing your favicon in /public
-//app.use(favicon(path.join(__dirname, 'public', 'favicon.ico')));
+// app.use(favicon(path.join(__dirname, 'public', 'favicon.ico')));
 
 app.use(session({
-	secret: "The Session secret, which should not be public. Put into a secrets.js",
+	secret: 'The Session secret, which should not be public. Put into a secrets.js',
 	resave: true,
 	saveUninitialized: false,
-	store: new MongoStore({
-		mongooseConnection: db //db must first be defined.
-	})
+	store: new MongoStore({mongooseConnection: db})
 }));
 
 
-
-if(config.util.getEnv('NODE_ENV') !== 'test') {
+if (config.util.getEnv('NODE_ENV') !== 'test') {
 	app.use(logger('dev'));
 }
 app.use(bodyParser.json());
-app.use(bodyParser.urlencoded({ extended: false }));
+app.use(bodyParser.urlencoded({extended: false}));
 app.use(cookieParser());
 app.use(express.static(path.join(__dirname, 'public')));
 
 app.use('/api/courses', course);
 app.use('/api/users', user);
 
 // catch 404 and forward to error handler
-app.use(function(req, res, next) {
+app.use(function (req, res, next) {
 	const err = new Error('Not Found');
+	
 	err.status = 404;
 	next(err);
 });
 
 // error handler
-app.use(function(err, req, res, next) {
+app.use(function (err, req, res, next) {
 	// set locals, only providing error in development
 	res.locals.message = err.message;
 	res.locals.error = req.app.get('env') === 'development' ? err : {};
 
 	const status = err.status || 500;
 	// render the error page
+	
 	res.status(status).json({success: false, status: status, message: err.message});
 });
 
-module.exports = app; //This for testing...
+module.exports = app; // This for testing...

src/finishSeed.js

@@ -1,9 +1,9 @@
-const server = require("./app");
+const server = require('./app');
 
 let dataBaseFinishSeed = false;
 
 server.on('appStarted', () => {
-	dataBaseFinishSeed = true;
+  dataBaseFinishSeed = true;
 });
 
-export {dataBaseFinishSeed}
+export { dataBaseFinishSeed };

src/middleware/index.js

@@ -1,17 +1,20 @@
 const auth = require('basic-auth');
 const User = require('../models/user');
+
 import Utils from '../utils';
 
-export function authorizeUser(req, res, next) {
-	let user = auth(req);
-	if(!user || !user.name || !user.pass){
-		return Utils.throwError(401, "Access Denied: Please supply login credentials!", next);
-	}
-	User.authenticate(user.name, user.pass, (error, user) => {
-		if(error || !user){
-			return Utils.throwError(401, "Access Denied: Wrong email or password", next);
-		}
-		req.session.userId = user._id;
-		return next();
-	});
-}
+export function authorizeUser (req, res, next) {
+  let user = auth(req);
+
+  if (!user || !user.name || !user.pass) {
+    return Utils.throwError(401, 'Access Denied: Please supply login credentials!', next);
+  }
+  User.authenticate(user.name, user.pass, (error, user) => {
+    if (error || !user) {
+      return Utils.throwError(401, 'Access Denied: Wrong email or password', next);
+    }
+    req.session.userId = user._id;
+
+    return next();
+  });
+}

src/models/course.js

@@ -4,48 +4,43 @@ const User = require('./user');
 const Review = require('./review');
 
 const CourseSchema = new mongoose.Schema({
-	user: {
-		type: Schema.Types.ObjectId,
-		ref: 'User'
-	},
-	title:{
-		type: String,
-		required: true,
-		trim: true
-	},
-	description: {
-		type: String,
-		required: true
-	},
-	estimatedTime: {
-		type: String
-	},
-	materialsNeeded: {
-		type: String
-	},
-	steps: [
-		{
-			stepNumber:{
-				type: Number
-			},
-			title: {
-				type: String,
-				required: true
-			},
-			description: {
-				type: String,
-				required: true
-			}
-		}
-	],
-	reviews: [
-		{
-			type: Schema.Types.ObjectId,
-			ref: 'Review'
-		}
-	]
+  user: {
+    type: Schema.Types.ObjectId,
+    ref: 'User'
+  },
+  title: {
+    type: String,
+    required: true,
+    trim: true
+  },
+  description: {
+    type: String,
+    required: true
+  },
+  estimatedTime: { type: String },
+  materialsNeeded: { type: String },
+  steps: [
+    {
+      stepNumber: { type: Number },
+      title: {
+        type: String,
+        required: true
+      },
+      description: {
+        type: String,
+        required: true
+      }
+    }
+  ],
+  reviews: [
+    {
+      type: Schema.Types.ObjectId,
+      ref: 'Review'
+    }
+  ]
 });
 
 const Course = mongoose.model('Course', CourseSchema);
+
 module.exports = Course;
 

src/models/review.js

@@ -3,25 +3,24 @@ const Schema = mongoose.Schema;
 const User = require('./user');
 
 const ReviewSchema = new Schema({
-	user: {
-		type: Schema.Types.ObjectId,
-		ref: 'User'
-	},
-	postedOn: {
-		type: Date,
-		default: Date.now
-	},
-	rating: {
-		type: Number,
-		required: true,
-		min: 1,
-		max: 5
-	},
-	review: {
-		type: String
-	}
+  user: {
+    type: Schema.Types.ObjectId,
+    ref: 'User'
+  },
+  postedOn: {
+    type: Date,
+    default: Date.now
+  },
+  rating: {
+    type: Number,
+    required: true,
+    min: 1,
+    max: 5
+  },
+  review: { type: String }
 });
 
 const Review = mongoose.model('Review', ReviewSchema);
+
 module.exports = Review;