Skip to content

Commit e8b281d

Browse files
committed
Sort main and bulk actions by position (#276)
Solves #270 This issue only mentioned the main action group, but bulk actions were not sorted either
2 parents 7311f97 + 0652245 commit e8b281d

File tree

4 files changed

+4
-3
lines changed

4 files changed

+4
-3
lines changed

app/Entity/Book.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
operations: [
3636
new Create(),
3737
new Update(),
38-
new Index(grid: BookGrid::class),
38+
new Index(),
3939
new Delete(),
4040
new BulkDelete(),
4141
new Show(),

composer.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,7 @@
4949
"phpstan/phpstan": "^1.10",
5050
"phpstan/phpstan-symfony": "^1.3",
5151
"phpunit/phpunit": "^9.6",
52+
"slevomat/coding-standard": "<8.23",
5253
"sylius-labs/coding-standard": "^4.0",
5354
"symfony/browser-kit": "^6.4 || ^7.0",
5455
"symfony/console": "^6.4 || ^7.0",

src/BootstrapAdminUi/templates/shared/crud/common/content/header/title_block/actions.html.twig

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
<div class="col-12 col-md-auto ms-auto d-print-none">
44
<div class="btn-list">
55
{% if resources.definition.actionGroups.main is defined %}
6-
{% for action in resources.definition.getEnabledActions('main') %}
6+
{% for action in resources.definition.getEnabledActions('main')|sylius_sort_by('position') %}
77
{{ sylius_grid_render_action(resources, action, null) }}
88
{% endfor %}
99
{% endif %}

src/BootstrapAdminUi/templates/shared/crud/index/content/grid/data_table.html.twig

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
<div class="d-flex">
1212
{% if data|length > 0 and definition.actionGroups.bulk is defined and definition.getEnabledActions('bulk')|length > 0 %}
1313
<div class="sylius-grid-nav__bulk grid">
14-
{% for action in definition.getEnabledActions('bulk') %}
14+
{% for action in definition.getEnabledActions('bulk')|sylius_sort_by('position') %}
1515
{{ sylius_grid_render_bulk_action(resources, action, null) }}
1616
{% endfor %}
1717
</div>

0 commit comments

Comments
 (0)