Merge branch 'feature/15' into develop

Close #15

Author: weierophinney

.travis.yml

@@ -11,19 +11,34 @@ cache:
   directories:
     - $HOME/.composer/cache
 
+env:
+  global:
+    - CODE_VERSION="^3.0"
+
 matrix:
   fast_finish: true
   include:
     - php: 5.5
       env:
         - EXECUTE_CS_CHECK=true
+    - php: 5.5
+      env:
+        - CODE_VERSION="^2.6"
     - php: 5.6
       env:
         - EXECUTE_TEST_COVERALLS=true
+    - php: 5.6
+      env:
+        - CODE_VERSION="^2.6"
     - php: 7
-    - php: hhvm 
-  allow_failures:
     - php: 7
+      env:
+        - CODE_VERSION="^2.6"
+    - php: hhvm
+    - php: hhvm
+      env:
+        - CODE_VERSION="^2.6"
+  allow_failures:
     - php: hhvm
 
 notifications:
@@ -34,6 +49,7 @@ before_install:
   - if [[ $EXECUTE_TEST_COVERALLS != 'true' ]]; then phpenv config-rm xdebug.ini || return 0 ; fi
   - composer self-update
   - if [[ $EXECUTE_TEST_COVERALLS == 'true' ]]; then composer require --dev --no-update satooshi/php-coveralls ; fi
+  - composer require --no-update "zendframework/zend-code:$CODE_VERSION"
 
 install:
   - travis_retry composer install --no-interaction --ignore-platform-reqs

CHANGELOG.md

