search

menatwork / MultiColumnWizard

Contao Extension :: Define unlimited dca fields dynamically
http://contao.org/en/extension-list/view/MultiColumnWizard.html
28 stars 30 forks source link

"Default" funktioniert nicht #260

Open ReneLuecking opened 6 years ago

ReneLuecking commented 6 years ago

Auch wenn man einem Feld einen Default-Wert zuweist, wird dieser nicht ausgegeben. Beispiel:

'columnFields' => array(
    'textField' => array(
        'inputType' => 'text',
        'default' => 'abcd',
    ),
),

Grundsätzlich wird der Default-Wert zwar in der MultiColumnWizard.php genutzt, aber nur, wenn $varValue an der Stelle ein leerer String ist. Tatsächlich ist $varValue hier aber null. Somit wäre meiner Meinung nach eine empty()-Prüfung besser geeignet:

$arrField['value'] = empty($varValue) ? $arrField['default'] : $varValue;
zonky2 commented 6 years ago

klingt gut - kannst Du einen PR machen...

stefanheimes commented 6 years ago

Brauchen wir noch Contao 3 Support oder reicht Contao 4?

zonky2 commented 6 years ago

m.M.: wenn es nur ein "Einzeiler" ist, gern für C3 - ansonsten Blick nach vorne und auf C4 konzentrieren