Merge pull request #24 from florianbelhomme/issue-22

Fix issue #22

Resources/views/Menu/foundation_breadcrumb.html.twig

@@ -1,9 +1,9 @@
 {% block root -%}
-
-    <ul class="breadcrumbs">
-        {%- for breadcrumb in breadcrumbs -%}
-            <li><a href="{{ breadcrumb.uri }}">{{ breadcrumb.label }}</a></li>
-        {%- endfor -%}
-    </ul>
-
+    {% if breadcrumbs|length > 0 -%}
+        <ul class="breadcrumbs">
+            {%- for breadcrumb in breadcrumbs -%}
+                <li><a href="{{ breadcrumb.uri }}">{{ breadcrumb.label }}</a></li>
+            {%- endfor -%}
+        </ul>
+    {%- endif %}
 {%- endblock %}

Twig/MenuExtension.php

@@ -98,9 +98,17 @@ public function renderAsBreadcrumb($menu, array $options = array())
         $itemFilterIterator = new CurrentItemFilterIterator($treeIterator, $this->matcher);
         $itemFilterIterator->rewind();
 
-        // Extract the items for the breadcrumb
+        // Watch for a current item
+        $current = $itemFilterIterator->current();
+
         $manipulator = new MenuManipulator();
-        $breadcrumbs = $manipulator->getBreadcrumbsArray($itemFilterIterator->current());
+        if ($current instanceof ItemInterface) {
+            // Extract the items for the breadcrumb
+            $breadcrumbs = $manipulator->getBreadcrumbsArray($current);
+        } else {
+            // Current item could not be located, we only send the first item
+            $breadcrumbs = $manipulator->getBreadcrumbsArray($menu);
+        }
 
         // Load the template if needed
         if (!$options['template'] instanceof \Twig_Template) {