puzzle / decidim-zuerich

Mitwirkungsportal für die Stadt Zürich, basierend auf Decidim
https://mitwirken.stadt-zuerich.ch
GNU Affero General Public License v3.0
8 stars 2 forks source link

Datenverlust bei Umfragen #380

Closed larsUE closed 1 year ago

larsUE commented 1 year ago

Wir haben einen merkwürdigen Bug bei uns auf den Umfragen. Wenn ich eine Frage mit einer Bedingung habe, dann wird die Antwort der Bedingung nicht gespeichert, wenn eine Fehlermeldung in der Umfrage aufgetreten ist.

Ausführliche Beschreibung

Schritt 1: Umfrage mit einer bedingten Frage erstellen Bildschirm­foto 2023-04-20 um 09 30 31

Schritt 2: Umfrage inkl. Bedingung und Fehler in erster Antwort ausfüllen Bildschirm­foto 2023-04-20 um 09 31 41

Schritt 3: Umfrage abschicken und Fehler provozieren Bildschirm­foto 2023-04-20 um 09 32 47

Schritt 4: Fehler beheben und Umfrage erneut abschicken Bildschirm­foto 2023-04-20 um 09 33 36

Schritt 5: Antworten prüfen, feststellen, dass die Antwort der Bedingung nicht gespeichert wurde Bildschirm­foto 2023-04-20 um 09 35 11

Test Cases

Getestet hier: https://mitwirken.integ.stadt-zuerich.ch/admin/participatory_processes/testlars/components/394/manage/

Vermutung

Ich vermute deshalb, dass dieser Error mit den von uns eingeführten Dropdown/Sort Funktion #351 zusammenhängt.

Kagemaru commented 1 year ago

Lokal kann ich dies nicht nachstellen. Da funktioniert es in allen Fällen.

larsUE commented 1 year ago

Ich kann es auch nur auf den ZH INT und PROD nachstellen. Merkwürdig! Wichtig ist, dass du zuerst einen Fehler auslöst, bevor du die Umfrage abschickst. Wir hatten es nun bereits in verschiedenen Umfragen, dass deshalb keine Daten gesammelt wurden für einzelne Fragen.

Kagemaru commented 1 year ago

Nun konnte ich es lokal reproduzieren, ich grab’ mich mal durch den Code. :grin:

Update 1: Hmm, bei mir passiert es lokal auch, ohne den Fehler davor auszulösen.

Update 2: Anscheinend hat sich etwas verändert von 0.26.40.27.2 Nun werden die Antworten nicht mehr als context gesetzt, wodurch die Conditions nicht mehr aufgelöst werden können. Dies ist ein Teil von decidim-forms, ich muss noch schauen, ob dies ein Bug in Base Decidim ist.

larsUE commented 1 year ago

@Kagemaru evtl. reicht ein update auf 27.3: https://github.com/decidim/decidim/pull/10448

Kagemaru commented 1 year ago

Good catch, deployed auf INT.

larsUE commented 1 year ago

Super, funktioniert nun alles :)