ng-annotate - npm Package Compare versions

Comparing version 0.3.1 to 0.3.2

build/es5/ng-annotate-main.js

@@ -57,3 +57,3 @@ "use strict";
     var matchAngularModule = (obj.$chained || isShortDef(obj, re) || isLongDef(obj)) &&
-        is.someof(prop.name, ["provider", "value", "config", "factory", "directive", "filter", "run", "controller", "service"]);
+        is.someof(prop.name, ["provider", "value", "constant", "config", "factory", "directive", "filter", "run", "controller", "service", "decorator"]);
     if (!matchAngularModule) {
@@ -64,3 +64,3 @@ return false;
 
-    if (is.someof(prop.name, ["provider", "value"])) {
+    if (is.someof(prop.name, ["provider", "value", "constant"])) {
         return false; // affects matchAngularModule because of chaining
@@ -67,0 +67,0 @@ }

build/es5/ng-annotate.js

@@ -6,3 +6,3 @@ "use strict";
 var ngAnnotate = require("./ng-annotate-main");
-var version = "0.3.1";
+var version = "0.3.2";
 var optimist = require("optimist")
@@ -9,0 +9,0 @@ .usage("ng-annotate v" + version + "\n\nUsage: ng-annotate OPTIONS file.js")

build/es5/tests/original.js

@@ -114,2 +114,3 @@ "use strict";
     }).value("foo", "bar")
+    .constant("foo", "bar")
     .factory("foo", function() {
@@ -122,1 +123,11 @@ b;
     });
+
+// $provide
+angular.module("MyMod").directive("foo", function($a, $b) {
+    $provide.decorator("foo", function($scope, $timeout) {
+        a;
+    });
+    $provide.factory("bar", function($timeout, $scope) {
+        b;
+    });
+});

build/es5/tests/with_annotations.js

@@ -114,2 +114,3 @@ "use strict";
     }).value("foo", "bar")
+    .constant("foo", "bar")
     .factory("foo", function() {
@@ -122,1 +123,11 @@ b;
     }]);
+
+// $provide
+angular.module("MyMod").directive("foo", ["$a","$b", function($a, $b) {
+    $provide.decorator("foo", ["$scope","$timeout", function($scope, $timeout) {
+        a;
+    }]);
+    $provide.factory("bar", ["$timeout","$scope", function($timeout, $scope) {
+        b;
+    }]);
+}]);

CHANGES.md

@@ -0,1 +1,5 @@
+## v0.3.2 2013-09-30
+ * bugfix angular.module("MyMod").constant("foo", "bar") disrupting chaining
+ * match $provide.decorator (in addition to other $provide methods)
+
 ## v0.3.1 2013-09-30
@@ -2,0 +6,0 @@ * bugfix angular.module("MyMod").value("foo", "bar") disrupting chaining

ng-annotate-main.js

@@ -57,3 +57,3 @@ "use strict";
     const matchAngularModule = (obj.$chained || isShortDef(obj, re) || isLongDef(obj)) &&
-        is.someof(prop.name, ["provider", "value", "config", "factory", "directive", "filter", "run", "controller", "service"]);
+        is.someof(prop.name, ["provider", "value", "constant", "config", "factory", "directive", "filter", "run", "controller", "service", "decorator"]);
     if (!matchAngularModule) {
@@ -64,3 +64,3 @@ return false;
 
-    if (is.someof(prop.name, ["provider", "value"])) {
+    if (is.someof(prop.name, ["provider", "value", "constant"])) {
         return false; // affects matchAngularModule because of chaining
@@ -67,0 +67,0 @@ }

package.json

 {
   "name": "ng-annotate",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "description": "add, remove and rebuild angularjs dependency injection annotations",
@@ -5,0 +5,0 @@ "main": "build/es5/ng-annotate-main.js",

README.md

@@ -76,8 +76,11 @@ # ng-annotate
 
-ng-annotate understands `this.$get = function($scope)` and
-`{.., $get: function($scope), ..}` inside a `provider`.
+ng-annotate understands `this.$get = function($scope) ..` and
+`{.., $get: function($scope) ..}` inside a `provider`.
 
-ng-annotate understands `return {.., controller: function($scope), ..}` inside a
+ng-annotate understands `return {.., controller: function($scope) ..}` inside a
 `directive`.
 
+ng-annotate understands `$provide.decorator("bar", function($scope) ..)` and other methods
+on `provide` such as `factory`.
+
 ng-annotate understands chaining.
@@ -84,0 +87,0 @@

tests/original.js

@@ -114,2 +114,3 @@ "use strict";
     }).value("foo", "bar")
+    .constant("foo", "bar")
     .factory("foo", function() {
@@ -122,1 +123,11 @@ b;
     });
+
+// $provide
+angular.module("MyMod").directive("foo", function($a, $b) {
+    $provide.decorator("foo", function($scope, $timeout) {
+        a;
+    });
+    $provide.factory("bar", function($timeout, $scope) {
+        b;
+    });
+});

tests/with_annotations.js

@@ -114,2 +114,3 @@ "use strict";
     }).value("foo", "bar")
+    .constant("foo", "bar")
     .factory("foo", function() {
@@ -122,1 +123,11 @@ b;
     }]);
+
+// $provide
+angular.module("MyMod").directive("foo", ["$a","$b", function($a, $b) {
+    $provide.decorator("foo", ["$scope","$timeout", function($scope, $timeout) {
+        a;
+    }]);
+    $provide.factory("bar", ["$timeout","$scope", function($timeout, $scope) {
+        b;
+    }]);
+}]);

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

36749

increased by4.31%

Lines of code

961

increased by4.34%

Number of lines in readme file

103

increased by3%

No dependency changes