Add breadcrumb for credit memo, fix refund breadcrumb

Sylius · Jan 13, 2020 · 7bd9ed7 · 7bd9ed7
1 parent 63842d4
commit 7bd9ed7
Show file tree

Hide file tree

Showing 11 changed files with 28 additions and 73 deletions.
diff --git a/src/Resources/config/admin_routing.yml b/src/Resources/config/admin_routing.yml
@@ -73,13 +73,3 @@ sylius_refund_admin_credit_memo_send:
     methods: [GET]
     defaults:
         _controller: Sylius\RefundPlugin\Action\Admin\SendCreditMemoAction
-
-sylius_refund_admin_order_show_by_number:
-    path: /orders/by-number/{number}
-    methods: [GET]
-    defaults:
-        _controller: Sylius\RefundPlugin\Ui\RedirectToOrderShowAction
-        _sylius:
-            section: admin
-            permission: true
-            alias: refund_plugin
diff --git a/src/Resources/config/app/config.yml b/src/Resources/config/app/config.yml
@@ -65,7 +65,7 @@ sylius_grid:
                 orderNumber:
                     type: twig
                     label: sylius.ui.order
-                    path: order.number
+                    path: .
                     sortable: ~
                     options:
                         template: "@SyliusRefundPlugin/Order/Grid/Field/number.html.twig"

diff --git a/src/Resources/config/doctrine/CreditMemo.orm.xml b/src/Resources/config/doctrine/CreditMemo.orm.xml
@@ -21,7 +21,7 @@
             <join-column name="channel_id" referenced-column-name="id" nullable="true" />
         </many-to-one>
 
-        <many-to-one field="order" target-entity="Sylius\Component\Core\Model\OrderInterface">
+        <many-to-one field="order" target-entity="Sylius\Component\Order\Model\OrderInterface">
             <join-column name="order_id" referenced-column-name="id" nullable="true" />
         </many-to-one>
 

diff --git a/src/Resources/config/services.xml b/src/Resources/config/services.xml
@@ -75,11 +75,6 @@
             <argument type="service" id="sylius.manager.order" />
         </service>
 
-        <service id="Sylius\RefundPlugin\Ui\RedirectToOrderShowAction">
-            <argument type="service" id="router" />
-            <argument type="service" id="sylius.repository.order" />
-        </service>
-
         <service id="Sylius\RefundPlugin\Checker\OrderFullyRefundedTotalChecker">
             <argument type="service" id="Sylius\RefundPlugin\Provider\OrderRefundedTotalProvider" />
         </service>

diff --git a/src/Resources/views/Order/Admin/CreditMemo/_breadcrumb.html.twig b/src/Resources/views/Order/Admin/CreditMemo/_breadcrumb.html.twig
@@ -0,0 +1,12 @@
+{% import '@SyliusAdmin/Macro/breadcrumb.html.twig' as breadcrumb %}
+
+{% set breadcrumbs = [
+    { label: 'sylius.ui.administration'|trans, url: path('sylius_admin_dashboard') },
+    { label: 'sylius.ui.orders'|trans, url: path('sylius_admin_order_index') },
+    { label: '#' ~ credit_memo.order.number, url: path('sylius_admin_order_show', {'id': credit_memo.order.id}) },
+    { label: 'sylius_refund.ui.credit_memo'|trans },
+    { label:  credit_memo.number },
+]
+%}
+
+{{ breadcrumb.crumble(breadcrumbs) }}
diff --git a/src/Resources/views/Order/Admin/CreditMemo/details.html.twig b/src/Resources/views/Order/Admin/CreditMemo/details.html.twig
@@ -10,6 +10,7 @@
     <div class="ui stackable two column grid">
         <div class="eight wide column">
             {% include '@SyliusRefundPlugin/Order/Admin/CreditMemo/_header.html.twig' %}
+            {% include '@SyliusRefundPlugin/Order/Admin/CreditMemo/_breadcrumb.html.twig' %}
         </div>
         <div class="eight wide right aligned column">
             {{ buttons.default(

diff --git a/src/Resources/views/Order/Grid/Field/number.html.twig b/src/Resources/views/Order/Grid/Field/number.html.twig
@@ -1 +1 @@
-<a href="{{ path('sylius_refund_admin_order_show_by_number', {'number': data}) }}">#{{ data }}</a>
+<a href="{{ path('sylius_admin_order_show', {'id': data.order.id}) }}">#{{ data.order.number }}</a>
diff --git a/src/Resources/views/_breadcrumb.html.twig b/src/Resources/views/_breadcrumb.html.twig
@@ -0,0 +1,11 @@
+{% import '@SyliusAdmin/Macro/breadcrumb.html.twig' as breadcrumb %}
+
+{% set breadcrumbs = [
+    { label: 'sylius.ui.administration'|trans, url: path('sylius_admin_dashboard') },
+    { label: 'sylius.ui.orders'|trans, url: path('sylius_admin_order_index') },
+    { label: '#' ~ order.number, url: path('sylius_admin_order_show', {'id': order.id}) },
+    { label: 'sylius_refund.ui.refund'|trans },
+]
+%}
+
+{{ breadcrumb.crumble(breadcrumbs) }}
diff --git a/src/Resources/views/orderRefunds.html.twig b/src/Resources/views/orderRefunds.html.twig
@@ -7,7 +7,7 @@
 
 {% block content %}
     {% include '@SyliusRefundPlugin/_header.html.twig' %}
-    {% include '@SyliusAdmin/Order/Show/_breadcrumb.html.twig' %}
+    {% include '@SyliusRefundPlugin/_breadcrumb.html.twig' %}
 
     <div class="ui stackable grid">
         <div class="sixteen wide column">

diff --git a/src/Ui/RedirectToOrderShowAction.php b/src/Ui/RedirectToOrderShowAction.php
diff --git a/tests/Application/config/packages/doctrine.yaml b/tests/Application/config/packages/doctrine.yaml
@@ -12,6 +12,3 @@ doctrine:
         charset: UTF8
 
         url: '%env(resolve:DATABASE_URL)%'
-    orm:
-        resolve_target_entities:
-            Sylius\Component\Core\Model\OrderInterface: Sylius\Component\Core\Model\Order
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		<a href="{{ path('sylius_refund_admin_order_show_by_number', {'number': data}) }}">#{{ data }}</a>
		<a href="{{ path('sylius_admin_order_show', {'id': data.order.id}) }}">#{{ data.order.number }}</a>