NGSTACK-906 eztags priority sorting

bundle/API/Repository/Values/Enums/TagSortBy.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Netgen\TagsBundle\API\Repository\Values\Enums;
+
+enum TagSortBy: string
+{
+    case ID = 'id';
+    case KEYWORD = 'keyword';
+    case MODIFIED = 'modified';
+    case PRIORITY = 'priority';
+}

bundle/API/Repository/Values/Enums/TagSortOrder.php

@@ -0,0 +1,11 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Netgen\TagsBundle\API\Repository\Values\Enums;
+
+enum TagSortOrder: string
+{
+    case ASC = 'asc';
+    case DESC = 'desc';
+}

bundle/API/Repository/Values/Tags/Tag.php

@@ -6,6 +6,8 @@
 
 use DateTimeInterface;
 use Ibexa\Contracts\Core\Repository\Values\ValueObject;
+use Netgen\TagsBundle\API\Repository\Values\Enums\TagSortBy;
+use Netgen\TagsBundle\API\Repository\Values\Enums\TagSortOrder;
 
 use function array_map;
 use function count;
@@ -28,6 +30,9 @@
  * @property-read bool $alwaysAvailable Indicates if the Tag object is shown in the main language if it is not present in an other requested language
  * @property-read string $mainLanguageCode The main language code of the Tag object
  * @property-read string[] $languageCodes List of languages in this Tag object
