Cleaning up npm/bower dependencies and grunt tasks

- Removed unneeded dependencies and updated needed dependencies in
package.json and bower.json
- Cleaned up package.json and bower.json
- Remove unneeded “relaxing” options and added some “enforcing” options
in .jshintrc
- Updated angularfire.js to conform to new .jshintrc options
- Removed changelog task from grunt
- Removed CasperJS link in README
- Removed link to license

Author: jacobawenger

.gitignore

@@ -1,5 +1,3 @@
 bower_components/
 node_modules/
-bower_components/
-selenium/
 .idea

.jshintrc

@@ -1,22 +1,21 @@
 {
-  "bitwise" : true,
-  "boss"    : true,
-  "browser" : true,
-  "curly"   : true,
-  "devel"   : true,
-  "eqnull"  : true,
-  "globals" : {
-    "angular"             : false,
-    "Firebase"            : false,
-    "FirebaseSimpleLogin" : false
-  },
-  "globalstrict" : true,
-  "indent"       : 2,
-  "latedef"      : true,
-  "maxlen"       : 115,
-  "noempty"      : true,
-  "nonstandard"  : true,
-  "undef"        : true,
-  "unused"       : true,
-  "trailing"     : true
-}
+  "predef": [
+    "angular",
+    "Firebase",
+    "FirebaseSimpleLogin"
+  ],
+  "bitwise": true,
+  "browser": true,
+  "curly": true,
+  "forin": true,
+  "indent": 2,
+  "latedef": true,
+  "maxlen": 115,
+  "noempty": true,
+  "nonbsp": true,
+  "quotmark": "double",
+  "strict": true,
+  "trailing": true,
+  "undef": true,
+  "unused": true
+}

Gruntfile.js

@@ -41,26 +41,7 @@ module.exports = function(grunt) {
     // Lint JavaScript
     jshint : {
       options : {
-        'bitwise' : true,
-        'boss'    : true,
-        'browser' : true,
-        'curly'   : true,
-        'devel'   : true,
-        'eqnull'  : true,
-        'globals' : {
-          'angular'             : false,
-          'Firebase'            : false,
-          'FirebaseSimpleLogin' : false
-        },
-        'globalstrict' : true,
-        'indent'       : 2,
-        'latedef'      : true,
-        'maxlen'       : 115,
-        'noempty'      : true,
-        'nonstandard'  : true,
-        'undef'        : true,
-        'unused'       : true,
-        'trailing'     : true
+        jshintrc: '.jshintrc'
       },
       all : ['angularfire.js']
     },
@@ -115,13 +96,6 @@ module.exports = function(grunt) {
           message: 'Build Finished'
         }
       }
-    },
-
-    // Auto-populating changelog
-    changelog: {
-      options: {
-        dest: 'CHANGELOG.md'
-      }
     }
   });
 

README.md

