Skip to content

Commit a81d3f6

Browse files
committed
NGSTACK-906 update sort_children_tags template to check if sortBy or sortOrder is null and fix TagsIntegrationTest
1 parent fd507dd commit a81d3f6

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

bundle/Resources/views/admin/tag/sort_children_tags.html.twig

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,8 @@
66
<label for="sort_by">{{ 'tag.subitems.sort_by'|trans }}:</label>
77
<select id="sort_by" name="sort_by">
88
{% for case in enum_cases('Netgen\\TagsBundle\\API\\Repository\\Values\\Enums\\TagSortBy') %}
9-
<option value="{{ case.value }}" {% if tag.sortBy.value == case.value %}selected="selected"{% endif %}>
9+
<option value="{{ case.value }}"
10+
{% if tag.sortBy is defined and tag.sortBy.value == case.value %}selected="selected"{% endif %}>
1011
{{ ('tag.enum.sort_by.' ~ case.value)|trans }}
1112
</option>
1213
{% endfor %}
@@ -17,7 +18,8 @@
1718
<label for="sort_order">{{ 'tag.subitems.sort_order'|trans }}:</label>
1819
<select id="sort_order" name="sort_order">
1920
{% for case in enum_cases('Netgen\\TagsBundle\\API\\Repository\\Values\\Enums\\TagSortOrder') %}
20-
<option value="{{ case.value }}" {% if tag.sortOrder.value == case.value %}selected="selected"{% endif %}>
21+
<option value="{{ case.value }}"
22+
{% if tag.sortOrder is defined and tag.sortOrder.value == case.value %}selected="selected"{% endif %}>
2123
{{ ('tag.enum.sort_order.' ~ case.value)|trans }}
2224
</option>
2325
{% endfor %}

tests/API/Repository/FieldType/TagsIntegrationTest.php

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,6 @@
88
use Ibexa\Contracts\Core\Repository\Values\Content\Field;
99
use Ibexa\Core\Base\Exceptions\InvalidArgumentType;
1010
use Ibexa\Tests\Integration\Core\Repository\FieldType\BaseIntegrationTest;
11-
use Netgen\TagsBundle\API\Repository\Values\Enums\TagSortBy;
12-
use Netgen\TagsBundle\API\Repository\Values\Enums\TagSortOrder;
1311
use Netgen\TagsBundle\API\Repository\Values\Tags\Tag;
1412
use Netgen\TagsBundle\Core\FieldType\Tags\Type;
1513
use Netgen\TagsBundle\Core\FieldType\Tags\Value as TagsValue;
@@ -256,8 +254,8 @@ private function getTag1(): Tag
256254
'languageCodes' => ['eng-GB'],
257255
'prioritizedLanguageCode' => 'eng-GB',
258256
'priority' => 0,
259-
'sortBy' => TagSortBy::Id,
260-
'sortOrder' => TagSortOrder::Ascending,
257+
'sortBy' => null,
258+
'sortOrder' => null,
261259
],
262260
);
263261
}
@@ -282,8 +280,8 @@ private function getTag2(): Tag
282280
'languageCodes' => ['eng-GB'],
283281
'prioritizedLanguageCode' => 'eng-GB',
284282
'priority' => 0,
285-
'sortBy' => TagSortBy::Id,
286-
'sortOrder' => TagSortOrder::Ascending,
283+
'sortBy' => null,
284+
'sortOrder' => null,
287285
],
288286
);
289287
}
@@ -308,8 +306,8 @@ private function getTag3(): Tag
308306
'languageCodes' => ['eng-GB'],
309307
'prioritizedLanguageCode' => 'eng-GB',
310308
'priority' => 0,
311-
'sortBy' => TagSortBy::Id,
312-
'sortOrder' => TagSortOrder::Ascending,
309+
'sortBy' => null,
310+
'sortOrder' => null,
313311
],
314312
);
315313
}

0 commit comments

Comments
 (0)