+ * @property-read int $priority Tag priority
+ * @property-read TagSortBy $sortBy Specifies by which property the child tags should be sorted on
+ * @property-read TagSortOrder $sortOrder Specifies whether the sort order should be ascending or descending
  */
 final class Tag extends ValueObject
 {
@@ -104,6 +109,23 @@ final class Tag extends ValueObject
      */
     protected ?string $prioritizedLanguageCode;
 
+    /**
+     * Tag priority.
+     *
+     * Position of the Tag among its siblings when sorted by priority.
+     */
+    protected int $priority;
+
+    /**
+     * Specifies by which property the child tags should be sorted on.
+     */
+    protected TagSortBy $sortBy;
+
+    /**
+     * Specifies whether the sort order should be ascending or descending.
+     */
+    protected TagSortOrder $sortOrder;
+
     /**
      * Construct object optionally with a set of properties.
      *

bundle/API/Repository/Values/Tags/TagStruct.php

@@ -5,6 +5,8 @@
 namespace Netgen\TagsBundle\API\Repository\Values\Tags;
 
 use Ibexa\Contracts\Core\Repository\Values\ValueObject;
+use Netgen\TagsBundle\API\Repository\Values\Enums\TagSortBy;
+use Netgen\TagsBundle\API\Repository\Values\Enums\TagSortOrder;
 
 use function array_key_exists;
 
@@ -22,6 +24,16 @@ abstract class TagStruct extends ValueObject
      */
     public ?string $remoteId = null;
 
+    /**
+     * Specifies by which property the child tags should be sorted on.
+     */
+    public ?TagSortBy $sortBy;
+
+    /**
+     * Specifies whether the sort order should be ascending or descending.
+     */
+    public ?TagSortOrder $sortOrder;
+
     /**
      * Tag keywords in the target languages
      * Eg. array( "cro-HR" => "Hrvatska", "eng-GB" => "Croatia" ).

bundle/Controller/Admin/TagController.php

@@ -7,7 +7,10 @@
 use Ibexa\Contracts\Core\Repository\ContentTypeService;
 use Ibexa\Contracts\Core\Repository\Exceptions\NotFoundException;
 use Netgen\TagsBundle\API\Repository\TagsService;
+use Netgen\TagsBundle\API\Repository\Values\Enums\TagSortBy;
+use Netgen\TagsBundle\API\Repository\Values\Enums\TagSortOrder;
 use Netgen\TagsBundle\API\Repository\Values\Tags\Tag;
+use Netgen\TagsBundle\API\Repository\Values\Tags\TagUpdateStruct;
 use Netgen\TagsBundle\Core\Pagination\Pagerfanta\SearchTagsAdapter;
 use Netgen\TagsBundle\Form\Type\CopyTagsType;
 use Netgen\TagsBundle\Form\Type\LanguageSelectType;
@@ -44,7 +47,6 @@ public function showTagAction(Request $request, ?Tag $tag = null): Response
 
         if (!$tag instanceof Tag || !$tag->isSynonym()) {
             $configResolver = $this->getConfigResolver();
-
             $currentPage = (int) $request->query->get('page');
             $pager = $this->createPager(
                 $this->tagChildrenAdapter,
@@ -264,6 +266,26 @@ public function updateTagAction(Request $request, Tag $tag, string $languageCode
         );
     }
 
+    public function updateTagSortAction(Request $request, Tag $tag): Response
+    {
+        if (!$this->isCsrfTokenValid('netgen_tags_admin', (string) ($request->request->get('_csrf_token') ?? ''))) {
+            $this->addFlashMessage('errors', 'invalid_csrf_token');
+
+            return $this->redirectToTag($tag);
+        }
+
+        $sortBy = $request->request->get('sort_by');
+        $sortOrder = $request->request->get('sort_order');
+
+        $tagUpdateStruct = new TagUpdateStruct();
+        $tagUpdateStruct->sortBy = TagSortBy::from((string) $sortBy);
+        $tagUpdateStruct->sortOrder = TagSortOrder::from((string) $sortOrder);
+
+        $this->tagsService->updateTag($tag, $tagUpdateStruct);
+
+        return $this->redirectToTag($tag);
+    }
+
     /**
      * This method is called for delete tag or synonym action.
      * It shows a confirmation view.

bundle/Core/Pagination/Pagerfanta/ChildrenTagsAdapter.php

@@ -11,10 +11,9 @@
 final class ChildrenTagsAdapter implements AdapterInterface, TagAdapterInterface
 {
     private ?Tag $tag = null;
-
     private int $nbResults;
 
-    public function __construct(private TagsService $tagsService) {}
+    public function __construct(private readonly TagsService $tagsService) {}
 
     public function setTag(Tag $tag): void
     {
@@ -30,7 +29,11 @@ public function getNbResults(): int
 
     public function getSlice($offset, $length): iterable
     {
-        $childrenTags = $this->tagsService->loadTagChildren($this->tag, $offset, $length);
+        $childrenTags = $this->tagsService->loadTagChildren(
+            $this->tag,
+            $offset,
+            $length,
+        );
 
         $this->nbResults = $this->nbResults ?? $this->tagsService->getTagChildrenCount($this->tag);
 

bundle/Core/Persistence/Legacy/Tags/Gateway/DoctrineDatabase.php

@@ -128,6 +128,10 @@ public function getFullTagDataByKeywordAndParentId(string $keyword, int $parentI
 
     public function getChildren(int $tagId, int $offset = 0, int $limit = -1, ?array $translations = null, bool $useAlwaysAvailable = true): array
     {
+        $tagData = $tagId !== 0 ? $this->getBasicTagData($tagId) : [];
+        $sortBy = $tagData['sort_by'] ?? 'id';
+        $sortOrder = $tagData['sort_order'] ?? 'asc';
+
         $tagIdsQuery = $this->createTagIdsQuery($translations, $useAlwaysAvailable);
         $tagIdsQuery->andWhere(
             $tagIdsQuery->expr()->andX(
@@ -137,10 +141,11 @@ public function getChildren(int $tagId, int $offset = 0, int $limit = -1, ?array
                 ),
                 $tagIdsQuery->expr()->eq('eztags.main_tag_id', 0),
             ),
-        )->setParameter('parent_id', $tagId, Types::INTEGER)
-        ->orderBy('eztags.keyword', 'ASC')
-        ->setFirstResult($offset)
-        ->setMaxResults($limit > 0 ? $limit : PHP_INT_MAX);
+        )
+            ->orderBy('eztags.' . $sortBy, $sortOrder)
+            ->setParameter('parent_id', $tagId, Types::INTEGER)
+            ->setFirstResult($offset)
+            ->setMaxResults($limit > 0 ? $limit : PHP_INT_MAX);
 
         $statement = $tagIdsQuery->execute();
 
@@ -160,8 +165,8 @@ public function getChildren(int $tagId, int $offset = 0, int $limit = -1, ?array
                 [':id'],
             ),
         )
-        ->setParameter('id', $tagIds, Connection::PARAM_INT_ARRAY)
-        ->orderBy('eztags_keyword.keyword', 'ASC');
+            ->orderBy('eztags.' . $sortBy, $sortOrder)
+            ->setParameter('id', $tagIds, Connection::PARAM_INT_ARRAY);
 
         return $query->execute()->fetchAll(FetchMode::ASSOCIATIVE);
     }
@@ -446,6 +451,12 @@ public function update(UpdateStruct $updateStruct, int $tagId): void
             )->set(
                 'language_mask',
                 ':language_mask',
+            )->set(
+                'sort_by',
+                ':sort_by',
+            )->set(
+                'sort_order',
+                ':sort_order',
             )->where(
                 $query->expr()->eq(
                     'id',
@@ -469,7 +480,9 @@ public function update(UpdateStruct $updateStruct, int $tagId): void
                     is_bool($updateStruct->alwaysAvailable) ? $updateStruct->alwaysAvailable : true,
                 ),
                 Types::INTEGER,
-            );
+            )
+            ->setParameter('sort_by', $updateStruct->sortBy->value ?? 'id', Types::STRING)
+            ->setParameter('sort_order', $updateStruct->sortOrder->value ?? 'asc', Types::STRING);
 
         $query->execute();
 
@@ -847,7 +860,7 @@ public function deleteTag(int $tagId): void
     private function createTagIdsQuery(?array $translations = null, bool $useAlwaysAvailable = true): QueryBuilder
     {
         $query = $this->connection->createQueryBuilder();
-        $query->select('DISTINCT eztags.id, eztags.keyword')
+        $query->select('DISTINCT eztags.id, eztags.keyword, eztags.modified, eztags.priority')
         ->from('eztags', 'eztags')
         // @todo: Joining with eztags_keyword is probably a VERY bad way to gather that information
         // since it creates an additional cartesian product with translations.
@@ -930,6 +943,9 @@ private function createTagFindQuery(?array $translations = null, bool $useAlways
             'eztags.remote_id',
             'eztags.main_language_id',
             'eztags.language_mask',
+            'eztags.priority',
+            'eztags.sort_by',
+            'eztags.sort_order',
             // Tag keywords
             'eztags_keyword.keyword',
             'eztags_keyword.locale',

bundle/Core/Persistence/Legacy/Tags/Mapper.php

@@ -6,6 +6,8 @@
 
 use Ibexa\Contracts\Core\Persistence\Content\Language\Handler as LanguageHandler;
 use Ibexa\Core\Persistence\Legacy\Content\Language\MaskGenerator as LanguageMaskGenerator;
+use Netgen\TagsBundle\API\Repository\Values\Enums\TagSortBy;
+use Netgen\TagsBundle\API\Repository\Values\Enums\TagSortOrder;
 use Netgen\TagsBundle\SPI\Persistence\Tags\Tag;
 use Netgen\TagsBundle\SPI\Persistence\Tags\TagInfo;
 
@@ -35,6 +37,9 @@ public function createTagInfoFromRow(array $row): TagInfo
         $tagInfo->alwaysAvailable = (bool) ((int) $row['language_mask'] & 1);
         $tagInfo->mainLanguageCode = $this->languageHandler->load($row['main_language_id'])->languageCode;
         $tagInfo->languageIds = $this->languageMaskGenerator->extractLanguageIdsFromMask((int) $row['language_mask']);
+        $tagInfo->priority = (int) $row['priority'];
+        $tagInfo->sortBy = TagSortBy::from($row['sort_by']);
+        $tagInfo->sortOrder = TagSortOrder::from($row['sort_order']);
 
         return $tagInfo;
     }
@@ -60,6 +65,9 @@ public function extractTagListFromRows(array $rows): array
                 $tag->alwaysAvailable = (bool) ((int) $row['language_mask'] & 1);
                 $tag->mainLanguageCode = $this->languageHandler->load($row['main_language_id'])->languageCode;
                 $tag->languageIds = $this->languageMaskGenerator->extractLanguageIdsFromMask((int) $row['language_mask']);
+                $tag->priority = (int) $row['priority'];
+                $tag->sortBy = TagSortBy::from($row['sort_by']);
+                $tag->sortOrder = TagSortOrder::from($row['sort_order']);
                 $tagList[$tagId] = $tag;
             }
 

bundle/Core/Repository/TagsMapper.php

@@ -67,6 +67,9 @@ public function buildTagDomainList(array $spiTags, array $prioritizedLanguages =
                     'mainLanguageCode' => $spiTag->mainLanguageCode,
                     'languageCodes' => $languageCodes,
                     'prioritizedLanguageCode' => $prioritizedLanguageCode,
+                    'priority' => $spiTag->priority,
+                    'sortBy' => $spiTag->sortBy,
+                    'sortOrder' => $spiTag->sortOrder,
                 ],
             );
         }

bundle/Core/Repository/TagsService.php

@@ -453,6 +453,8 @@ public function updateTag(Tag $tag, TagUpdateStruct $tagUpdateStruct): Tag
         $updateStruct->remoteId = trim($tagUpdateStruct->remoteId ?? $spiTag->remoteId);
         $updateStruct->mainLanguageCode = $mainLanguageCode;
         $updateStruct->alwaysAvailable = $tagUpdateStruct->alwaysAvailable ?? $spiTag->alwaysAvailable;
+        $updateStruct->sortBy = $tagUpdateStruct->sortBy ?? $spiTag->sortBy;
+        $updateStruct->sortOrder = $tagUpdateStruct->sortOrder ?? $spiTag->sortOrder;
 
         $this->repository->beginTransaction();
 

bundle/Resources/config/routing/admin/tag.yaml

@@ -43,6 +43,11 @@ netgen_tags_admin_tag_update_select:
     controller: netgen_tags.admin.controller.tag:updateTagSelectAction
     methods: [GET, POST]
 
+netgen_tags_admin_tag_update_sort:
+    path: /{tagId}/update/sort
+    controller: netgen_tags.admin.controller.tag:updateTagSortAction
+    methods: [POST]
+
 netgen_tags_admin_tag_update:
     path: /{tagId}/update/{languageCode}
     controller: netgen_tags.admin.controller.tag:updateTagAction

bundle/Resources/sql/mysql/schema.sql

@@ -9,6 +9,7 @@ CREATE TABLE `eztags` (
   `remote_id` varchar(100) NOT NULL default '',
   `main_language_id` int(11) NOT NULL default '0',
   `language_mask` int(11) NOT NULL default '0',
+  `priority` int(11) NOT NULL default '0',
   PRIMARY KEY ( `id` ),
   KEY `idx_eztags_keyword` ( `keyword`(191) ),
   KEY `idx_eztags_keyword_id` ( `keyword`(191), `id` ),

bundle/Resources/sql/postgresql/schema.sql

@@ -10,6 +10,7 @@ CREATE TABLE eztags (
    remote_id varchar(100) NOT NULL default '',
    main_language_id integer not null default 0,
    language_mask integer not null default 0,
+   priority integer not null default 0,
    PRIMARY KEY (id),
    CONSTRAINT idx_eztags_remote_id UNIQUE  (remote_id)
 );

bundle/Resources/translations/netgen_tags_admin.en.yml

@@ -11,6 +11,7 @@ tag.remote_id: 'Remote ID'
 tag.tag_name: 'Keyword'
 tag.translations: 'Translations'
 tag.modified: 'Modified'
+tag.priority: 'Priority'
 tag.main_tag: 'Main tag'
 tag.parent_tag: 'Parent tag'
 
@@ -43,6 +44,12 @@ tag.children.no_children: 'No tags found'
 tag.subtree_limitations.title: 'Subtree limitations'
 tag.subtree_limitations.no_limitations: 'No subtree limitations'
 
+tag.subitems.title: 'Subitems'
+tag.subitems.sort_by: 'Sort by'
+tag.subitems.sort_order: 'Sort order'
+tag.subitems.sort_order.ascending: 'Ascending'
+tag.subitems.sort_order.descending: 'Descending'
+
 tag.related_content.title: 'List of content related to tag'
 tag.related_content.filter.content_type: 'Content type filter'
 tag.related_content.filter.sort: 'Sort'

bundle/Resources/translations/netgen_tags_admin.fr.yml

@@ -11,6 +11,7 @@ tag.remote_id: 'Identification à distance'
 tag.tag_name: 'Mot-clé'
 tag.translations: 'Traductions'
 tag.modified: 'Modifié'
+tag.priority: 'Priorité'
 tag.main_tag: 'Tag principale'
 tag.parent_tag: 'Tag parent'
 
@@ -43,6 +44,12 @@ tag.children.no_children: 'Pas de tags trouvés'
 tag.subtree_limitations.title: 'Limites des sous-arbres'
 tag.subtree_limitations.no_limitations: 'Aucune limite de sous-arbre'
 
+tag.subitems.title: 'Sous-éléments'
+tag.subitems.sort_by: 'Trier par'
+tag.subitems.sort_order: 'Ordre de tri'
+tag.subitems.sort_order.ascending: 'Ascendant'
+tag.subitems.sort_order.descending: 'Descendant'
+
 tag.related_content.title: 'Liste des contenus liés au tag'
 
 tag.add.title: 'Nouveau tag'

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

@@ -26,6 +26,7 @@
                     <th>{{ 'tag.tag_name'|trans }}</th>
                     <th>{{ 'tag.translations'|trans }}</th>
                     <th>{{ 'tag.modified'|trans }}</th>
+                    <th>{{ 'tag.priority'|trans }}</th>
                 </tr>
             </thead>
             <tbody>
@@ -48,6 +49,7 @@
                             {% endfor %}
                         </td>
                         <td>{{ child.modificationDate|date }}</td>
+                        <td>{{ child.priority }}</td>
                     </tr>
                 {% endfor %}
             </tbody>
@@ -71,6 +73,10 @@
             {% endif %}
         </div>
     </form>
+
+    {% if tag is defined and not tag.isSynonym %}
+        {% include '@NetgenTags/admin/tag/sort_children_tags.html.twig' %}
+    {% endif %}
 {% else %}
     <p>{{ 'tag.children.no_children'|trans }}</p>
 {% endif %}