Merge pull request #167 from kokspflanze/bugfix/pdoexception_code_return_string

bugfix PDOException return string as code

Author: weierophinney

src/ServiceManager.php

@@ -755,7 +755,7 @@ private function doCreate($resolvedName, array $options = null)
                 'Service with name "%s" could not be created. Reason: %s',
                 $resolvedName,
                 $exception->getMessage()
-            ), $exception->getCode(), $exception);
+            ), (int)$exception->getCode(), $exception);
         }
 
         foreach ($this->initializers as $initializer) {

test/CommonServiceLocatorBehaviorsTrait.php

@@ -23,6 +23,7 @@
 use Zend\ServiceManager\ServiceLocatorInterface;
 use ZendTest\ServiceManager\TestAsset\FailingAbstractFactory;
 use ZendTest\ServiceManager\TestAsset\FailingFactory;
+use ZendTest\ServiceManager\TestAsset\FailingExceptionWithStringAsCodeFactory;
 use ZendTest\ServiceManager\TestAsset\InvokableObject;
 use ZendTest\ServiceManager\TestAsset\SimpleAbstractFactory;
 
@@ -227,6 +228,19 @@ public function testThrowExceptionIfServiceCannotBeCreated()
         $serviceManager->get(stdClass::class);
     }
 
+    public function testThrowExceptionWithStringAsCodeIfServiceCannotBeCreated()
+    {
+        $serviceManager = $this->createContainer([
+            'factories' => [
+                stdClass::class => FailingExceptionWithStringAsCodeFactory::class
+            ]
+        ]);
+
+        $this->setExpectedException(ServiceNotCreatedException::class);
+
+        $serviceManager->get(stdClass::class);
+    }
+
     public function testConfigureCanAddNewServices()
     {
         $serviceManager = $this->createContainer([

test/TestAsset/ExceptionWithStringAsCode.php

@@ -0,0 +1,18 @@
+<?php
+/**
+ * Zend Framework (http://framework.zend.com/)
+ *
+ * @link      http://github.com/zendframework/zf2 for the canonical source repository
+ * @copyright Copyright (c) 2005-2016 Zend Technologies USA Inc. (http://www.zend.com)
+ * @license   http://framework.zend.com/license/new-bsd New BSD License
+ */
+
+namespace ZendTest\ServiceManager\TestAsset;
+
+use Exception;
+
+class ExceptionWithStringAsCode extends Exception
+{
+    /** @var string */
+    protected $code = 'ExceptionString';
+}

test/TestAsset/FailingExceptionWithStringAsCodeFactory.php

@@ -0,0 +1,24 @@
+<?php
+/**
+ * Zend Framework (http://framework.zend.com/)
+ *
+ * @link      http://github.com/zendframework/zf2 for the canonical source repository
+ * @copyright Copyright (c) 2005-2016 Zend Technologies USA Inc. (http://www.zend.com)
+ * @license   http://framework.zend.com/license/new-bsd New BSD License
+ */
+
+namespace ZendTest\ServiceManager\TestAsset;
+
+use Interop\Container\ContainerInterface;
+use Zend\ServiceManager\Factory\FactoryInterface;
+
+class FailingExceptionWithStringAsCodeFactory implements FactoryInterface
+{
+    /**
+     * {@inheritDoc}
+     */
+    public function __invoke(ContainerInterface $container, $requestedName, array $options = null)
+    {
+        throw (new ExceptionWithStringAsCode('There is an error'));
+    }
+}