Frontend: Mehrere XML-Formdeklarationen laden und in View bereitstellen

Aligator13 · 26. März 2016

Hallo zusammen,

ich würde gerne Duplikate in den Formdeklarationen einsparen und daher würde ich gerne mehrere XML-Formdeklarationen in einer View einbinden.

Bisher nutze ich folgendes:

Code

public function getForm($data = array(), $loadData = true)
    {
        $form            =         $this->loadForm('com_bestia.comment', 'comment', array('control' => 'jform', 'load_data' => $loadData));

        if (empty($form))
        {
            return false;
        }

        return $form;
    }

Alles anzeigen

Hier gibt es in /models/forms entsprechend die comment.xml.

Wie würde ich nun vorgehen, wenn ich nun auch Elemente aus einer user.xml in der View einbinden wollen würde?

Freue mich über eure Antworten und wünsche frohe Ostern!

Re:Later · 26. März 2016

Ich mach so was über Plugins (public function onContentPrepareForm($form, $data)) sollte aber wohl auch anderswo so gehen.
3 Varianten aus einem Demo-Plugin von mir

Spoiler anzeigen

Aligator13 · 26. März 2016

Vielen Dank. Ich habe es nun etwas weniger dynamisch gelöst bekommen:

Code

/** * getCommentForm function. *  * @access public * @param array $data (default: array()) * @param bool $loadData (default: true) * @return void */public function getCommentForm($data = array(), $loadData = true){    $form            =         $this->loadForm('com_bestia.Comment', 'comment', array('control' => 'jform', 'load_data' => $loadData));    if (empty($form))    {        return false;    }    return $form;}/** * getUserForm function. *  * @access public * @param array $data (default: array()) * @param bool $loadData (default: true) * @return void */public function getUserForm($data = array(), $loadData = true){    $form            =         $this->loadForm('com_bestia.user', 'user', array('control' => 'jform', 'load_data' => $loadData));    if (empty($form))    {        return false;    }    return $form;}

Dies habe ich dann in der view.html.php so eingebunden:

Code

$this->commentform    =    $this->get('CommentForm'); 
        $this->userform        =    $this->get('UserForm');

Re:Later · 26. März 2016

Na ja die erste Variante sind 2 Zeilen

Außerdem dachte ich, du willst in eine bestehende $form weitere Felder nachladen aus anderer XML o.ä.

Aligator13 · 26. März 2016

Deine Lösung werde ich auf jeden Fall auch antesten, vielen Dank noch mal

kann es sein, dass die showon-Methode (in der XML-Formdeklaration) im Frontend nicht funktioniert?

Re:Later · 27. März 2016

Sollte funktionieren.

Im Normalfall extenden ja alle Formularfelder letztlich
libraries/joomla/form/field.php (JFormField)

Dort wird $options['showonEnabled'] = true; neben anderem gesetzt. Wichtig noch das dort gesetzte vorbereitete $options['rel'] (data-showon=usw.)

Dann im Normalfall von dort weitergereicht an JLayout /layouts/joomla/form/renderfield.php (Wird Frontend und Backend verwendet) oder Template/Komponenten-Override dieses JLayouts.

Das muss dann JQuery-JavaScript cms.js laden und rel setzen wie im o.g. renderfield.php zu sehen.

Re:Later · 27. März 2016

Aber showon scheint generell gewisse Einschränkungen zu haben, die wohl noch nicht gefixt:
https://github.com/joomla/joomla-cms/issues/6744

Aligator13 · 27. März 2016

Hm, okay...

Ich stelle leider fest, dass bei folgender Konfiguration

Code

<field    name        =        "title"
        type        =        "list"
        label        =        "COM_BESTIA_FIELDS_USERTITLE_LABEL"
        description    =        "COM_BESTIA_FIELDS_USERTITLE_DESC"
        filter        =        "safehtml"
        default        =        "">
            <option value    =    "">COM_BESTIA_FIELDS_VALUES_NOTITLE</option>
            <option value    =    "other">COM_BESTIA_FIELDS_VALUES_OTHERTITLE</option>
            <option value    =    "dr">COM_BESTIA_FIELDS_VALUES_DR</option>
</field> 
<field    name        =        "testtest" 
        type        =        "text"     
        class        =        "inputbox"
        size        =        "40" 
        label        =        "Test"
        description    =        "Test" 
        showon        =        "title:dr"
        filter        =        "safehtml"/>

Alles anzeigen

Das Feld "testtest" wird allerdings immer eingeblendet. Das sollte eigentlich nichts mit dem Thema auf GitHub zu tun haben, oder? Beide Felder stehen im gleichen Fieldset.

Re:Later · 27. März 2016

