From 44e8b0b7aaf623e1257303a5e203067c2a887be8 Mon Sep 17 00:00:00 2001
From: Marcel Werk <burntime@woltlab.com>
Date: Wed, 18 Feb 2026 14:29:35 +0100
Subject: [PATCH] Add option to sort labels automatically in alphabetical order

Closes #6532
---
 .../update_com.woltlab.wcf_6.3_step1.php      | 19 +++++++++++++++++++
 .../files/acp/templates/labelGroupAdd.tpl     | 13 ++++++++++---
 .../lib/acp/form/LabelGroupAddForm.class.php  |  8 ++++++++
 .../lib/acp/form/LabelGroupEditForm.class.php |  2 ++
 .../lib/data/label/group/LabelGroup.class.php |  1 +
 .../label/group/ViewableLabelGroup.class.php  | 15 +++++++++++++++
 .../cache/builder/LabelCacheBuilder.class.php |  3 ++-
 .../groups/ChangeLabelShowOrder.class.php     |  5 +++++
 .../labels/groups/GetLabelShowOrder.class.php |  5 +++++
 .../lib/system/label/LabelHandler.class.php   | 16 +++++++++++++---
 wcfsetup/install/lang/de.xml                  |  1 +
 wcfsetup/install/lang/en.xml                  |  1 +
 wcfsetup/setup/db/install.sql                 |  1 +
 13 files changed, 83 insertions(+), 7 deletions(-)
 create mode 100644 wcfsetup/install/files/acp/database/update_com.woltlab.wcf_6.3_step1.php

diff --git a/wcfsetup/install/files/acp/database/update_com.woltlab.wcf_6.3_step1.php b/wcfsetup/install/files/acp/database/update_com.woltlab.wcf_6.3_step1.php
new file mode 100644
index 00000000000..340c3ced66c
--- /dev/null
+++ b/wcfsetup/install/files/acp/database/update_com.woltlab.wcf_6.3_step1.php
@@ -0,0 +1,19 @@
+<?php
+
+/**
+ * Updates the database layout during the update from 6.2 to 6.3.
+ *
+ * @author    Marcel Werk
+ * @copyright 2001-2026 WoltLab GmbH
+ * @license   GNU Lesser General Public License <http://opensource.org/licenses/lgpl-license.php>
+ */
+
+use wcf\system\database\table\column\DefaultFalseBooleanDatabaseTableColumn;
+use wcf\system\database\table\PartialDatabaseTable;
+
+return [
+    PartialDatabaseTable::create('wcf1_label_group')
+        ->columns([
+            DefaultFalseBooleanDatabaseTableColumn::create('sortAlphabetically')
+        ]),
+];
diff --git a/wcfsetup/install/files/acp/templates/labelGroupAdd.tpl b/wcfsetup/install/files/acp/templates/labelGroupAdd.tpl
index d67873c4635..78779b2195f 100644
--- a/wcfsetup/install/files/acp/templates/labelGroupAdd.tpl
+++ b/wcfsetup/install/files/acp/templates/labelGroupAdd.tpl
@@ -23,7 +23,9 @@
 	<nav class="contentHeaderNavigation">
 		<ul>
 			{if $action == 'edit'}
-				<li><button type="button" class="button jsChangeShowOrder">{icon name='up-down'} <span>{lang}wcf.global.changeShowOrder{/lang}</span></button></li>
+				{if !$labelGroup->sortAlphabetically}
+					<li><button type="button" class="button jsChangeShowOrder">{icon name='up-down'} <span>{lang}wcf.global.changeShowOrder{/lang}</span></button></li>
+				{/if}
 				<li>
 					{unsafe:$interactionContextMenu->render()}
 				</li>
@@ -84,7 +86,12 @@
 					<dt></dt>
 					<dd><label><input type="checkbox" name="forceSelection" id="forceSelection" value="1"{if $labelForceSelection} checked{/if}> {lang}wcf.acp.label.group.forceSelection{/lang}</label></dd>
 				</dl>
-				
+
+				<dl>
+					<dt></dt>
+					<dd><label><input type="checkbox" name="sortAlphabetically" id="sortAlphabetically" value="1"{if $sortAlphabetically} checked{/if}> {lang}wcf.acp.label.group.sortAlphabetically{/lang}</label></dd>
+				</dl>
+
 				<dl id="groupPermissions">
 					<dt>{lang}wcf.acl.permissions{/lang}</dt>
 					<dd></dd>
@@ -125,7 +132,7 @@
 	</div>
 </form>
 
