FormBuilder: Buttons beachten Dependencies nicht

  • Affected Version
    WoltLab Suite 5.4

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


  • 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

    • Official Post

    Hallo,


    alles klar. An dieser Stelle gilt dann wie üblich: Wenn du einen wenig invasiven PR hast, der deine Anforderungen erfüllt, dann nehme ich den gerne und reviewe den kurzfristig. Ansonsten schaue ich mir das selbst bei Gelegenheit an.

  • 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?

Participate now!

Don’t have an account yet? Register yourself now and be a part of our community!