Merge pull request #110 from froschdesign/hotfix/navigation-fallback

Added fallback for navigation container names in view helpers

Conflicts:
	test/Helper/Navigation/AbstractHelperTest.php

Author: weierophinney

src/Helper/Navigation/AbstractHelper.php

@@ -251,6 +251,21 @@ protected function parseContainer(&$container = null)
                 ));
             }
 
+            // Fallback
+            if (in_array($container, ['default', 'navigation'], true)) {
+                // Uses class name
+                if ($services->has(Navigation\Navigation::class)) {
+                    $container = $services->get(Navigation\Navigation::class);
+                    return;
+                }
+
+                // Uses old service name
+                if ($services->has('navigation')) {
+                    $container = $services->get('navigation');
+                    return;
+                }
+            }
+
             /**
              * Load the navigation container from the root service locator
              */

test/Helper/Navigation/AbstractHelperTest.php

@@ -9,7 +9,9 @@
 
 namespace ZendTest\View\Helper\Navigation;
 
-use Zend\View\Helper\Navigation;
+use Zend\Navigation\Navigation;
+use Zend\ServiceManager\ServiceManager;
+use Zend\View\Helper\Navigation as NavigationHelper;
 
 class AbstractHelperTest extends AbstractTest
 {
@@ -19,12 +21,12 @@ class AbstractHelperTest extends AbstractTest
      *
      * @var string
      */
-    protected $_helperName = Navigation::class;
+    protected $_helperName = NavigationHelper::class;
 
     /**
      * View helper
      *
-     * @var \Zend\View\Helper\Navigation\Breadcrumbs
+     * @var NavigationHelper\Breadcrumbs
      */
     protected $_helper;
     // @codingStandardsIgnoreEnd
@@ -85,4 +87,29 @@ public function testEventManagerIsNullByDefault()
     {
         $this->assertNull($this->_helper->getEventManager());
     }
+
+    public function testFallBackForContainerNames()
+    {
+        // Register navigation service with name equal to the documentation
+        $this->serviceManager->setAllowOverride(true);
+        $this->serviceManager->setService(
+            'navigation',
+            $this->serviceManager->get('Navigation')
+        );
+        $this->serviceManager->setAllowOverride(false);
+
+        $this->_helper->setServiceLocator($this->serviceManager);
+
+        $this->_helper->setContainer('navigation');
+        $this->assertInstanceOf(
+            Navigation::class,
+            $this->_helper->getContainer()
+        );
+
+        $this->_helper->setContainer('default');
+        $this->assertInstanceOf(
+            Navigation::class,
+            $this->_helper->getContainer()
+        );
+    }
 }

test/Helper/Navigation/AbstractTest.php

@@ -32,7 +32,7 @@
 abstract class AbstractTest extends TestCase
 {
     /**
-     * @var
+     * @var ServiceManager
      */
     protected $serviceManager;