fix(express): import connect-mongo/es5 if node < 4

closes #1844

Author: Awk34

Diff for: package.json

@@ -46,6 +46,7 @@
     "gulp-filter": "^4.0.0",
     "insight": "~0.8.1",
     "lodash": "^4.6.1",
+    "semver": "^5.1.0",
     "underscore.string": "^3.1.1",
     "yeoman-generator": "~0.23.3",
     "yeoman-welcome": "^1.0.1"

Diff for: src/generators/app/index.js

@@ -2,6 +2,7 @@
 
 import fs from 'fs';
 import path from 'path';
+import { runCmd } from '../util';
 import chalk from 'chalk';
 import {Base} from 'yeoman-generator';
 import {genBase} from '../generator-base';
@@ -10,6 +11,7 @@ import {exec} from 'child_process';
 import babelStream from 'gulp-babel';
 import beaufityStream from 'gulp-beautify';
 import filter from 'gulp-filter';
+import semver from 'semver';
 
 export class Generator extends Base {
   constructor(...args) {
@@ -33,32 +35,30 @@ export class Generator extends Base {
   get initializing() {
     return {
       init: function () {
-        var cb = this.async();
-
         this.config.set('generatorVersion', this.rootGeneratorVersion());
         this.filters = {};
 
         // init shared generator properies and methods
         genBase(this);
 
+        insight.track('generator', this.rootGeneratorVersion());
+        this.nodeVersion = semver.clean(process.version);
+        this.semver = semver;
+        insight.track('node', this.nodeVersion);
+        insight.track('platform', process.platform);
+
         if(process.env.CI) {
           insight.optOut = true;
-          return cb();
         } else if(insight.optOut === undefined) {
           insight.askPermission(null, cb);
-        } else {
-          return cb();
         }
+
+        return runCmd('npm --version').then(stdout => {
+          this.npmVersion = stdout.toString().trim();
+          return insight.track('npm', this.npmVersion);
+        });
       },
       info: function () {
-        insight.track('generator', this.rootGeneratorVersion());
-        insight.track('node', process.version);
-        exec('npm --version', (err, stdin, stderr) => {
-          if(err || stderr.length > 0) return insight.track('npm', 'error');
-          else return insight.track('npm', stdin.toString().trim());
-        });
-        insight.track('platform', process.platform);
-
         this.log(this.yoWelcome);
         this.log('Out of the box I create an AngularJS app with an Express server.\n');
       },

Diff for: src/generators/util.js

@@ -3,6 +3,34 @@
 import path from 'path';
 import fs from 'fs';
 import glob from 'glob';
+import Promise from 'bluebird';
+import {exec} from 'child_process';
+
+const DEBUG = !!process.env.DEBUG;
+
+/**
+ * @callback doneCallback
+ * @param {null|Error} err
+ */
+
+/**
+ * Run the given command in a child process
+ * @param {string} cmd - command to run
+ * @returns {Promise}
+ */
+export function runCmd(cmd) {
+  return new Promise((resolve, reject) => {
+    exec(cmd, {}, function(err, stdout, stderr) {
+      if(err) {
+        console.error(stdout);
+        return reject(err);
+      } else {
+        if(DEBUG) console.log(`${cmd} stdout: ${stdout}`);
+        return resolve(stdout);
+      }
+    });
+  });
+}
 
 function expandFiles(pattern, options) {
   options = options || {};

Diff for: templates/app/server/config/express.js

@@ -17,7 +17,9 @@ import lusca from 'lusca';
 import config from './environment';<% if (filters.auth) { %>
 import passport from 'passport';<% } %>
 import session from 'express-session';<% if (filters.mongoose) { %>
-import connectMongo from 'connect-mongo';
+<%_ if(semver.satisfies(nodeVersion, '>= 4')) { _%>
+import connectMongo from 'connect-mongo';<% } else { _%>
+import connectMongo from 'connect-mongo/es5';<% } %>
 import mongoose from 'mongoose';
 var MongoStore = connectMongo(session);<% } else if(filters.sequelize) { %>
 import sqldb from '../sqldb';