-{if $action == 'edit'}
+{if $action == 'edit' && !$labelGroup->sortAlphabetically}
 	<script data-relocate="true">
 		require(["WoltLabSuite/Core/Component/ChangeShowOrder"], ({ setup }) => {
 			{jsphrase name='wcf.global.changeShowOrder'}
diff --git a/wcfsetup/install/files/lib/acp/form/LabelGroupAddForm.class.php b/wcfsetup/install/files/lib/acp/form/LabelGroupAddForm.class.php
index f0f12ee1b4d..8038418c427 100644
--- a/wcfsetup/install/files/lib/acp/form/LabelGroupAddForm.class.php
+++ b/wcfsetup/install/files/lib/acp/form/LabelGroupAddForm.class.php
@@ -40,6 +40,8 @@ class LabelGroupAddForm extends AbstractForm
      */
     public $forceSelection = false;
 
+    public bool $sortAlphabetically = false;
+
     /**
      * group name
      * @var string
@@ -113,6 +115,9 @@ public function readFormParameters()
         if (isset($_POST['forceSelection'])) {
             $this->forceSelection = true;
         }
+        if (isset($_POST['sortAlphabetically'])) {
+            $this->sortAlphabetically = true;
+        }
         if (isset($_POST['objectTypes']) && \is_array($_POST['objectTypes'])) {
             $this->objectTypes = $_POST['objectTypes'];
         }
@@ -179,6 +184,7 @@ public function save()
         $this->objectAction = new LabelGroupAction([], 'create', [
             'data' => \array_merge($this->additionalFields, [
                 'forceSelection' => $this->forceSelection ? 1 : 0,
+                'sortAlphabetically' => $this->sortAlphabetically ? 1 : 0,
                 'groupName' => $this->groupName,
                 'groupDescription' => $this->groupDescription,
                 'showOrder' => $this->showOrder,
@@ -216,6 +222,7 @@ public function save()
 
         // reset values
         $this->forceSelection = false;
+        $this->sortAlphabetically = false;
         $this->groupName = $this->groupDescription = '';
         $this->objectTypes = [];
         $this->showOrder = 0;
@@ -246,6 +253,7 @@ public function assignVariables()
         WCF::getTPL()->assign([
             'action' => 'add',
             'forceSelection' => $this->forceSelection,
+            'sortAlphabetically' => $this->sortAlphabetically,
             'groupName' => $this->groupName,
             'groupDescription' => $this->groupDescription,
             'labelObjectTypeContainers' => $this->labelObjectTypeContainers,
diff --git a/wcfsetup/install/files/lib/acp/form/LabelGroupEditForm.class.php b/wcfsetup/install/files/lib/acp/form/LabelGroupEditForm.class.php
index 3347ff02a9a..0390b286a48 100644
--- a/wcfsetup/install/files/lib/acp/form/LabelGroupEditForm.class.php
+++ b/wcfsetup/install/files/lib/acp/form/LabelGroupEditForm.class.php
@@ -83,6 +83,7 @@ public function save()
             [
                 'data' => \array_merge($this->additionalFields, [
                     'forceSelection' => $this->forceSelection ? 1 : 0,
+                    'sortAlphabetically' => $this->sortAlphabetically ? 1 : 0,
                     'groupName' => $this->groupName,
                     'groupDescription' => $this->groupDescription,
                     'showOrder' => $this->showOrder,
@@ -119,6 +120,7 @@ public function readData()
             I18nHandler::getInstance()->setOptions('groupName', 1, $this->group->groupName, 'wcf.acp.label.group\d+');
 
             $this->forceSelection = ($this->group->forceSelection ? true : false);
+            $this->sortAlphabetically = ($this->group->sortAlphabetically ? true : false);
             $this->groupName = $this->group->groupName;
             $this->groupDescription = $this->group->groupDescription;
             $this->showOrder = $this->group->showOrder;
diff --git a/wcfsetup/install/files/lib/data/label/group/LabelGroup.class.php b/wcfsetup/install/files/lib/data/label/group/LabelGroup.class.php
index dd4052d446a..19ce9be856b 100644
--- a/wcfsetup/install/files/lib/data/label/group/LabelGroup.class.php
+++ b/wcfsetup/install/files/lib/data/label/group/LabelGroup.class.php
@@ -17,6 +17,7 @@
  * @property-read   string $groupName      name of the label group or name of language item which contains the label text
  * @property-read   string $groupDescription   description of the label group (only shown in ACP)
  * @property-read   1|0 $forceSelection     is `1` if a label in the label group has to be selected when creating an object for which the label group is available, otherwise `0`
+ * @property-read   1|0 $sortAlphabetically is `1` if labels in the label group are sorted alphabetically by their translated name, otherwise `0`
  * @property-read   int $showOrder      position of the label group in relation to the other label groups
  */
 class LabelGroup extends DatabaseObject implements IRouteController
diff --git a/wcfsetup/install/files/lib/data/label/group/ViewableLabelGroup.class.php b/wcfsetup/install/files/lib/data/label/group/ViewableLabelGroup.class.php
index 8d2bbfbd749..44b51441157 100644
--- a/wcfsetup/install/files/lib/data/label/group/ViewableLabelGroup.class.php
+++ b/wcfsetup/install/files/lib/data/label/group/ViewableLabelGroup.class.php
@@ -263,6 +263,21 @@ public function search($objectID)
         }
     }
 
