Added separator to model in renderPartial function

library/Zend/ModuleManager/ModuleManager.php

@@ -78,6 +78,9 @@ public function onLoadModules()
         }
 
         foreach ($this->getModules() as $moduleName => $module) {
+            if ($module == null) {
+                continue;
+            }
             if (is_object($module)) {
                 if (!is_string($moduleName)) {
                     throw new Exception\RuntimeException(sprintf(

library/Zend/View/Helper/Navigation/Breadcrumbs.php

@@ -172,7 +172,10 @@ public function renderPartial($container = null, $partial = null)
         }
 
         // put breadcrumb pages in model
-        $model  = array('pages' => array());
+        $model = array(
+            'pages' => array(),
+            'separator' => $this->getSeparator()
+        );
         $active = $this->findActive($container);
         if ($active) {
             $active = $active['page'];

tests/ZendTest/ModuleManager/ModuleManagerTest.php

@@ -89,6 +89,17 @@ public function testCanLoadSomeModule()
         $this->assertSame($config->some, 'thing');
     }
 
+    public function testIgnoreNullModule()
+    {
+        $configListener = $this->defaultListeners->getConfigListener();
+        $moduleManager  = new ModuleManager(array('SomeModule', null), new EventManager);
+        $moduleManager->getEventManager()->attachAggregate($this->defaultListeners);
+        $moduleManager->loadModules();
+        $loadedModules = $moduleManager->getLoadedModules();
+        $this->assertInstanceOf('SomeModule\Module', $loadedModules['SomeModule']);
+        $this->assertSame(count($loadedModules), 1);
+    }
+
     public function testCanLoadMultipleModules()
     {
         $configListener = $this->defaultListeners->getConfigListener();

tests/ZendTest/View/Helper/Navigation/BreadcrumbsTest.php

@@ -198,6 +198,14 @@ public function testRenderingPartial()
         $this->assertEquals($expected, $this->_helper->render());
     }
 
+    public function testRenderingPartialWithSeparator()
+    {
+        $this->_helper->setPartial('bc_separator.phtml')->setSeparator(' / ');
+
+        $expected = $this->_getExpected('bc/partialwithseparator.html');
+        $this->assertEquals($expected, $this->_helper->render());
+    }
+
     public function testRenderingPartialBySpecifyingAnArrayAsPartial()
     {
         $this->_helper->setPartial(array('bc.phtml', 'application'));

tests/ZendTest/View/Helper/Navigation/_files/expected/bc/partialwithseparator.html

@@ -0,0 +1 @@
+Page 2 / Page 2.3 / Page 2.3.3 / Page 2.3.3.1

tests/ZendTest/View/Helper/Navigation/_files/mvc/views/bc_separator.phtml

@@ -0,0 +1,4 @@
+<?php
+echo implode($this->vars()->separator, array_map(
+        create_function('$a', 'return $a->getLabel();'),
+        $this->vars()->pages));