Merge pull request #755 from weseek/support/refactor-installer

Support/refactor installer

src/server/routes/installer.js

@@ -35,50 +35,49 @@ module.exports = function(crowi, app) {
   actions.createAdmin = function(req, res, next) {
     const registerForm = req.body.registerForm || {};
 
-    if (req.form.isValid) {
-      var name = registerForm.name;
-      var username = registerForm.username;
-      var email = registerForm.email;
-      var password = registerForm.password;
-      var language = registerForm['app:globalLang'] || (req.language || 'en-US');
+    if (!req.form.isValid) {
+      return res.render('installer');
+    }
 
-      User.createUserByEmailAndPassword(name, username, email, password, language, function(err, userData) {
-        if (err) {
-          req.form.errors.push('管理ユーザーの作成に失敗しました。' + err.message);
-          // TODO
-          return res.render('installer');
-        }
+    const name = registerForm.name;
+    const username = registerForm.username;
+    const email = registerForm.email;
+    const password = registerForm.password;
+    const language = registerForm['app:globalLang'] || 'en-US';
 
-        userData.makeAdmin(function(err, userData) {
-          Config.applicationInstall(function(err, configs) {
-            if (err) {
-            logger.error(err);
-              return;
-            }
+    User.createUserByEmailAndPassword(name, username, email, password, language, function(err, userData) {
+      if (err) {
+        req.form.errors.push('管理ユーザーの作成に失敗しました。' + err.message);
+        // TODO
+        return res.render('installer');
+      }
 
-            // save the globalLang config, and update the config cache
-            Config.updateNamespaceByArray('crowi', {'app:globalLang': language}, function(err, config) {
-              Config.updateConfigCache('crowi', config);
-            });
+      userData.makeAdmin(function(err, userData) {
+        Config.applicationInstall(function(err, configs) {
+          if (err) {
+            logger.error(err);
+            return;
+          }
 
-            // login with passport
-            req.logIn(userData, (err) => {
-              if (err) { return next() }
-              else {
-                req.flash('successMessage', 'GROWI のインストールが完了しました！はじめに、このページで各種設定を確認してください。');
-                return res.redirect('/admin/app');
-              }
-            });
+          // save the globalLang config, and update the config cache
+          Config.updateNamespaceByArray('crowi', {'app:globalLang': language}, function(err, config) {
+            Config.updateConfigCache('crowi', config);
           });
 
-          // create initial pages
-          createInitialPages(userData, language);
+          // login with passport
+          req.logIn(userData, (err) => {
+            if (err) { return next() }
+            else {
+              req.flash('successMessage', 'GROWI のインストールが完了しました！はじめに、このページで各種設定を確認してください。');
+              return res.redirect('/admin/app');
+            }
+          });
         });
+
+        // create initial pages
+        createInitialPages(userData, language);
       });
-    }
-    else {
-      return res.render('installer');
-    }
+    });
   };
 
   return actions;