Eingebaut wird das in
JFormField::renderfield(...)

Ein z.B.
echo $this->form->getInput('title');
funktioniert deshalb nicht.

Es muss heißern:
echo $this->form->renderfield('title');

Oder musst den Code für showon anderweitig einbauen aus renderfield() sowie o.g. JLayout "abgucken".

Aligator13 · 27. März 2016

Hm, ich weiß nun nicht, ob ich Dich richtig verstehe:

Aktuell nutze ich folgendes

PHP

<?php foreach($this->userform->getFieldset('contact') as $field) : ?>
<div class="col-sm-3"><label for="<?php echo $field->name; ?>"><?php echo $field->label; ?></label></div>
<div class="col-sm-9"><?php echo $field->input; ?></div>
<?php endforeach; ?>

Wie müsste ich das nun anpassen?

Re:Later · 27. März 2016

Vorab: Ursprünglich waren die JLayouts ja gedacht, es auch Normalusern einfach möglich zu machen, Overrides für häufig verwendete Elemente zu erstellen, in diesem Fall für Ausgabe Formularfelder. Daraus geworden ist teils ein vollkommen verquastes, intransparentes vonDateiZuDateiGehangel, bevor man rausbekommt, welches JLayout jetzt eigentlich gezogen wird oder um am Ende die verwendbaren Optionen/Variablen herauszubekommen. Es gibt nämlich oft weitaus mehr als dann im JLayout via $displayData landen. Und dieses extract() ist zusätzlich unnötige, echte Seuche... Egal. Vorgemaule. War mal wieder nötig

Also:
Für /layouts/joomla/form/renderfield.php erzeugst du einen Override.
Folgende Pfade können dafür verwendet werden, also auch Komponente
[0] => templates/mytemplate/html/layouts/com_mycomponent
[1] => components/com_mycomponent/layouts
[2] => templates/mytemplate/html/layouts

Diesen passt an Bootstrap 3 an: col-sm-3 usw.

In deinem Code oben ersetzt Zeilen 2 und 3 durch
$field->renderField();
Denke das klappt, außer das spezifische Formularfeld überschreibt nun wieder renderfield und/oder setzt ein anderes JLayout via $this->renderLayout

Ob das JLayout /layouts/joomla/form/renderlabel.php ebenfalls gezogen wird und relevant ist, weiß ich nicht mehr und hab auch gerade keine Lust auf Gehangel Hilft nur ausprobieren.

Re:Later · 27. März 2016

Ach ja, und vermutlich kannst dann abschließend Zeilen 1-4 auch gleich ersetzen durch Einzeiler
$this->userform->renderFieldset('contact');

Aligator13 · 28. März 2016

Hallo und vielen Dank für Deine Mühen.

Ich verstehe es aber auch richtig, dass folgendes auch ohne Override erst einmal grundsätzlich funktionieren müsste oder?

Zitat von Re:Later

$this->userform->renderFieldset('contact');

Wenn ich das mache werden die einzelnen Felder zwar wieder ausgegeben (ohne foreach-Schleife im Layout-File, also ist das schon einmal nett), aaaaber das Showon funktioniert weiterhin nicht. Das Feld bleibt nach wie vor standardmäßig eingeblendet und verschwindet auch nicht, egal was im Dropdownfeld (welches ja die Bedingung darstellen sollte) ausgewählt ist.

Re:Later · 28. März 2016

Ich habe es mit einem Kontaktformular einer Seite, die ebenfalls Bootstrap3 hat, gestern getestet.

In die contact.xml deine beiden Felder oben in fieldset=contact.

In contact_form.php die ganzen Feldausgaben ersetzt dutch

Code

echo $this->form->renderFieldset('contact');

Funktioniert.
Im Quelltext: cms.js geladen.
Im HTML ein JSON-kodiertes data-showon fürs testtest-Input:

Code

<div class="control-group"  data-showon='[{"field":"jform[title]","values":["dr"],"op":""}]'>

Was gibts jetzt noch. JavaScriptfehler: Webkonsole im Browser prüfen.
Mehrere Felder mit selbem name oder selber id.
Generell Verwirrung durch doppelte IDs, names etc., die ja pro Seite eindeutig sein müssen.

Vielleicht erst mal rudimentäres Formular aufbauen, nur mit diesen beiden Feldern und nur mit 1 Formular testen. Dann schrittweise hochbauen.

Aligator13 · 28. März 2016

Zitat von Re:Later

Im Quelltext: cms.js geladen

Ist das eine Bedingung? Wie lade ich diese?

Re:Later · 28. März 2016

Frontend: Mehrere XML-Formdeklarationen laden und in View bereitstellen