feat: improve breadcrumb for shadowcontainer and home + refacto duplicate template

Author: roadiz-ci

src/Controller/Node/NodeDuplicateController.php

@@ -8,6 +8,7 @@
 use RZ\Roadiz\CoreBundle\Entity\Node;
 use RZ\Roadiz\CoreBundle\Event\Node\NodeCreatedEvent;
 use RZ\Roadiz\CoreBundle\Event\Node\NodeDuplicatedEvent;
+use RZ\Roadiz\CoreBundle\Explorer\ExplorerItemFactoryInterface;
 use RZ\Roadiz\CoreBundle\Node\NodeDuplicator;
 use RZ\Roadiz\CoreBundle\Node\NodeNamePolicyInterface;
 use RZ\Roadiz\CoreBundle\Security\Authorization\Voter\NodeVoter;
@@ -18,6 +19,7 @@
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\HttpKernel\Attribute\AsController;
 use Symfony\Component\Routing\Attribute\Route;
+use Symfony\Component\String\UnicodeString;
 use Symfony\Contracts\EventDispatcher\EventDispatcherInterface;
 use Symfony\Contracts\Translation\TranslatorInterface;
 
@@ -26,6 +28,7 @@ final class NodeDuplicateController extends AbstractController
 {
     public function __construct(
         private readonly NodeNamePolicyInterface $nodeNamePolicy,
+        private readonly ExplorerItemFactoryInterface $explorerItemFactory,
         private readonly ManagerRegistry $managerRegistry,
         private readonly EventDispatcherInterface $eventDispatcher,
         private readonly TranslatorInterface $translator,
@@ -53,9 +56,17 @@ public function duplicateAction(
         $form->handleRequest($request);
 
         if (!$form->isSubmitted() || !$form->isValid()) {
-            return $this->render('@RoadizRozier/nodes/duplicate.html.twig', [
-                'node' => $existingNode,
+            return $this->render('@RoadizRozier/admin/confirm_action.html.twig', [
+                'title' => new UnicodeString($this->translator->trans('duplicate.node')),
+                'headPath' => '@RoadizRozier/nodes/head.html.twig',
+                'cancelPath' => $this->generateUrl('nodesEditPage', ['nodeId' => $existingNode->getId()]),
+                'alertMessage' => 'are_you_sure.duplicate.node',
+                'messageType' => 'warning',
+                'action_color' => 'success',
+                'action_label' => $this->translator->trans('duplicate'),
+                'action_icon' => 'rz-icon-ri--file-copy-line',
                 'form' => $form->createView(),
+                'items' => [$this->explorerItemFactory->createForEntity($existingNode)->toArray()],
             ]);
         }
 

templates/macros/rz_page_header.html.twig

@@ -62,13 +62,21 @@
                             {%- if breadcrumbsItem.home -%}
                                 <span class="rz-button__icon rz-icon-ri--home-2-fill"></span>
                             {%- endif -%}
+                            {% if 'list' in breadcrumbsItem.url %}
+                                <span class="rz-button__icon rz-icon-ri--list-unordered"></span>
+                            {% endif %}
                             {{- breadcrumbsItem.label|truncate_title -}}
                         </a>
                     </li>
                 {% endif %}
             {% endfor %}
             <li class="rz-breadcrumb__list-item">
-                <a href="" class="rz-breadcrumb__item" aria-current="page">{{ breadcrumb.current|truncate_title }}</a>
+                <a href="" class="rz-breadcrumb__item" aria-current="page">
+                    {%- if breadcrumb.home -%}
+                        <span class="rz-button__icon rz-icon-ri--home-2-fill"></span>
+                    {%- endif -%}
+                    {{ breadcrumb.current|truncate_title }}
+                </a>
             </li>
         </ol>
     </nav>

templates/nodes/add.html.twig

@@ -15,7 +15,7 @@
             label: 'nodes-breadcrumb'|trans,
             parent: [
                 {
-                    label: 'nodes'|trans,
+                    label: 'all.nodes'|trans,
                     type: 'listing',
                     url: path('nodesHomePage'),
                 },

templates/nodes/duplicate.html.twig

@@ -1,11 +1,24 @@
 {% if not source %}
     {% set source = node.nodeSources.first %}
 {% endif %}
+{% if node.parent.shadow == true %}
+    {% set breadcrumbParent = [source|parent(null, true)] %}
+{% else %}
+    {% set breadcrumbParent = source|parents(null, true)|reverse %}
+    {% set homePage = [{
+        type: 'listing',
+        url: path('nodesHomePage'),
+        label: 'all.nodes'|trans,
+    }] %}
+    {% set breadcrumbParent = homePage|merge(breadcrumbParent) %}
+{% endif %}
+
 {% if source is defined and source %}
     {% set breadcrumb = {
         'label': 'node-breadcrumb',
-        'parent': source|parents(null, true)|reverse,
-        'current': source.title|default(node.nodeName)
+        'parent': breadcrumbParent,
+        'current': source.title|default(node.nodeName),
+        'home': source.node.home,
     } %}
 {% endif %}
 {% embed "@RoadizRozier/macros/rz_page_header.html.twig" with {