@@ -21,8 +21,7 @@ Development
 [![Built with Grunt](https://cdn.gruntjs.com/builtwith.png)](http://gruntjs.com/)
 
 If you'd like to hack on AngularFire itself, you'll need
-[node.js](http://nodejs.org/download/), [Bower](http://bower.io), and
-[CasperJS](https://github.com/n1k0/casperjs).
+[node.js](http://nodejs.org/download/) and [Bower](http://bower.io).
 
 You can also start hacking on AngularFire in a matter of seconds on
 [Nitrous.IO](https://www.nitrous.io/?utm_source=github.com&utm_campaign=angularFire&utm_medium=hackonnitrous)

angularfire.js

@@ -9,9 +9,8 @@
 //      http://angularfire.com
 //      License: MIT
 
-"use strict";
-
 (function() {
+  "use strict";
 
   var AngularFire, AngularFireAuth;
 
@@ -41,7 +40,7 @@
     return function(input) {
       var sorted = [];
       if (input) {
-        if (!input.$getIndex || typeof input.$getIndex != "function") {
+        if (!input.$getIndex || typeof input.$getIndex !== "function") {
           // input is not an angularFire instance
           if (angular.isArray(input)) {
             // If input is an array, copy it
@@ -134,7 +133,7 @@
       child_removed: []
     };
 
-    if (typeof ref == "string") {
+    if (typeof ref === "string") {
       throw new Error("Please provide a Firebase reference instead " +
         "of a URL, eg: new Firebase(url)");
     }
@@ -191,7 +190,7 @@
           }
         }
 
-        if (typeof item == "object") {
+        if (typeof item === "object") {
           ref = self._fRef.ref().push(self._parseObject(item), _addCb);
         } else {
           ref = self._fRef.ref().push(item, _addCb);
@@ -307,7 +306,7 @@
             }
           },
         applyLocally);
-        
+
         return deferred.promise;
       };
 
@@ -490,7 +489,7 @@
       });
 
       function _isPrimitive(v) {
-        return v === null || typeof(v) !== 'object';
+        return v === null || typeof(v) !== "object";
       }
 
       function _initialLoad(value) {
@@ -527,7 +526,7 @@
       // child_* listeners attached; if the data suddenly changes between an object
       // and a primitive, the child_added/removed events will fire, and our data here
       // will get updated accordingly so we should be able to transition without issue
-      self._fRef.on('value', function(snap) {
+      self._fRef.on("value", function(snap) {
         // primitive handling
         var value = snap.val();
         if( _isPrimitive(value) ) {
@@ -539,7 +538,7 @@
         }
 
         // broadcast the value event
-        self._broadcastEvent('value', self._makeEventSnapshot(snap.name(), value));
+        self._broadcastEvent("value", self._makeEventSnapshot(snap.name(), value));
 
         // broadcast initial loaded event once data and indices are set up appropriately
         if( !self._loaded ) {
@@ -551,7 +550,7 @@
     // Called whenever there is a remote change. Applies them to the local
     // model for both explicit and implicit sync modes.
     _updateModel: function(key, value) {
-      if (value == null) {
+      if (value === null) {
         delete this._object[key];
       } else {
         this._object[key] = value;
@@ -622,7 +621,7 @@
     // If event handlers for a specified event were attached, call them.
     _broadcastEvent: function(evt, param) {
       var cbs = this._on[evt] || [];
-      if( evt === 'loaded' ) {
+      if( evt === "loaded" ) {
         this._on[evt] = []; // release memory
       }
       var self = this;
@@ -635,7 +634,7 @@
 
       if (cbs.length > 0) {
         for (var i = 0; i < cbs.length; i++) {
-          if (typeof cbs[i] == "function") {
+          if (typeof cbs[i] === "function") {
             _wrapTimeout(cbs[i], param);
           }
         }
@@ -645,18 +644,18 @@
     // triggers an initial event for loaded, value, and child_added events (which get immediate feedback)
     _sendInitEvent: function(evt, callback) {
       var self = this;
-      if( self._loaded && ['child_added', 'loaded', 'value'].indexOf(evt) > -1 ) {
+      if( self._loaded && ["child_added", "loaded", "value"].indexOf(evt) > -1 ) {
         self._timeout(function() {
-          var parsedValue = self._object.hasOwnProperty('$value')?
+          var parsedValue = self._object.hasOwnProperty("$value")?
             self._object.$value : self._parseObject(self._object);
           switch(evt) {
-          case 'loaded':
+          case "loaded":
             callback(parsedValue);
             break;
-          case 'value':
+          case "value":
             callback(self._makeEventSnapshot(self._fRef.name(), parsedValue, null));
             break;
-          case 'child_added':
+          case "child_added":
             self._iterateChildren(parsedValue, function(name, val, prev) {
               callback(self._makeEventSnapshot(name, val, prev));
             });
@@ -710,7 +709,7 @@
 
       // If the local model is an object, call an update to set local values.
       var local = self._parse(name)(scope);
-      if (local !== undefined && typeof local == "object") {
+      if (local !== undefined && typeof local === "object") {
         self._fRef.ref().update(self._parseObject(local));
       }
 
@@ -720,9 +719,9 @@
       });
 
       // Once we receive the initial value, the promise will be resolved.
-      self._object.$on('loaded', function(value) {
+      self._object.$on("loaded", function(value) {
         self._timeout(function() {
-          if(value === null && typeof defaultFn === 'function') {
+          if(value === null && typeof defaultFn === "function") {
             scope[name] = defaultFn();
           }
           else {
@@ -768,10 +767,10 @@
       function _findReplacePriority(item) {
         for (var prop in item) {
           if (item.hasOwnProperty(prop)) {
-            if (prop == "$priority") {
+            if (prop === "$priority") {
               item[".priority"] = item.$priority;
               delete item.$priority;
-            } else if (typeof item[prop] == "object") {
+            } else if (typeof item[prop] === "object") {
               _findReplacePriority(item[prop]);
             }
           }
@@ -822,7 +821,7 @@
     this._getCurrentUserDeferred = [];
     this._currentUserData = undefined;
 
-    if (typeof ref == "string") {
+    if (typeof ref === "string") {
       throw new Error("Please provide a Firebase reference instead " +
         "of a URL, eg: new Firebase(url)");
     }

angularfire.min.js

@@ -1,15 +1,26 @@
 {
   "name": "angularfire",
+  "description": "An officially supported AngularJS binding for Firebase.",
   "version": "0.7.1",
-  "main": ["./angularfire.js"],
-  "ignore": ["Gruntfile.js", "package.js", "tests", "README.md", ".travis.yml"],
+  "main": "angularfire.js",
+  "ignore": [
+    "Gruntfile.js",
+    "bower_components",
+    "node_modules",
+    "package.json",
+    "tests",
+    "README.md",
+    "LICENSE",
+    ".travis.yml",
+    ".jshintrc",
+    ".gitignore"
+  ],
   "dependencies": {
     "angular": "~1.2.0",
-    "firebase": "~1.0.5",
-    "firebase-simple-login": "~1.3.0"
+    "firebase": "1.0.x",
+    "firebase-simple-login": "1.6.x"
   },
   "devDependencies": {
-    "angular-mocks" : "~1.2.0",
-    "observe-js": "~0.1.4"
+    "angular-mocks" : "~1.2.0"
   }
 }

bower.json

@@ -1,40 +1,35 @@
 {
   "name": "angularfire",
-  "version": "0.7.1",
   "description": "An officially supported AngularJS binding for Firebase.",
+  "version": "0.7.1",
   "main": "angularfire.js",
+  "private": true,
   "repository": {
     "type": "git",
     "url": "https://github.com/firebase/angularFire.git"
   },
   "bugs": {
     "url": "https://github.com/firebase/angularFire/issues"
   },
+  "dependencies": {
+  },
   "devDependencies": {
     "grunt": "~0.4.1",
-    "load-grunt-tasks": "~0.2.0",
-    "grunt-contrib-uglify": "~0.2.2",
+    "grunt-karma": "~0.6.2",
     "grunt-notify": "~0.2.7",
+    "load-grunt-tasks": "~0.2.0",
+    "grunt-shell-spawn": "^0.3.0",
     "grunt-contrib-watch": "~0.5.1",
-    "grunt-contrib-jshint": "~0.6.2",
-    "grunt-karma": "~0.6.2",
-    "grunt-exec": "~0.4.2",
-    "grunt-conventional-changelog": "~1.0.0",
+    "grunt-contrib-jshint": "~0.10.0",
+    "grunt-contrib-uglify": "~0.2.2",
     "grunt-contrib-connect": "^0.7.1",
-    "grunt-shell-spawn": "^0.3.0",
     "grunt-protractor-runner": "^1.0.0",
+
+    "karma": "~0.10.4",
     "karma-jasmine": "~0.1.3",
-    "karma-script-launcher": "~0.1.0",
-    "karma-firefox-launcher": "~0.1.0",
-    "karma-html2js-preprocessor": "~0.1.0",
-    "karma-requirejs": "~0.2.0",
-    "karma-coffee-preprocessor": "~0.1.0",
     "karma-phantomjs-launcher": "~0.1.0",
-    "karma": "~0.10.4",
-    "karma-chrome-launcher": "~0.1.0",
-    "protractor": "^0.23.1",
-    "lodash": "~2.4.1",
-    "karma-safari-launcher": "~0.1.1",
-    "firebase": "^1.0.15-3"
+
+    "firebase": "1.0.x",
+    "protractor": "^0.23.1"
   }
 }