refactoring

update/Repository.js

@@ -1,10 +1,9 @@
-/***
+/**
  * @constructor
  * @param {Object} data
  * @returns {Repository}
  */
 function Repository(data) {
- var self = this;
  /**
  * @type {string}
  */
@@ -30,8 +29,8 @@ function Repository(data) {
  * @returns {string} Folder name
  */
 Repository.prototype.getFolderName = function() {
- var lastSlash = self.url.lastIndexOf("/");
- return this.url.substring(lastSlash + 1, self.url.length);
+ var lastSlash = this.url.lastIndexOf("/");
+ return this.url.substring(lastSlash + 1, this.url.length);
 };
 
 Repository.prototype.getDotGitFolder = function() {

update/commitsToJson.js

@@ -15,7 +15,7 @@ var colors = require('colors');
  * ]
  * @returns {void}
  */
-function commitsArray(repo, callBack) {
+module.exports.commitsArray = function(repo, callBack) {
 
  var result = '';
  var separator = ' qqqqqqqq ';
@@ -83,4 +83,3 @@ function commitsArray(repo, callBack) {
 }
 
 
-module.exports.commitsArray = commitsArray;

update/getCommitsData.js

@@ -10,7 +10,7 @@ var colors = require('colors');
 
 
 
-/***
+/**
  * @description write File
  * @param {type} commits
  * @param {Repository} repo

update/github.js

@@ -22,7 +22,7 @@ var functionToRun;
  * @param {Function} func To run after all of the getLogForFile instances were done
  * @returns {void}
  */
-var setFunction = function(func) {
+module.exports.setFunction = function(func) {
  functionToRun = func;
 };
 
@@ -45,7 +45,7 @@ var dotGitPath = '';
  * @param {string} path
  * @returns {void}
  */
-var setDotGitPath = function(path) {
+module.exports.setDotGitPath = function(path) {
  dotGitPath = path;
 };
 
@@ -74,7 +74,7 @@ parentPath = findParentPath(parentPath);
  * @param {string} str Repository URL.
  * @returns {string} Folder name
  */
-var getFolderName = function(str) {
+module.exports.getFolderName = function(str) {
  var lastSlash = str.lastIndexOf("/");
  var end = str.length;
  if (str.indexOf('.git') === -1)
@@ -91,7 +91,7 @@ var getFolderName = function(str) {
  * @param {boolean} verbose Do you want a lot of output?
  * @returns {void}
  */
-var cloneOrPull = function(repoURL, callBack, verbose) {
+module.exports.cloneOrPull = function(repoURL, callBack, verbose) {
  containingFolder = getFolderName(repoURL);
  var clone = false;
  var gitFolder = '--git-dir=' + parentPath + '/repositories/' + containingFolder + '/.git';
@@ -155,7 +155,7 @@ function getRelativePath(fullPath) {
  * @param {string} fullPath of file to get its log.
  * @returns {void}
  */
-var getLogForFile = function(fullPath, callBack) {
+module.exports.getLogForFile = function(fullPath, callBack) {
  if (dotGitPath === '')
  throw new Error('\ngit-dir must be set.\nUse setGitDir to set it.'.red);
 
@@ -229,11 +229,3 @@ var getLogForFile = function(fullPath, callBack) {
  console.log(buff.toString('utf8').red);
  });
 };
-
-
-module.exports.setFunction = setFunction;
-module.exports.setDotGitPath = setDotGitPath;
-module.exports.cloneOrPull = cloneOrPull;
-module.exports.getLogForFile = getLogForFile;
-module.exports.getFolderName = getFolderName;
-

update/jsonSanitizer.js

@@ -1,11 +1,11 @@
 
-/***
+/**
  *@description Sanitize JSON string before saving, so it can be read again. (Escapes newlines etc)
  * @param {string} unsanitized string
  * @returns {string} Sanitized string
  * @see https://gist.github.com/jamischarles/1046671
  */
-function sanitize(unsanitized) {
+module.exports.sanitize = function(unsanitized) {
  return unsanitized
  .trim()
  .replace(/\\/g, "\\\\")
@@ -15,7 +15,5 @@ function sanitize(unsanitized) {
  .replace(/\f/g, "\\f")
  .replace(/"/g, "\\\"") //.replace(/'/g,"\\\'")
  .trim();
-}
+};
 
-
-module.exports.sanitize = sanitize;

update/lineCounter.js

@@ -1,6 +1,6 @@
 
 
-/***
+/**
  * @description Count size(in bytes), and lines in files and folders.
  * @constructor
  */
@@ -15,7 +15,7 @@ function LineCounter() {
  */
  var allFiles = new Array();
 
- /***
+ /**
  * @description Fill allFiles array with all file names under currentPath - ignor folders
  * @param {string} currentPath
  * @returns {void}
@@ -37,7 +37,7 @@ function LineCounter() {
  }
  };
 
- /***
+ /**
  * @description Return true if fileName is or within the .git folder.
  * @param {string} fileName To check if its a git file.
  * @returns {boolean}
@@ -55,7 +55,7 @@ function LineCounter() {
  }
 
 
- /***
+ /**
  * @description Return file size in bytes - for hard links return 0.
  * @param {string} path to get is size.
  * @returns {number} File size in bytes
@@ -68,7 +68,7 @@ function LineCounter() {
  }
  };
 
- /***
+ /**
  * @param {string} directory to get is size.
  * @returns {number} Directory size in bytes
  */
@@ -151,7 +151,7 @@ function LineCounter() {
 
 
 
-exports.createLineCounter = function() {
+module.exports.createLineCounter = function() {
  return new LineCounter();
 };
 

update/numStat.js

@@ -1,12 +1,12 @@
 
-/***
+/**
+ * @module
  * @description Module to parse the result of git log --numstat
  * @see man git log --numstat
  */
 
 
-/***
- * 
+/**
  * Column1 = inserted
  * Column2 = removed 
  * columns are separeted by tab (\t)
@@ -39,7 +39,7 @@ var parseLineStat = function(lineStat) {
  * @param {string} numstat
  * @returns {Object<total:<number>,files:<Array>}
  */
-var parseNumStat = function(numstat) {
+module.exports.parseNumStat = function(numstat) {
  var lines = numstat.split('\n');
  var statArr = new Array();
  for (var i = 0; i < lines.length; i++) {
@@ -57,6 +57,3 @@ var parseNumStat = function(numstat) {
  // ,'files': statArr
  };
 };
-
-
-module.exports.parseNumStat = parseNumStat;

update/repositories.js

@@ -1,4 +1,4 @@
-var eKnights = require('../eKnightsData').eKnightsData;
+var eKnights = require('../src/eKnightsData').eKnightsData;
 var Repository = require('./Repository').Repository;
 
 

update/rootChecker.js

@@ -1,7 +1,7 @@
 var spawn = require('child_process').spawn;
 var colors = require('colors');
 
-module.exports.isRootUser = isRootUser;
+
 
 
 /**
@@ -13,7 +13,7 @@ module.exports.isRootUser = isRootUser;
  * @returns {void}
  * TODO: maybe it's better to use id 
  */
-function isRootUser(callback, testing) {
+module.exports.isRootUser = function(callback, testing) {
  if (testing) {
  callback();
  return;
@@ -30,4 +30,4 @@ function isRootUser(callback, testing) {
 
  callback();
  });
-}
+};