@@ -0,0 +1,41 @@
+# Changelog
+
+All notable changes to this project will be documented in this file, in reverse chronological order by release.
+
+## 2.6.0 - 2016-02-23
+
+### Added
+
+- [#16](https://github.com/zendframework/zend-di/pull/16) adds container-interop
+  as a dependency, and updates the `LocatorInterface` to extend
+  `Interop\Container\ContainerInterface`. This required adding the following
+  methods:
+  - `Zend\Di\Di::has()`
+  - `Zend\Di\ServiceLocator::has()`
+
+### Deprecated
+
+- Nothing.
+
+### Removed
+
+- [#15](https://github.com/zendframework/zend-di/pull/15) and
+  [#16](https://github.com/zendframework/zend-di/pull/16) remove most
+  development dependencies, as the functionality could be reproduced with
+  generic test assets or PHP built-in classes. These include:
+  - zend-config
+  - zend-db
+  - zend-filter
+  - zend-log
+  - zend-mvc
+  - zend-view
+  - zend-servicemanager
+
+### Fixed
+
+- [#16](https://github.com/zendframework/zend-di/pull/16) updates the try/catch
+  block in `Zend\Di\Di::resolveMethodParameters()` to catch container-interop
+  exceptions instead of the zend-servicemanager-specific exception class. Since
+  all zend-servicemanager exceptions derive from container-interop, this
+  provides more flexibility in using any container-interop implementation as a
+  peering container.

composer.json

@@ -13,31 +13,21 @@
         }
     },
     "require": {
-        "php": ">=5.5",
-        "zendframework/zend-code": "~2.5",
-        "zendframework/zend-stdlib": "dev-develop as 2.8.0"
+        "php": "^5.5 || ^7.0",
+        "container-interop/container-interop": "^1.1",
+        "zendframework/zend-code": "^2.6 || ^3.0",
+        "zendframework/zend-stdlib": "^2.7 || ^3.0"
     },
     "require-dev": {
-        "zendframework/zend-config": "dev-develop as 2.6.0",
-        "zendframework/zend-db": "dev-develop as 2.7.0",
-        "zendframework/zend-filter": "~2.5",
-        "zendframework/zend-form": "~2.5",
-        "zendframework/zend-log": "~2.5",
-        "zendframework/zend-mvc": "dev-develop as 2.7.0",
-        "zendframework/zend-servicemanager": "dev-develop as 2.7.0",
-        "zendframework/zend-view": "dev-develop as 2.6.0",
         "fabpot/php-cs-fixer": "1.7.*",
         "phpunit/PHPUnit": "~4.0"
     },
-    "suggest": {
-        "zendframework/zend-servicemanager": "Zend\\ServiceManager component"
-    },
     "minimum-stability": "dev",
     "prefer-stable": true,
     "extra": {
         "branch-alias": {
             "dev-master": "2.5-dev",
-            "dev-develop": "3.0-dev"
+            "dev-develop": "2.6-dev"
         }
     },
     "autoload-dev": {

src/Di.php

@@ -10,8 +10,8 @@
 namespace Zend\Di;
 
 use Closure;
+use Interop\Container\Exception\ContainerException;
 use Zend\Di\Exception\RuntimeException as DiRuntimeException;
-use Zend\ServiceManager\Exception\ExceptionInterface as ServiceManagerException;
 
 /**
  * Dependency injector that can generate instances using class definitions and configured instance parameters
@@ -212,6 +212,24 @@ protected function getCallParameters($name, array $params, $method = "__construc
         return $params;
     }
 
+    /**
+     * Is the DI container capable of returning the named instance?
+     *
+     * @param string $name
+     * @return bool
+     */
+    public function has($name)
+    {
+        $definitions     = $this->definitions;
+        $instanceManager = $this->instanceManager();
+
+        $class = $instanceManager->hasAlias($name)
+            ? $instanceManager->getClassFromAlias($name)
+            : $name;
+
+        return $definitions->hasClass($class);
+    }
+
     /**
      * Lazy-load a class
      *
@@ -831,8 +849,8 @@ protected function resolveMethodParameters($class, $method, array $callTimeUserP
                         }
                         return false;
                     }
-                } catch (ServiceManagerException $e) {
-                    // Zend\ServiceManager\Exception\ServiceNotCreatedException
+                } catch (ContainerException $e) {
+                    // Exceptions thrown by nested/peered containers (e.g., zend-servicemanager)
                     if ($methodRequirementType & self::RESOLVE_STRICT) {
                         //finally ( be aware to do at the end of flow)
                         array_pop($this->currentDependencies);

src/LocatorInterface.php

@@ -9,14 +9,8 @@
 
 namespace Zend\Di;
 
-interface LocatorInterface
+use Interop\Container\ContainerInterface;
+
+interface LocatorInterface extends ContainerInterface
 {
-    /**
-     * Retrieve a class instance
-     *
-     * @param  string      $name   Class name or service name
-     * @param  null|array  $params Parameters to be used when instantiating a new instance of $name
-     * @return object|null
-     */
-    public function get($name, array $params = []);
 }

src/ServiceLocator.php

@@ -52,6 +52,23 @@ public function set($name, $service)
         return $this;
     }
 
+    /**
+     * Can the locator return the named instance?
+     *
+     * @param string $name
+     * @return bool
+     */
+    public function has($name)
+    {
+        if (! isset($this->services[$name])
+            && ! isset($this->map[$name])
+        ) {
+            return false;
+        }
+
+        return true;
+    }
+
     /**
      * Retrieve a registered service
      *

test/ConfigTest.php

@@ -11,15 +11,15 @@
 
 use Zend\Di\Config;
 use Zend\Di\Di;
-use Zend\Config\Factory as ConfigFactory;
 use PHPUnit_Framework_TestCase as TestCase;
 
 class ConfigTest extends TestCase
 {
     public function testConfigCanConfigureInstanceManagerWithIniFile()
     {
-        $ini = ConfigFactory::fromFile(__DIR__ . '/_files/sample.ini', true)->get('section-a');
-        $config = new Config($ini->di);
+        $ini = include __DIR__ . '/_files/sample-definitions.php';
+        $ini = $ini['section-a'];
+        $config = new Config($ini['di']);
         $di = new Di();
         $di->configure($config);
 
@@ -52,8 +52,9 @@ public function testConfigCanConfigureInstanceManagerWithIniFile()
     public function testConfigCanConfigureBuilderDefinitionFromIni()
     {
         $this->markTestIncomplete('Builder not updated to new DI yet');
-        $ini = ConfigFactory::fromFile(__DIR__ . '/_files/sample.ini', true)->get('section-b');
-        $config = new Config($ini->di);
+        $ini = include __DIR__ . '/_files/sample-definitions.php';
+        $ini = $ini['section-b'];
+        $config = new Config($ini['di']);
         $di = new Di($config);
         $definition = $di->getDefinition();
 
@@ -62,27 +63,28 @@ public function testConfigCanConfigureBuilderDefinitionFromIni()
         $this->assertEquals(
             ['username' => null, 'password' => null],
             $definition->getInjectionMethodParameters('My\DbAdapter', '__construct')
-            );
+        );
 
         $this->assertTrue($definition->hasClass('My\Mapper'));
         $this->assertEquals('__construct', $definition->getInstantiator('My\Mapper'));
         $this->assertEquals(
             ['dbAdapter' => 'My\DbAdapter'],
             $definition->getInjectionMethodParameters('My\Mapper', '__construct')
-            );
+        );
 
         $this->assertTrue($definition->hasClass('My\Repository'));
         $this->assertEquals('__construct', $definition->getInstantiator('My\Repository'));
         $this->assertEquals(
             ['mapper' => 'My\Mapper'],
             $definition->getInjectionMethodParameters('My\Repository', '__construct')
-            );
+        );
     }
 
     public function testConfigCanConfigureRuntimeDefinitionDefaultFromIni()
     {
-        $ini = ConfigFactory::fromFile(__DIR__ . '/_files/sample.ini', true)->get('section-c');
-        $config = new Config($ini->di);
+        $ini = include __DIR__ . '/_files/sample-definitions.php';
+        $ini = $ini['section-c'];
+        $config = new Config($ini['di']);
         $di = new Di();
         $di->configure($config);
         $definition = $di->definitions()->getDefinitionByType('Zend\Di\Definition\RuntimeDefinition');
@@ -92,8 +94,9 @@ public function testConfigCanConfigureRuntimeDefinitionDefaultFromIni()
 
     public function testConfigCanConfigureRuntimeDefinitionDisabledFromIni()
     {
-        $ini = ConfigFactory::fromFile(__DIR__ . '/_files/sample.ini', true)->get('section-d');
-        $config = new Config($ini->di);
+        $ini = include __DIR__ . '/_files/sample-definitions.php';
+        $ini = $ini['section-d'];
+        $config = new Config($ini['di']);
         $di = new Di();
         $di->configure($config);
         $definition = $di->definitions()->getDefinitionByType('Zend\Di\Definition\RuntimeDefinition');
@@ -102,8 +105,9 @@ public function testConfigCanConfigureRuntimeDefinitionDisabledFromIni()
 
     public function testConfigCanConfigureRuntimeDefinitionUseAnnotationFromIni()
     {
-        $ini = ConfigFactory::fromFile(__DIR__ . '/_files/sample.ini', true)->get('section-e');
-        $config = new Config($ini->di);
+        $ini = include __DIR__ . '/_files/sample-definitions.php';
+        $ini = $ini['section-e'];
+        $config = new Config($ini['di']);
         $di = new Di();
         $di->configure($config);
         $definition = $di->definitions()->getDefinitionByType('Zend\Di\Definition\RuntimeDefinition');
@@ -112,8 +116,8 @@ public function testConfigCanConfigureRuntimeDefinitionUseAnnotationFromIni()
 
     public function testConfigCanConfigureCompiledDefinition()
     {
-        $config = ConfigFactory::fromFile(__DIR__ . '/_files/sample.php', true);
-        $config = new Config($config->di);
+        $config = include __DIR__ . '/_files/sample.php';
+        $config = new Config($config['di']);
         $di = new Di();
         $di->configure($config);
         $definition = $di->definitions()->getDefinitionByType('Zend\Di\Definition\ArrayDefinition');

test/Definition/BuilderDefinitionTest.php

@@ -11,7 +11,6 @@
 
 use Zend\Di\Definition\BuilderDefinition;
 use Zend\Di\Definition\Builder;
-use Zend\Config\Factory as ConfigFactory;
 use PHPUnit_Framework_TestCase as TestCase;
 
 class BuilderDefinitionTest extends TestCase
@@ -73,7 +72,7 @@ public function testBuilderDefinitionHasMethods()
 
     public function testBuilderCanBuildFromArray()
     {
-        $ini = ConfigFactory::fromFile(__DIR__ . '/../_files/sample.ini');
+        $ini = include __DIR__ . '/../_files/sample-definitions.php';
         $iniAsArray = $ini['section-b'];
         $definitionArray = $iniAsArray['di']['definitions'][1];
         unset($definitionArray['class']);

test/DiCompatibilityTest.php

@@ -12,6 +12,7 @@
 use BadMethodCallException;
 use Exception;
 use PHPUnit_Framework_Error;
+use SplStack;
 use Zend\Di\Di;
 
 class DiCompatibilityTest extends \PHPUnit_Framework_TestCase
@@ -41,13 +42,9 @@ public function testDiSimple($class)
     public function providesSimpleClasses()
     {
         return [
-            ['Zend\Di\Di'],
-            ['Zend\EventManager\EventManager'],
-            ['Zend\Filter\ToNull'],
-            ['Zend\Form\Form'],
-            ['Zend\Log\Logger'],
-            ['Zend\Stdlib\SplStack'],
-            ['Zend\View\Model\ViewModel'],
+            [Di::class],
+            [SplStack::class],
+            [TestAsset\BasicClass::class],
         ];
     }
 
@@ -123,21 +120,9 @@ public function testCanCreateInstanceWithConstructorRequiredParameterWithDi($cla
 
     public function providesClassWithConstructionParameters()
     {
-        $serviceManager = new \Zend\ServiceManager\ServiceManager;
-        $eventManager   = new \Zend\EventManager\EventManager;
-        $request        = $this->getMockBuilder('Zend\Stdlib\RequestInterface')->getMock();
-        $response       = $this->getMockBuilder('Zend\Stdlib\ResponseInterface')->getMock();
-
         return [
-            ['Zend\Config\Config', ['array' => []]],
-            ['Zend\Db\Adapter\Adapter', ['driver' => ['driver' => 'Pdo_Sqlite']]],
-            ['Zend\Mvc\Application', [
-                'configuration'  => [],
-                'serviceManager' => $serviceManager,
-                'events'         => $eventManager,
-                'request'        => $request,
-                'response'       => $response
-            ]],
+            [TestAsset\BasicClassWithParam::class, ['foo' => 'bar']],
+            [TestAsset\ConstructorInjection\X::class, ['one' => 1, 'two' => 2]],
         ];
     }
 }