Merge pull request #1134 from farajfarook/master

fix(users): Removing redundant variables

modules/users/server/controllers/users/users.authentication.server.controller.js

@@ -22,11 +22,8 @@ exports.signup = function (req, res) {
   // For security measurement we remove the roles from the req.body object
   delete req.body.roles;
 
-  // Init Variables
+  // Init user and add missing fields
   var user = new User(req.body);
-  var message = null;
-
-  // Add missing user fields
   user.provider = 'local';
   user.displayName = user.firstName + ' ' + user.lastName;
 

modules/users/server/controllers/users/users.password.server.controller.js

@@ -122,7 +122,6 @@ exports.validateResetToken = function (req, res) {
 exports.reset = function (req, res, next) {
   // Init Variables
   var passwordDetails = req.body;
-  var message = null;
 
   async.waterfall([
 

modules/users/server/controllers/users/users.profile.server.controller.js

@@ -55,7 +55,6 @@ exports.update = function (req, res) {
  */
 exports.changeProfilePicture = function (req, res) {
   var user = req.user;
-  var message = null;
   var upload = multer(config.uploads.profileUpload).single('newProfilePicture');
   var profileUploadFileFilter = require(path.resolve('./config/lib/multer')).profileUploadFileFilter;