FormBuilder: Buttons beachten Dependencies nicht

Buttons werden trotz nicht erfüllter Dependencies angezeigt. FormButton used TFormNode, was wiederum die API für Attribute, CSS-Klassen und Dependencies bereitstellt.

        $orderAsGuestField = BooleanFormField::create('orderAsGuest')
            ->label('shop.cart.account.orderAsGuest')
            ->available(SOME_CONDITION);

        $loginButton = FormButton::create('loginButton')
            ->label('wcf.user.login.login')
            ->submit(!$this->form->isAjax())
            ->addClass('buttonPrimary')
            ->available(!WCF::getUser()->userID);
        if ($orderAsGuestField->isAvailable()) {
            $loginButton->addDependency(
                EmptyFormFieldDependency::create('orderAsGuest')
                    ->fieldId('orderAsGuest')
            );
        }
        $this->form->addButton($loginButton);

Nein.

Das Problem hier ist scheinbar aber, dass die Dependencies im Template gar nicht erst geladen werden:

WCF/__form.tpl at fc7ce007ed395cd1b35925507ac8ad69a6e53c52 · WoltLab/WCF

WoltLab Suite Core (previously WoltLab Community Framework) - WCF/__form.tpl at fc7ce007ed395cd1b35925507ac8ad69a6e53c52 · WoltLab/WCF

github.com

WCF/FormButton.class.php at fc7ce007ed395cd1b35925507ac8ad69a6e53c52 · WoltLab/WCF

WoltLab Suite Core (previously WoltLab Community Framework) - WCF/FormButton.class.php at fc7ce007ed395cd1b35925507ac8ad69a6e53c52 · WoltLab/WCF

github.com

WCF/__formButton.tpl at fc7ce007ed395cd1b35925507ac8ad69a6e53c52 · WoltLab/WCF

WoltLab Suite Core (previously WoltLab Community Framework) - WCF/__formButton.tpl at fc7ce007ed395cd1b35925507ac8ad69a6e53c52 · WoltLab/WCF

github.com

vs. bei Feldern

WCF/__formField.tpl at fc7ce007ed395cd1b35925507ac8ad69a6e53c52 · WoltLab/WCF

WoltLab Suite Core (previously WoltLab Community Framework) - WCF/__formField.tpl at fc7ce007ed395cd1b35925507ac8ad69a6e53c52 · WoltLab/WCF

github.com

Ich versuche mich mal daran. Kleine Frage vorab, die aber erst mal nicht wichtig für die Umsetzung ist:
Seht ihr es lieber, wenn ich für die Dependencies ein eigenes Template __formButtonDependencies anlege oder soll ich per {include file='__formFieldDependencies' field=$button} "pfuschen"?

An sich wäre die korrekteste Lösung __formNodeDependencies und jeweils als $node das jeweilige Objekt zu übergeben. Das halte ich aber für vor 6.0 für zu invasiv.

TFormElement::description ist aktuell auch verfügbar, wird im Template aber nicht umgesetzt. Das würde ich direkt auch angehen.

FormButton::requiresLabel() müsste eigentlich auch stets auf true gesetzt sein, oder?

Zur Referenz: https://github.com/WoltLab/WCF/pull/4676

Ich habe die Änderungen mit obigem Fall erfolgreich getestet, würde mich aber über eine Review bezüglich eventueller Edge-Cases freuen.