Felder Select, Checkbox, Radiobutton usw. in Eingabemaske "Nur lesen" brauchen disabled statt readonly

[zonky2]

Die Option "Nur lesen" hab bei einem Auswahlfeld (Select), Checkboxen und Radiobutton keine Wirkung - da muss das HTML-Attribut 'disabled' statt 'readonly' hin...

Contao selbst stellt ein Attribut 'disabled' zur Verfügung - für einen PR habe ich leider nicht gefunden, wo man die Unterscheidung zwischen Input-Typ Text und Textarea (für readonly) und dem Rest machen könnte :-(

TNG NB von 28.08.2014

• Checkbox
• Select - Select
• Select - Radio
• Select - Popup
• Tags - Checkboxen
• Tags - Checkboxwizard
• Tags - List
• Tags - Popup

[zonky2]

Hallo,

ich habe nochmal nachgekramt und versucht die Sache zu lösen - m.E. gibt es folgende Lösungen

a) Anzeige als "normales" Textfeld, ggf. mit Icon statt Checkbox/Radiobutton [mein Favorit!]
b) Option "nur Lesen" ist bei "SCR" disabled statt readonly (muss bei DCA-Attributen erweitert werden)
c) im Contao-Core muss das /system/modules/core/library/Contao/Widget.php umgeschrieben werden z.B.

        // Add the remaining attributes
        foreach ($this->arrAttributes as $k=>$v)
        {
            if ($k == 'disabled' || $k == 'readonly' || $k == 'required' || $k == 'autofocus' || $k == 'multiple')
            {
                if (TL_MODE == 'FE') // see #3878
                {
                    $strAttributes .= $blnIsXhtml ? ' ' . $k . '="' . $v . '"' : ' ' . $k;
                }
                elseif ($k == 'disabled' || $k == 'readonly' || $k == 'multiple') // see #4131
                {
                    if(($this->arrConfiguration['type'] == 'select' || $this->arrConfiguration['type'] == 'radio' || $this->arrConfiguration['type'] == 'checkbox') && $k == 'readonly')
                    {
                        $k = 'disabled';
                    }
                    $strAttributes .= ' ' . $k;
                }
            }
            else
            {
                if ($v != '')
                {
                    $strAttributes .= ' ' . $k . '="' . $v . '"';
                }
            }
        }

(überschreiben geht glaube ich nicht da protected)

=> contao/core#7282

[zonky2]

ein weiteres Problem ist aufgetaucht: sofern der Datensatz nochmal gespeichert werden muss, gehen die Werte der Checkboxen und Radiobutton verloren, da diese zus. immer mit einem Hidden-Field ausgeliefert werden. In dem ist aber als value nicht der aktuelle Wert, sondern nix - daher werden die Werte in der DB gelöscht, auch wenn von den eigentlichen Elementen kein Wert übermittelt wird...

[zonky2]

Lösung wäre z.B. die Hidden Field zu entfernen, wenn readonly
siehe contao/core#7282

[discordier]

Das Problem liegt meiner Meinung nach primaer im DCG Widget Manager.
Da muss ich mal nachsehen.

[zonky2]

... die Option einer "nur Text Ausgabe" wäre natürlich auch "schick"... ;-)

[discordier]

Im DCG ist der Check nun erstmal drin.
Siehe contao-community-alliance/dc-general#67
contao-community-alliance/dc-general@6281e73

Die Nur Text Ausgabe ist jedoch meiner Meinung nach ein gesonderter Feature request.
Klappt dein initiales Problem nun?

[zonky2]

o.k. - werde ich testen...

das FR-Ticket steht hier: #568

[zonky2]

hmmm... habe das TNG NB vom 27.08.14 mit dem Update (s.o.) des DCG und ein kleinen Test mit MM_Trainings und MM_Buchungen wobei die Trainings als Auswahl (Select) in Buchungen angezeigt werden. Die Anzeige ist auf "nur Lesen" eingestellt.

Man kann die Trainings in den Buchungen dennoch ändern => siehe die 3 Screenshots...

[discordier]

Aeh, moment mal.
Es ging im Contao ticket doch lediglich darum, dass wir faelschlicherweise abspeichern obwohl das Widget meldet, dass es nicht gespeichert werden soll.

Dein aktuelles Problem scheint jedoch nun auf ein Select bezogen zu sein wo das stylect Script anscheinend das read only attribut nicht korrekt auswertet. (siehe dein screenshot).
Das Select Element ist bei dir naemlich in der Tat read only, jedoch das <div class="styled_select tl_select"> ist das, was du in deinem Ausschnitt mit :( markiert hast (das reale <select> is unsichtbar "dahinter").

Dazu muss ich dich leider wieder nach https://github.com/contao/core/issues schicken, sorry wegen dem tracker ping pong. :(

[zonky2]

hier ist nun wenigstens das hidden-field weg, was die Nicht-Speicherung durchkreuzt hat - aber es wird trotzdem gespeichert :-(

bei Leo werde ich mit dem Anliegen (sicher) kein Erfolg haben - in den DCA-Attributen gibt es das "readonly", was aber im MM/DCG nicht verwendet wird sondern "nur" disabled - zudem wird Leo seine hiddden-fields nicht entfernen...

Hier dreht sich die Katze im Kreise...