fix more styling issue

Author: im6

server/config/env.js

@@ -1,9 +1,7 @@
-"use strict";
-
-const env = process.env;
+const { env } = process;
 
 module.exports = {
   isDev: env.NODE_ENV === 'dev',
   port: env.PORT || 8080,
   ip: 'localhost',
-};
+};

server/resource/mysqlConnection.js

@@ -1,28 +1,24 @@
-var mysql = require('mysql');
+const mysql = require('mysql');
 
-var pool = mysql.createPool({
-  connectionLimit : 2,
-  host     : process.env['SQL_HOST'],
-  port     : process.env['SQL_PORT'],
-  user     : process.env['SQL_USERNAME'],
-  password : process.env['SQL_PASSWORD'],
-  database : process.env['SQL_DATABASE']
+const pool = mysql.createPool({
+  connectionLimit: 2,
+  host: process.env.SQL_HOST,
+  port: process.env.SQL_PORT,
+  user: process.env.SQL_USERNAME,
+  password: process.env.SQL_PASSWORD,
+  database: process.env.SQL_DATABASE,
 });
 
 module.exports = {
-  getPool: function(){
-    return pool;
-  },
-  sqlExecOne: function(qr){
-    return new Promise(function(resolve, reject){
-      pool.query(qr, function(err, rows, fields){
-        if(err){
-          console.error(err.message);
-          reject(err);
-        }else{
-          resolve(rows);
-        }
-      });
+  getPool: () => pool,
+  sqlExecOne: qr => new Promise((resolve, reject) => {
+    pool.query(qr, (err, rows) => {
+      if (err) {
+        console.error(err.message);
+        reject(err);
+      } else {
+        resolve(rows);
+      }
     });
-  }
-};
+  }),
+};

server/task/crawler/github.js

@@ -1,61 +1,52 @@
-"use strict";
+const cheerio = require('cheerio');
+const async = require('async');
+const rp = require('request-promise');
+const sqlConn = require('../../resource/mysqlConnection');
+const globalConfig = require('../../config/env');
 
-const cheerio = require('cheerio'),
-  async = require('async'),
-  fs = require('fs'),
-  rp = require('request-promise'),
-  sqlConn = require('../../resource/mysqlConnection'),
-  globalConfig = require('../../config/env'),
-
-  CONCURRENCY = 10,
-  TIMEOUT = 5 * 1000,
-  JOBFLAG = '_JOBDONE';
+const CONCURRENCY = 10;
+const TIMEOUT = 5 * 1000;
+const JOBFLAG = '_JOBDONE';
 
 let finished = null;
 const privateFn = {
-  checkNameExist: (v) => {
-    if(v.name.length === 0){
-      let gitList = v.github.split('/');
-      v.name = gitList[1];
-    }
-  },
-
   promiseLoop: (data0, resolve, reject) => {
     console.log(`downloading ${data0.length} packages...`);
     async.mapLimit(data0, CONCURRENCY, (v, cb) => {
-      privateFn.checkNameExist(v);
+      if (v.name.length === 0) {
+        const gitList = v.github.split('/');
+        v.name = gitList[1];
+      }
       privateFn.getNum(v, cb);
     }, (error, data) => {
-      let unfinished = data.filter(v => !v[JOBFLAG]);
+      const unfinished = data.filter(v => !v[JOBFLAG]);
       finished = finished.concat(data.filter(v => v[JOBFLAG]));
-      if(unfinished.length > 0){
+      if (unfinished.length > 0) {
         return privateFn.promiseLoop(unfinished, resolve, reject);
-      }else{
-        let result = finished.filter(v => v[JOBFLAG]);
-        resolve(result);
       }
+      const result = finished.filter(v => v[JOBFLAG]);
+      resolve(result);
+      return 0;
     });
   },
 
-  getNum: (obj, cb)=> {
-    //console.log('download ' + obj.name);
+  getNum: (obj0, cb) => {
+    const obj = Object.assign({}, obj0);
     rp({
-      uri: 'https://github.com/' + obj.github,
+      uri: `https://github.com/${obj.github}`,
       timeout: TIMEOUT,
-      transform: function (body) {
-        return cheerio.load(body);
-      }
+      transform: body => cheerio.load(body),
     })
-      .then(function ($) {
-        let elems = $('.social-count.js-social-count');
-        let starElem = elems[0];
-        let num = starElem.children[0].data.trim();
-        obj['star'] = num;
+      .then(($) => {
+        const elems = $('.social-count.js-social-count');
+        const starElem = elems[0];
+        const num = starElem.children[0].data.trim();
+        obj.star = num;
         obj[JOBFLAG] = true;
         cb(null, obj);
       })
-      .catch(function (err) {
-        console.error('crawler timeout on ' + obj.name);
+      .catch(() => {
+        console.error(`crawler timeout on ${obj.name}`);
         cb(null, obj);
       });
   },
@@ -65,19 +56,19 @@ const publicFn = {
   start: () => {
     finished = [];
     const deferred = new Promise((resolve, reject) => {
-      var qr = "SELECT * FROM git WHERE `group` IS NOT NULL";
-      if(globalConfig.isDev){
-        qr = "SELECT * FROM git WHERE `group` IS NOT NULL AND id > 419";
+      let qr = 'SELECT * FROM git WHERE `group` IS NOT NULL';
+      if (globalConfig.isDev) {
+        qr = 'SELECT * FROM git WHERE `group` IS NOT NULL AND id > 419';
       }
       sqlConn.sqlExecOne(qr).then((db) => {
-        privateFn.promiseLoop(db, resolve, reject)
-      }, (err) => {
+        privateFn.promiseLoop(db, resolve, reject);
+      }, () => {
         console.error('sql executed fails1.');
       });
     });
 
     return deferred;
-  }
+  },
 };
 
 module.exports = publicFn;

server/task/index.js

@@ -1,32 +1,27 @@
-'use strict';
+const inst1 = require('./render/main');
+const inst2 = require('./render/site');
+const inst3 = require('./render/article2');
+const inst4 = require('./render/article1');
+const vm = require('./render/viewModel_article.json');
 
-const globalConfig = require('../config/env'),
-  mysql = require('../resource/mysqlConnection'),
-  path = require('path'),
-  inst1 = require('./render/main'),
-  inst2 = require('./render/site'),
-  inst3 = require('./render/article2'),
-  inst4 = require('./render/article1'),
-  vm = require('./render/viewModel_article.json');
 
-
-console.log("================================");
-console.log(`updating the article...`);
-console.log("================================");
+console.log('================================');
+console.log('updating the article...');
+console.log('================================');
 
 inst3.start();
-vm.article.forEach(v => {
-  if(v.fileName){
+vm.article.forEach((v) => {
+  if (v.fileName) {
     inst4.start(v.id);
   }
 });
 
-console.log("================================");
-console.log(`updating the views...`);
-console.log("================================");
+console.log('================================');
+console.log('updating the views...');
+console.log('================================');
 
 inst1.start(() => {
   console.log('Finished Sucess!');
   process.exit();
 });
-inst2.start();
+inst2.start();