fix(user): crash fix for undefined title

polonel · Sep 20, 2018 · 5e2b38c · 5e2b38c
1 parent 6929cc1
commit 5e2b38c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/models/user.js b/src/models/user.js
@@ -82,9 +82,9 @@ userSchema.pre('save', function(next) {
     var user = this;
 
     user.username = user.username.trim();
-    user.title = user.title.trim();
     user.email = user.email.trim();
-    user.fullname = user.fullname.trim();
+    if (user.fullname) user.fullname = user.fullname.trim();
+    if (user.title) user.title = user.title.trim();
 
     if (!user.isModified('password'))
         return next();

diff --git a/src/public/js/angularjs/controllers/settings.js b/src/public/js/angularjs/controllers/settings.js
@@ -331,9 +331,8 @@ define(['angular', 'underscore', 'jquery', 'modules/helpers', 'modules/ui', 'uik
                             $target.find('ul>li[data-key]').first().addClass('active');
 
 
-                        if (settings === 'settings-legal' && privacyPolicyMDE) {
+                        if (settings === 'settings-legal' && privacyPolicyMDE)
                             privacyPolicyMDE.codemirror.refresh();
-                        }
                     }
                 }
             };