From ace56c8dc2328424ae0c01e9e19204eaed5642b6 Mon Sep 17 00:00:00 2001
From: Michael Krotscheck <krotscheck@gmail.com>
Date: Wed, 4 Mar 2015 16:00:19 -0800
Subject: [PATCH] Enabled eslint:no-underscore-dangle

Rule enabled, discovered issues resolved.

Change-Id: If5fc53b0f1a6c3a31b4ef8b1858f4ef5d3ea0016
---
 .eslintrc                               |  1 -
 src/app/services/provider/preference.js | 10 +++++-----
 src/app/util/service/last_location.js   |  4 ++--
 3 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/.eslintrc b/.eslintrc
index 7606abc2..8fbef7b7 100644
--- a/.eslintrc
+++ b/.eslintrc
@@ -58,7 +58,6 @@
         "space-unary-ops": 0,
         "no-console": 0,
         "strict": 0,
-        "no-underscore-dangle": 0,
 
         // Stylistic
         "indent": [2, 4],
diff --git a/src/app/services/provider/preference.js b/src/app/services/provider/preference.js
index 1a450eaf..6bb4d5cb 100644
--- a/src/app/services/provider/preference.js
+++ b/src/app/services/provider/preference.js
@@ -61,7 +61,7 @@ angular.module('sb.services').provider('Preference',
              *
              * @returns {deferred.promise|*}
              */
-            this._resolveUserPreferences = function () {
+            this.resolveUserPreferences = function () {
                 var deferred = $q.defer();
 
                 // First resolve the session.
@@ -118,7 +118,7 @@ angular.module('sb.services').provider('Preference',
                         }
                     }
                 }
-                return this._save();
+                return this.save();
             };
 
             /**
@@ -158,7 +158,7 @@ angular.module('sb.services').provider('Preference',
                 // Store the preference.
                 preferences[key] = value;
 
-                return this._save();
+                return this.save();
             };
 
             /**
@@ -168,7 +168,7 @@ angular.module('sb.services').provider('Preference',
                 var deferred = $q.defer();
 
                 // This should never fail, see implementation above.
-                this._resolveUserPreferences().then(
+                this.resolveUserPreferences().then(
                     function (newPrefs) {
                         preferences = newPrefs;
                         deferred.resolve(preferences);
@@ -181,7 +181,7 @@ angular.module('sb.services').provider('Preference',
             /**
              * Private save method.
              */
-            this._save = function () {
+            this.save = function () {
                 var deferred = $q.defer();
 
                 // If preferences are defaults, they won't have a $save()
diff --git a/src/app/util/service/last_location.js b/src/app/util/service/last_location.js
index ca272938..1c1b44dd 100644
--- a/src/app/util/service/last_location.js
+++ b/src/app/util/service/last_location.js
@@ -50,7 +50,7 @@ angular.module('sb.util')
             /**
              * Initialize this service.
              */
-            _initialize: function () {
+            initialize: function () {
                 // Register (and disconnect) our listener.
                 $rootScope.$on('$destroy',
                     $rootScope.$on('$locationChangeStart', onLocationChange)
@@ -62,5 +62,5 @@ angular.module('sb.util')
         'use strict';
 
         // Initialize this service.
-        LastLocation._initialize();
+        LastLocation.initialize();
     });