LabelFormField keine Einschränkung nach Kategorie möglich

Hallo,

Quote from Darkwood.Design

Das LabelFormField sieht keine Möglichkeit vor die angezeigten Labelgruppen bei Kategoriewechsel ein-/auszublenden.

Ein LabelFormField zeigt jeweils die Label einer einzelnen Labelgruppe an. Wenn du mehrere Labelgruppen zur Auswahl stellen willst, dann hast du auch mehrere LabelFormFields.

Die Verfügbarkeit einzelnen Felder kannst du auf Basis der Dependencies (in deinem Fall wohl ValueFormFieldDependency) steuern. Entsprechend müsstest du passende Dependencies zwischen dem Kategorieauswahl-Feld und den Labelgruppen-Feldern definieren. Die Label kennen das Konzept von Kategorien nicht und denen dieses Konzept beizubringen wäre eine Layering-Verletzung.

Hallo,

Quote from Darkwood.Design

Das funktioniert zwar, fühlt sich aber unnötig kompliziert an. Hast du hier eine einfachere Lösung im Kopf oder kannst dir vorstellen, wie es im Framework einfacher abgebildet werden kann, um boilerplate code zu reduzieren?

ungetestet:

$labelFields = [];
$labelGroupsToCategories = ProjectCategoryLabelCacheBuilder::getInstance()->getData();
foreach ($labelGroupsToCategories as $categoryID => $__groupIDs) {
    $categoryFields = LabelFormField::createFields(
        'de.darkwood-studios.ticketsystem.projectmanager.project',
        $__groupIDs,
        'label'
    );
    foreach ($categoryFields as $field) {
        $field->addDependency(
            ValueFormFieldDependency::create('xxx')
                ->values([$categoryID])
                ->field($categoryField)
        );
    }
    $fields = array_merge($fields, $categoryFields);
}

Du müsstest allerdings dann noch schauen, ob es für Probleme sorgt, wenn dieselbe Labelgruppe in mehreren Kategorien verfügbar ist, dann werden wohl mehrere Felder erstellt. Das kannst du dann aber wohl lösen, indem du alle passenden Kategorien als Array in der Dependency angibst.

Hallo,

Quote from Darkwood.Design

So sah mein erster Versuch auch. Dann musste ich aber feststellen, dass LabelFormField ein array von Objekten benötigt und in $__groupIDs nur die IDs drin stehen. So einfach klappt das also auch nicht.

Der Weg von IDs zu tatsächlichen Objekten ist aber nun wirklich kein Hexenwerk mehr. Das größere Problem ist, dass du, wie schon vermutet:

Quote from Tim Düsterhus

Du müsstest allerdings dann noch schauen, ob es für Probleme sorgt, wenn dieselbe Labelgruppe in mehreren Kategorien verfügbar ist, dann werden wohl mehrere Felder erstellt. Das kannst du dann aber wohl lösen, indem du alle passenden Kategorien als Array in der Dependency angibst.

doppelte Felder erstellst.

Ich kann dir folgendes (getestet) auf Basis der Kategorien für Artikel anbieten:

        /** @var ArticleCategory[] $categories */
        $categories = [];
        foreach (ArticleCategory::getAccessibleCategoryIDs() as $categoryID) {
            $categories[$categoryID] = ArticleCategory::getCategory($categoryID);
        }

        $form->appendChildren([
            $categoryField = SingleSelectionFormField::create('categoryID')
                ->options($categories),
        ]);

        $labelGroupToCategory = [];
        foreach ($categories as $category) {
            $labelGroupIDs = \array_column($category->getLabelGroups(), 'groupID');
            foreach ($labelGroupIDs as $labelGroupID) {
                $labelGroupToCategory[$labelGroupID][] = $category->categoryID;
            }
        }

        $labelFields = LabelFormField::createFields(
            'com.woltlab.wcf.article',
            LabelHandler::getInstance()->getLabelGroups(\array_keys($labelGroupToCategory))
        );
        foreach ($labelFields as $labelField) {
            $labelField->addDependency(
                ValueFormFieldDependency::create('labelGroupToCategory')
                    ->values($labelGroupToCategory[$labelField->getLabelGroup()->groupID])
                    ->field($categoryField)
            );
        }
        $form->appendChildren($labelFields);

Quote from Darkwood.Design

Hier sehe ich ganz klar Verbesserungspotential beim LabelFormField, welche fast immer in Kombination mit Kategorien genutzt werden und auch entsprechend damit umgehen sollten. - ohne ständig 20 Zeilen Code von Form zu Form kopieren zu müssen. - vielleicht kannst du das ja mal als Feedback für eine kommende Version mitnehmen.

Wie ich schon erklärt habe, ist es nicht die Aufgabe des Feldes sich um seine eigenen Abhängigkeiten zu kümmern. Die Tatsache, dass du einen Zusammenhang zwischen Kategorie-Auswahl und Labelgruppen hast, ist letztlich eine Eigenschaft deines Formulars. Nicht jede Kategorie kennt überhaupt Label (unser Blog nutzt die Funktion beispielsweise nicht) und nicht immer enthält das Erstellungs-Formular auch die Kategorie-Auswahl (beispielsweise im Forum, wo die Kategorie [1] beim Aufruf des Formulars feststeht).

Insofern ist der Code da oben insofern „optimal“, als, dass er eine saubere separation of concerns umsetzt und die Verknüpfung der Felder an der Stelle vornimmt wo sie logisch korrekt aufgehoben ist. Nämlich in dem Formular. Ja, es entsteht ein bisschen Boilerplate-Code, aber der Control Flow ist sehr simpel und „offensichtlich“ korrekt. Die eigentliche Feld-Logik besteht aus 11 Zeilen, mit einer simplen Schleife und nur 3 Semikolons.

[1] In diesem Fall der Forenbereich.