Migration ES6: DAOs

Related OWASP#152
UlisesGascon · Jan 29, 2020 · d5b6976 · d5b6976
1 parent cad7959
commit d5b6976
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/app/data/profile-dao.js b/app/data/profile-dao.js
@@ -42,7 +42,7 @@ function ProfileDAO(db) {
     this.updateUser = (userId, firstName, lastName, ssn, dob, address, bankAcc, bankRouting, callback) => {
 
         // Create user document
-        var user = {};
+        const user = {};
         if (firstName) {
             user.firstName = firstName;
         }

diff --git a/app/data/user-dao.js b/app/data/user-dao.js
@@ -17,7 +17,7 @@ function UserDAO(db) {
     this.addUser = (userName, firstName, lastName, password, email, callback) => {
 
         // Create user document
-        var user = {
+        const user = {
             userName,
             firstName,
             lastName,
@@ -54,7 +54,7 @@ function UserDAO(db) {
         return `${year}-${("0" + month).slice(-2)}-${("0" + day).slice(-2)}`
     };
 
-    this.validateLogin = function(userName, password, callback) {
+    this.validateLogin = (userName, password, callback) => {
 
         // Helper function to compare passwords
         const comparePassword = (fromDB, fromUser) => {
@@ -75,13 +75,13 @@ function UserDAO(db) {
                 if (comparePassword(password, user.password)) {
                     callback(null, user);
                 } else {
-                    var invalidPasswordError = new Error("Invalid password");
+                    const invalidPasswordError = new Error("Invalid password");
                     // Set an extra field so we can distinguish this from a db error
                     invalidPasswordError.invalidPassword = true;
                     callback(invalidPasswordError, null);
                 }
             } else {
-                var noSuchUserError = new Error("User: " + user + " does not exist");
+                const noSuchUserError = new Error("User: " + user + " does not exist");
                 // Set an extra field so we can distinguish this from a db error
                 noSuchUserError.noSuchUser = true;
                 callback(noSuchUserError, null);