fix($injector): provider can now be defined in the array format

sudhirj · pkozlowski-opensource · commit cf89e8653cd7 · 2012-12-01T19:09:36.000+01:00
`injector.instantiate` is now called for arrays too, instead of only for functions. Closes angular#1452
diff --git a/src/auto/injector.js b/src/auto/injector.js
@@ -440,7 +440,7 @@ function createInjector(modulesToLoad) {
   }
 
   function provider(name, provider_) {
-    if (isFunction(provider_)) {
+    if (isFunction(provider_) || isArray(provider_)) {
       provider_ = providerInjector.instantiate(provider_);
     }
     if (!provider_.$get) {
diff --git a/test/auto/injectorSpec.js b/test/auto/injectorSpec.js
@@ -387,6 +387,20 @@ describe('injector', function() {
         });
 
 
+        it('should configure $provide using an array', function() {
+          function Type(PREFIX) {
+            this.prefix = PREFIX;
+          };
+          Type.prototype.$get = function() {
+            return this.prefix + 'def';
+          };
+          expect(createInjector([function($provide) {
+            $provide.constant('PREFIX', 'abc');
+            $provide.provider('value', ['PREFIX', Type]);
+          }]).get('value')).toEqual('abcdef');
+        });
+
+
         it('should configure a set of providers', function() {
           expect(createInjector([function($provide) {
             $provide.provider({value: valueFn({$get:Array})});

Original file line number	Diff line number	Diff line change
`@@ -440,7 +440,7 @@ function createInjector(modulesToLoad) {`
`440`	`440`	`}`
`441`	`441`
`442`	`442`	`function provider(name, provider_) {`
`443`		`- if (isFunction(provider_)) {`
	`443`	`+ if (isFunction(provider_) \|\| isArray(provider_)) {`
`444`	`444`	`provider_ = providerInjector.instantiate(provider_);`
`445`	`445`	`}`
`446`	`446`	`if (!provider_.$get) {`