MultipleSelectionFormField übergibt falschen bzw. alten Wert, wenn kein Wert ausgewählt wurde

VieCode · June 11, 2022 at 1:16 PM

Ich habe gerade das Problem beim Bearbeiten eines Eintrages, welcher ein MultipleSelectionFormField benutzt. Dieses Eingabefeld hat z.B. die Werte A, B und C. Das Element wird beim Laden mit den bestehenden Werten befüllt, z.B. ist A ausgewählt. Wenn nun A abgewählt wird, retuniert getValue() den falschen Wert (= [ A ]), weil readValue den bestehenden Wert nicht überschreibt, weil $this->getDocument()->hasRequestData($this->getPrefixedId()) false zurück liefert. $_POST['fieldName'] existiert nicht.

**Tim Düsterhus** · June 13, 2022 at 11:30 AM Official Post

Hallo,

ich bin mir noch nicht sicher, ob das die richtige oder beste Korrektur ist, aber korrigiert folgende Diff das Problem für dich?

Diff

diff --git i/wcfsetup/install/files/lib/system/form/builder/field/MultipleSelectionFormField.class.php w/wcfsetup/install/files/lib/system/form/builder/field/MultipleSelectionFormField.cl>
index 9283d5d1e7..1920fad040 100644
--- i/wcfsetup/install/files/lib/system/form/builder/field/MultipleSelectionFormField.class.php
+++ w/wcfsetup/install/files/lib/system/form/builder/field/MultipleSelectionFormField.class.php
@@ -84,6 +84,8 @@ public function readValue()
             if (\is_array($value)) {
                 $this->value = $value;
             }
+        } else {
+            $this->value = [];
         }
 
         return $this;

Display More

VieCode · June 13, 2022 at 6:34 PM

Auf den ersten Blick sollte es das Problem lösen, aber ich kann es erst am Wochenende genau testen. Ich melde mich.

VieCode · June 25, 2022 at 9:23 PM

Ich hab es nicht früher geschafft, aber jetzt konnte ich mit den Fix genauer ansehen und kann bestätigen, dass die Änderung das Problem behebt.

MultipleSelectionFormField übergibt falschen bzw. alten Wert, wenn kein Wert ausgewählt wurde

Tim Düsterhus June 13, 2022 at 10:18 AM

Participate now!

Tim Düsterhus June 13, 2022 at 10:18 AM

Participate now!

Tim Düsterhus June 13, 2022 at 10:18 AM