ng-annotate - npm Package Compare versions

Comparing version 0.7.0 to 0.7.1

build/es5/ng-annotate-main.js

@@ -73,16 +73,13 @@ "use strict";
 
-    if (args.length !== 2) {
+    // $stateProvider.state({ ... }) and $stateProvider.state("name", { ... })
+    if (is.noneof(args.length, [1, 2])) {
         return false;
     }
 
-    var firstArgIsStringLiteral = (args[0].type === "Literal" && is.string(args[0].value));
-    if (!firstArgIsStringLiteral) {
+    var configArg = args[args.length - 1];
+    if (configArg.type !== "ObjectExpression") {
         return false;
     }
 
-    if (args[1].type !== "ObjectExpression") {
-        return false;
-    }
-
-    var props = args[1].properties;
+    var props = configArg.properties;
     var res = [
@@ -89,0 +86,0 @@ matchProp("controller", props),

build/es5/ng-annotate.js

@@ -6,3 +6,3 @@ "use strict";
 var ngAnnotate = require("./ng-annotate-main");
-var version = "0.7.0";
+var version = "0.7.1";
 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

@@ -189,4 +189,9 @@ "use strict";
 }).state("myState2", {
-        controller: function($scope) {},
-    });
+    controller: function($scope) {},
+}).state({
+    name: "myState3",
+    controller: function($scope, simpleObj, promiseObj, translations) {
+        c;
+    },
+});
 $urlRouterProvider.when("", function($match) { a; });
@@ -193,0 +198,0 @@ $urlRouterProvider.otherwise("", function($location) { a; });

build/es5/tests/with_annotations_single.js

@@ -189,4 +189,9 @@ "use strict";
 }).state("myState2", {
-        controller: ['$scope', function($scope) {}],
-    });
+    controller: ['$scope', function($scope) {}],
+}).state({
+    name: "myState3",
+    controller: ['$scope', 'simpleObj', 'promiseObj', 'translations', function($scope, simpleObj, promiseObj, translations) {
+        c;
+    }],
+});
 $urlRouterProvider.when("", ['$match', function($match) { a; }]);
@@ -193,0 +198,0 @@ $urlRouterProvider.otherwise("", ['$location', function($location) { a; }]);

build/es5/tests/with_annotations.js

@@ -189,4 +189,9 @@ "use strict";
 }).state("myState2", {
-        controller: ["$scope", function($scope) {}],
-    });
+    controller: ["$scope", function($scope) {}],
+}).state({
+    name: "myState3",
+    controller: ["$scope", "simpleObj", "promiseObj", "translations", function($scope, simpleObj, promiseObj, translations) {
+        c;
+    }],
+});
 $urlRouterProvider.when("", ["$match", function($match) { a; }]);
@@ -193,0 +198,0 @@ $urlRouterProvider.otherwise("", ["$location", function($location) { a; }]);

CHANGES.md

@@ -0,1 +1,4 @@
+## v0.7.1 2014-04-30
+ * ui-router declarations improvements
+
 ## v0.7.0 2014-04-30
@@ -2,0 +5,0 @@ * ui-router declarations support

ng-annotate-main.js

@@ -73,16 +73,13 @@ "use strict";
 
-    if (args.length !== 2) {
+    // $stateProvider.state({ ... }) and $stateProvider.state("name", { ... })
+    if (is.noneof(args.length, [1, 2])) {
         return false;
     }
 
-    const firstArgIsStringLiteral = (args[0].type === "Literal" && is.string(args[0].value));
-    if (!firstArgIsStringLiteral) {
+    const configArg = args[args.length - 1];
+    if (configArg.type !== "ObjectExpression") {
         return false;
     }
 
-    if (args[1].type !== "ObjectExpression") {
-        return false;
-    }
-
-    const props = args[1].properties;
+    const props = configArg.properties;
     const res = [
@@ -89,0 +86,0 @@ matchProp("controller", props),

package.json

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

tests/original.js

@@ -189,4 +189,9 @@ "use strict";
 }).state("myState2", {
-        controller: function($scope) {},
-    });
+    controller: function($scope) {},
+}).state({
+    name: "myState3",
+    controller: function($scope, simpleObj, promiseObj, translations) {
+        c;
+    },
+});
 $urlRouterProvider.when("", function($match) { a; });
@@ -193,0 +198,0 @@ $urlRouterProvider.otherwise("", function($location) { a; });

tests/with_annotations_single.js

@@ -189,4 +189,9 @@ "use strict";
 }).state("myState2", {
-        controller: ['$scope', function($scope) {}],
-    });
+    controller: ['$scope', function($scope) {}],
+}).state({
+    name: "myState3",
+    controller: ['$scope', 'simpleObj', 'promiseObj', 'translations', function($scope, simpleObj, promiseObj, translations) {
+        c;
+    }],
+});
 $urlRouterProvider.when("", ['$match', function($match) { a; }]);
@@ -193,0 +198,0 @@ $urlRouterProvider.otherwise("", ['$location', function($location) { a; }]);

tests/with_annotations.js

@@ -189,4 +189,9 @@ "use strict";
 }).state("myState2", {
-        controller: ["$scope", function($scope) {}],
-    });
+    controller: ["$scope", function($scope) {}],
+}).state({
+    name: "myState3",
+    controller: ["$scope", "simpleObj", "promiseObj", "translations", function($scope, simpleObj, promiseObj, translations) {
+        c;
+    }],
+});
 $urlRouterProvider.when("", ["$match", function($match) { a; }]);
@@ -193,0 +198,0 @@ $urlRouterProvider.otherwise("", ["$location", function($location) { a; }]);

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

63335

increased by1.5%

Lines of code

1769

increased by1.49%

No dependency changes