client/models: discard field declarations

This has important side effect that matters when we check for data
changes using _orig dictionary. Previously, _orig was empty (so its
members fields were undefiend) whereas the real fields were declared as
nulls. This meant that for new entities, the conditions were always
true, which is unintended. Now both _orig and the class itself are
initially populated with _updateFromResponse which syncs the state
between them, removing the problem.

1 files changed, 1 insertions, 20 deletions

diff --git a/client/js/models/user.js b/client/js/models/user.js
index 4ebcac0..0b45866 100644
--- a/client/js/models/user.js
+++ b/client/js/models/user.js
@@ -6,26 +6,7 @@ const events = require('../events.js');
 class User extends events.EventTarget {
     constructor() {
         super();
-        this._name              = null;
-        this._rank              = null;
-        this._email             = null;
-        this._avatarStyle       = null;
-        this._avatarUrl         = null;
-        this._creationTime      = null;
-        this._lastLoginTime     = null;
-        this._commentCount      = null;
-        this._favoritePostCount = null;
-        this._uploadedPostCount = null;
-        this._likedPostCount    = null;
-        this._dislikedPostCount = null;
-
-        this._origName          = null;
-        this._origEmail         = null;
-        this._origRank          = null;
-        this._origAvatarStyle   = null;
-
-        this._password          = null;
-        this._avatarContent     = null;
+        this._updateFromResponse({});
     }
 
     get name()               { return this._name; }