+    /**
+     * Sorts the labels alphabetically by their translated name using locale-aware comparison.
+     *
+     * @since 6.3
+     */
+    public function sortLabelsAlphabetically(): void
+    {
+        $collator = new \Collator(WCF::getLanguage()->getLocale());
+        \uasort(
+            $this->labels,
+            static fn(Label $a, Label $b) => $collator->compare($a->getTitle(), $b->getTitle())
+        );
+        $this->indexToObject = \array_values(\array_keys($this->labels));
+    }
+
     /**
      * Returns true if any permissions have been set for this label group.
      *
diff --git a/wcfsetup/install/files/lib/system/cache/builder/LabelCacheBuilder.class.php b/wcfsetup/install/files/lib/system/cache/builder/LabelCacheBuilder.class.php
index 0158c8517d9..9cc960015d4 100644
--- a/wcfsetup/install/files/lib/system/cache/builder/LabelCacheBuilder.class.php
+++ b/wcfsetup/install/files/lib/system/cache/builder/LabelCacheBuilder.class.php
@@ -47,8 +47,9 @@ protected function rebuild(array $parameters)
         $data['options'] = $permissions['options']->getObjects();
 
         // assign permissions for each label group
-        /** @var ViewableLabelGroup $group */
         foreach ($data['groups'] as $groupID => $group) {
+            \assert($group instanceof ViewableLabelGroup);
+
             // group permissions
             if (isset($permissions['group'][$groupID])) {
                 $group->setGroupPermissions($permissions['group'][$groupID]);
diff --git a/wcfsetup/install/files/lib/system/endpoint/controller/core/labels/groups/ChangeLabelShowOrder.class.php b/wcfsetup/install/files/lib/system/endpoint/controller/core/labels/groups/ChangeLabelShowOrder.class.php
index 6e3b3616bfc..97235004cd0 100644
--- a/wcfsetup/install/files/lib/system/endpoint/controller/core/labels/groups/ChangeLabelShowOrder.class.php
+++ b/wcfsetup/install/files/lib/system/endpoint/controller/core/labels/groups/ChangeLabelShowOrder.class.php
@@ -11,6 +11,7 @@
 use wcf\http\Helper;
 use wcf\system\endpoint\IController;
 use wcf\system\endpoint\PostRequest;
+use wcf\system\exception\IllegalLinkException;
 use wcf\system\label\LabelHandler;
 use wcf\system\showOrder\ShowOrderHandler;
 use wcf\system\showOrder\ShowOrderItem;
@@ -32,6 +33,10 @@ public function __invoke(ServerRequestInterface $request, array $variables): Res
         WCF::getSession()->checkPermissions(['admin.content.label.canManageLabel']);
 
         $labelGroup = Helper::fetchObjectFromRequestParameter($variables['id'], LabelGroup::class);
+        if ($labelGroup->sortAlphabetically) {
+            throw new IllegalLinkException();
+        }
+
         $items = \array_map(
             static fn(Label $label) => new ShowOrderItem($label->labelID, $label->getTitle()),
             LabelHandler::getInstance()->getLabelGroup($labelGroup->groupID)->getLabels()
diff --git a/wcfsetup/install/files/lib/system/endpoint/controller/core/labels/groups/GetLabelShowOrder.class.php b/wcfsetup/install/files/lib/system/endpoint/controller/core/labels/groups/GetLabelShowOrder.class.php
index 34ad32e858a..f5a4b7e5d14 100644
--- a/wcfsetup/install/files/lib/system/endpoint/controller/core/labels/groups/GetLabelShowOrder.class.php
+++ b/wcfsetup/install/files/lib/system/endpoint/controller/core/labels/groups/GetLabelShowOrder.class.php
@@ -9,6 +9,7 @@
 use wcf\http\Helper;
 use wcf\system\endpoint\GetRequest;
 use wcf\system\endpoint\IController;
+use wcf\system\exception\IllegalLinkException;
 use wcf\system\label\LabelHandler;
 use wcf\system\showOrder\ShowOrderHandler;
 use wcf\system\showOrder\ShowOrderItem;
@@ -30,6 +31,10 @@ public function __invoke(ServerRequestInterface $request, array $variables): Res
         WCF::getSession()->checkPermissions(['admin.content.label.canManageLabel']);
 
         $labelGroup = Helper::fetchObjectFromRequestParameter($variables['id'], LabelGroup::class);
+        if ($labelGroup->sortAlphabetically) {
+            throw new IllegalLinkException();
+        }
+
         $items = \array_map(
             static fn(Label $label) => new ShowOrderItem($label->labelID, $label->getTitle()),
             LabelHandler::getInstance()->getLabelGroup($labelGroup->groupID)->getLabels()
diff --git a/wcfsetup/install/files/lib/system/label/LabelHandler.class.php b/wcfsetup/install/files/lib/system/label/LabelHandler.class.php
index 29d451cd131..68dc33c5881 100644
--- a/wcfsetup/install/files/lib/system/label/LabelHandler.class.php
+++ b/wcfsetup/install/files/lib/system/label/LabelHandler.class.php
@@ -381,7 +381,7 @@ public function getLabelGroups(array $groupIDs = [], $validatePermissions = true
                 }
             }
 
-            $data[$groupID] = $this->labelGroups['groups'][$groupID];
+            $data[$groupID] = $this->getLabelGroup($groupID);
         }
 
         // @phpstan-ignore argument.type
@@ -413,9 +413,19 @@ public function getAccessibleLabelIDs()
      * @param int $groupID
      * @return  ViewableLabelGroup|null
      */
-    public function getLabelGroup($groupID)
+    public function getLabelGroup($groupID): ?ViewableLabelGroup
     {
-        return $this->labelGroups['groups'][$groupID] ?? null;
+        if (!isset($this->labelGroups['groups'][$groupID])) {
+            return null;
+        }
+
+        $labelGroup = $this->labelGroups['groups'][$groupID];
+
+        if ($labelGroup->sortAlphabetically) {
+            $labelGroup->sortLabelsAlphabetically();
+        }
+
+        return $labelGroup;
     }
 
     /**
diff --git a/wcfsetup/install/lang/de.xml b/wcfsetup/install/lang/de.xml
index b2c35d60955..c2474357362 100644
--- a/wcfsetup/install/lang/de.xml
+++ b/wcfsetup/install/lang/de.xml
@@ -1008,6 +1008,7 @@ Sie erreichen das Fehlerprotokoll unter: {link controller='ExceptionLogView' isE
 		<item name="wcf.acp.label.group.edit"><![CDATA[Labelgruppe bearbeiten]]></item>
 		<item name="wcf.acp.label.group.error.invalid"><![CDATA[Die gewählte Labelgruppe ist ungültig.]]></item>
 		<item name="wcf.acp.label.group.forceSelection"><![CDATA[Label aus dieser Gruppe muss zwingend ausgewählt werden]]></item>
+		<item name="wcf.acp.label.group.sortAlphabetically"><![CDATA[Labels alphabetisch sortieren]]></item>
 		<item name="wcf.acp.label.group.list"><![CDATA[Labelgruppen]]></item>
 		<item name="wcf.acp.label.group.permanentSelection"><![CDATA[Die ausgewählte Labelgruppe kann nachträglich nicht mehr verändert werden.]]></item>
 		<item name="wcf.acp.label.label"><![CDATA[Label]]></item>
diff --git a/wcfsetup/install/lang/en.xml b/wcfsetup/install/lang/en.xml
index 405fc14e893..013fe48d0cf 100644
--- a/wcfsetup/install/lang/en.xml
+++ b/wcfsetup/install/lang/en.xml
@@ -984,6 +984,7 @@ You can access the error log at: {link controller='ExceptionLogView' isEmail=tru
 		<item name="wcf.acp.label.group.edit"><![CDATA[Edit Label Group]]></item>
 		<item name="wcf.acp.label.group.error.invalid"><![CDATA[The selected label group is invalid.]]></item>
 		<item name="wcf.acp.label.group.forceSelection"><![CDATA[Force selection of a label]]></item>
+		<item name="wcf.acp.label.group.sortAlphabetically"><![CDATA[Sort labels alphabetically]]></item>
 		<item name="wcf.acp.label.group.list"><![CDATA[Label Groups]]></item>
 		<item name="wcf.acp.label.group.permanentSelection"><![CDATA[The selected label group cannot be modified and is permanent for this label.]]></item>
 		<item name="wcf.acp.label.label"><![CDATA[Label]]></item>
diff --git a/wcfsetup/setup/db/install.sql b/wcfsetup/setup/db/install.sql
index 7211903d1bf..c749a47497b 100644
--- a/wcfsetup/setup/db/install.sql
+++ b/wcfsetup/setup/db/install.sql
@@ -672,6 +672,7 @@ CREATE TABLE wcf1_label_group (
 	groupName VARCHAR(80) NOT NULL,
 	groupDescription VARCHAR(255) NOT NULL DEFAULT '',
 	forceSelection TINYINT(1) NOT NULL DEFAULT 0,
+	sortAlphabetically TINYINT(1) NOT NULL DEFAULT 0,
 	showOrder INT(10) NOT NULL DEFAULT 0
 );