From 7c15f09e03595d2e38c19deecc628983fd4264e8 Mon Sep 17 00:00:00 2001
From: Adrien Lemaire <lemaire.adrien@gmail.com>
Date: Fri, 15 Nov 2013 00:51:54 +1100
Subject: [PATCH] code logic to update preferences

---
 public/js/controllers/userCtrl.js | 3 ---
 src/models/user.js                | 3 +++
 views/options/profile.jade        | 8 ++++----
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/public/js/controllers/userCtrl.js b/public/js/controllers/userCtrl.js
index 793f54c9f7..70a950c9cd 100644
--- a/public/js/controllers/userCtrl.js
+++ b/public/js/controllers/userCtrl.js
@@ -6,9 +6,6 @@ habitrpg.controller("UserCtrl", ['$rootScope', '$scope', '$location', 'User', '$
     $scope.hideUserAvatar = function() {
       $(".userAvatar").hide();
     };
-    $scope.toggleHelm = function(val){
-      User.log({op:'set', data:{'preferences.showHelm':val}});
-    }
 
     $scope.$watch('_editing.profile', function(value){
       if(value === true) $scope.editingProfile = angular.copy(User.user.profile);
diff --git a/src/models/user.js b/src/models/user.js
index f1ac9ea5e5..402dddf34f 100644
--- a/src/models/user.js
+++ b/src/models/user.js
@@ -191,6 +191,9 @@ var UserSchema = new Schema({
     hair: {type:String, 'default':'blond'},
     hideHeader: {type:Boolean, 'default':false},
     showHelm: {type:Boolean, 'default':true},
+    showWeapon: {type:Boolean, 'default':true},
+    showShield: {type:Boolean, 'default':true},
+    showArmor: {type:Boolean, 'default':true},
     skin: {type:String, 'default':'white'},
     timezoneOffset: Number,
     language: String
diff --git a/views/options/profile.jade b/views/options/profile.jade
index 18575f46b7..2ec556a09b 100644
--- a/views/options/profile.jade
+++ b/views/options/profile.jade
@@ -9,16 +9,16 @@ script(id='partials/options.profile.avatar.html', type='text/ng-template')
             button.m_head_0.customize-option(type='button', ng-click='set("preferences.gender","m")')
             button.f_head_0.customize-option(type='button', ng-click='set("preferences.gender","f")')
           label.checkbox
-            input(type='checkbox', ng-model='user.preferences.showHelm', ng-change='toggleHelm(user.preferences.showHelm)')
+            input(type='checkbox', ng-model='user.preferences.showHelm', ng-change='set("preferences.showHelm", user.preferences.showHelm?true: false)')
             | Show Helm
           label.checkbox
-            input(type='checkbox', ng-model='user.preferences.showWeapon', ng-change='toggleHelm(user.preferences.showWeapon)')
+            input(type='checkbox', ng-model='user.preferences.showWeapon', ng-change='set("preferences.showWeapon", user.preferences.showWeapon?true: false)')
             | Show Weapon
           label.checkbox
-            input(type='checkbox', ng-model='user.preferences.showShield', ng-change='toggleHelm(user.preferences.showShield)')
+            input(type='checkbox', ng-model='user.preferences.showShield', ng-change='set("preferences.showShield", user.preferences.showShield?true: false)')
             | Show Shield
           label.checkbox
-            input(type='checkbox', ng-model='user.preferences.showArmor', ng-change='toggleHelm(user.preferences.showArmor)')
+            input(type='checkbox', ng-model='user.preferences.showArmor', ng-change='set("preferences.showArmor", user.preferences.showArmor?true: false)')
             | Show Armor
 
         menu(ng-show='user.preferences.gender=="f"', type='list')