From 0d8e1f2043b45d9372900c42e15c4f8c6963dc36 Mon Sep 17 00:00:00 2001
From: Tyler Renelle <tylerrenelle@gmail.com>
Date: Thu, 12 Sep 2013 16:45:13 -0400
Subject: [PATCH] cleanup on end tour

---
 public/js/services/guideServices.js |  3 +--
 public/js/services/userServices.js  | 31 ++++++++++++++---------------
 2 files changed, 16 insertions(+), 18 deletions(-)

diff --git a/public/js/services/guideServices.js b/public/js/services/guideServices.js
index 4ba5400de4..45cbeff75b 100644
--- a/public/js/services/guideServices.js
+++ b/public/js/services/guideServices.js
@@ -51,8 +51,7 @@ angular.module('guideServices', []).
       $('.main-herobox').popover('destroy');
       var tour = new Tour({
         onEnd: function(){
-          User.user.flags.showTour = false;
-          User.log({op:'set',data:{'flags.showTour':false}});
+          User.set('flags.showTour', false);
         }
       });
       tourSteps.forEach(function(step) {
diff --git a/public/js/services/userServices.js b/public/js/services/userServices.js
index 1bb0e98001..31f285f471 100644
--- a/public/js/services/userServices.js
+++ b/public/js/services/userServices.js
@@ -135,38 +135,37 @@ angular.module('userServices', []).
             },
 
             log: function (action, cb) {
-                //push by one buy one if an array passed in.
-                if (_.isArray(action)) {
-                    action.forEach(function (a) {
-                        settings.sync.queue.push(a);
-                    });
-                } else {
-                    settings.sync.queue.push(action);
-                }
+              //push by one buy one if an array passed in.
+              if (_.isArray(action)) {
+                action.forEach(function (a) {
+                  settings.sync.queue.push(a);
+                });
+              } else {
+                settings.sync.queue.push(action);
+              }
 
-                save();
-                syncQueue(cb);
+              save();
+              syncQueue(cb);
             },
 
             /*
              Very simple path-set. `set('preferences.gender','m')` for example. We'll deprecate this once we have a complete API
              */
             set: function(k, v) {
-              var self = userServices;
               var log = { op: 'set', data: {} };
-              window.habitrpgShared.helpers.dotSet(k, v, userServices.user);
+              window.habitrpgShared.helpers.dotSet(k, v, this.user);
               log.data[k] = v;
-              userServices.log(log);
+              this.log(log);
             },
 
             setMultiple: function(obj){
               var self = this;
               var log = { op: 'set', data: {} };
               _.each(obj, function(v,k){
-                window.habitrpgShared.helpers.dotSet(k, v, userServices.user);
+                window.habitrpgShared.helpers.dotSet(k, v, self.user);
                 log.data[k] = v;
-              })
-              userServices.log(log);
+              });
+              self.log(log);
             },
 
             save: save,