From 2c405f417125c80c387a51baece8bf6e1e0c0a81 Mon Sep 17 00:00:00 2001
From: Sudhir Jonathan <sudhir.j@gmail.com>
Date: Tue, 27 Nov 2012 00:47:37 +0530
Subject: [PATCH] fix($injector): provider can now be defined in the array
 format

`injector.instantiate` is now called for arrays too, instead of only for functions.

Closes #1452
---
 src/auto/injector.js      |  2 +-
 test/auto/injectorSpec.js | 14 ++++++++++++++
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/src/auto/injector.js b/src/auto/injector.js
index 4f37214da526..bdc82da51a83 100644
--- a/src/auto/injector.js
+++ b/src/auto/injector.js
@@ -441,7 +441,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
index d2a637893363..6f84897bc900 100644
--- a/test/auto/injectorSpec.js
+++ b/test/auto/injectorSpec.js
@@ -394,6 +394,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})});