Added fallback for navigation container names in view helpers

- calling the navigation helper with a parameter `navigation` is confusing
- name corresponds to the documentation

Author: froschdesign

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,6 +9,9 @@
 
 namespace ZendTest\View\Helper\Navigation;
 
+use Zend\Navigation\Navigation;
+use Zend\ServiceManager\ServiceManager;
+
 class AbstractHelperTest extends AbstractTest
 {
     /**
@@ -81,4 +84,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()
+        );
+    }
 }