Vote-Button in Vorschläge-Übersicht liefert Fehlermeldung / Voting nicht abgeschlossen

[nicolefreypuzzle]

Codi
Im Codi Punkt 2

Describe the bug

• Voting Button auf der Liste der Vorschläge funktioniert meistens nicht, liefert Fehlermeldung in rotem Balken oben und leitet auf Startseite weiter (Fehlermeldung: Sie sind nicht berechtigt, diese Aktion auszuführen)

*Problem

• Button führt an einen seltsamen Ort (ev. Decidim Bug oder Backports Bug?)
• die beiden Buttons (Listenübersicht der Vorschläge & Detailansicht des Vorschlages) lösen unterschiedliche Actions aus (Div. Unterschiede in den Kommandos ersichtlich, z.B. fehlt beim neuen Button der X-CSRF-Token).

Actions done

• Vorübergehende Lösung: Mit CSS wurde der Button auf der Liste und auf Grid ausgeblendet, da man auf den einzelnen Vorschlägen noch abstimmen kann
  UPDATE: da die nicht eingeloggten User so keinen CTA haben (der Button ist bei nicht eingeloggten Usern auch in der Detailansicht des Vorschlages ausgeblendet), wurde er auf der Listenübersicht der Vorschläge wieder eingeblendet.
• Carlo hat für Puzzle ersichtliche Informationen hinter den Buttons zusammengestellt (Di, 14.4.)
• Lars hat Informationen zum Prüfen und Fixen an pokecode (Ivan) geschickt (Di, 14.4.)

Actions planned / in progress
Analyse, Fix durch pokecode (Orga durch Lars)

[carlobeltrame]

Unbestätigte Vermutung: Es könnte auch mit Caching zusammenhängen. Wenn die Buttons pro Prozess aber nicht pro User gecached werden, und abwechslungsweise eingeloggte und anonyme User auf der Seite herumklicken, könnte dem eingeloggten User ein gecachter Button vom anonymen User angezeigt werden. Da bei der Sortierung standardmässig die zufällige Reihenfolge ausgewählt ist, könnte man damit erklären, wieso nur ein zufälliger Teil der Buttons falsch ist. Und damit wäre auch erklärt, warum das auf der INT oder in der Nacht nicht so auftritt, wo weniger gleichzeitige User auf der Seite herumklicken.

[larsUE]

Ja, das würde auch das Verhalten erklären, dass wir derzeit auf der Plattform beobachten: Gewisse Punkte sind (auch für nicht eingeloggte User) ausgegraut ("Keine verbleibenden Stimmen"). Ich gebe diese Vermutung auch Ivan mit, er schaut sich alles heute Nachmittag an.

[larsUE]

@carlobeltrame Ivan meinte bei einem vorangehenden Gespräch noch folgendes:

1. On one side, I think the assets are not properly pre-compiled, if you are not logged in, when clicking the "vote" button, a modal window to log you in should appear. I've download the Zurich application and, properly compiled, this works correctly. It does not work on your stagin app. However this cannot be cause of this particular issue.

-> Das sieht man dadurch, dass im uneingeloggten Zustand auf allen Ideen, ausser dem ersten das Pop-Up zur Anmeldung kommt.

Ich glaube zwar auch nicht, dass das die Ursache des Bugs ist, aber irgendeinen Zusammenhang zwischen Anmeldung und Zustand des Buttons muss es geben: Gemäss https://github.com/decidim/decidim/blob/release/0.29-stable/decidim-core/app/helpers/decidim/action_authorization_helper.rb wird ja geprüft, ob der User angemeldet ist oder nicht (Wenn ja -> weiter mit Authorization & request, wenn nein dann Modal öffnen und target="#" setzen. Kann es sein, dass irgendein Anmeldestatus auf den CardCells gecached wird und deshalb die Links so gerendered werden? Irgendwie würde das ja mehrere Phänomene miteinander verbinden.

Kannst du mal die asset pre-compliation anschauen, bis Ivan antwortet?

[carlobeltrame]

@larsUE Die pre-compilation ist korrekt, denn wenn man nicht eingeloggt ist kommt das von ihm beschriebene Modal. Auf INT kann er das nicht testen, weil man dort die Seite gar nicht anschauen kann, ohne eingeloggt zu sein. Aber auf PROD führen die Buttons in <form action="#"> korrekt zum Modal, wenn man nicht eingeloggt ist.

Gerade habe ich noch die PROD abgerufen ohne eingeloggt zu sein, und einige der Buttons sind andersrum verkehrt in einem <form action=".../proposal_vote?from_proposals_list=true">. Diese öffnen dann natürlich kein Modal, sondern versuchen direkt abzustimmen, was vom Server korrekterweise mit 401 Unauthorized abgelehnt wird.

Nach wie vor ist das einzige was inkorrekt ist, dass die Buttons nicht alle den Login-Status des aktuellen Users berücksichtigen.

[larsUE]

OK, danke! Getestet haben wir dazumal auf DEV, dort lassen wir die Seite bewusst offen, um solche Dinge anschauen zu können.

Nach wie vor ist das einzige was inkorrekt ist, dass die Buttons nicht alle den Login-Status des aktuellen Users berücksichtigen.

Und zusätzlich noch, dass sie manchmal den falschen Vote-Zustand anzeigen. Screenshot als nicht eingeloggter User:

[carlobeltrame]

Ja, das deckt sich weiter mit der Vermutung, dass manchmal einem User die gecachete Version des Buttons von einem anderen User angezeigt wird.

[carlobeltrame]

Vielleicht weiss ich nicht, wie korrekt suchen, aber ich finde im Cache keine gespeicherten Buttons. Die Autoren auf den Proposals sind gecached, aber die Buttons finde ich nicht.

[carlobeltrame]

@larsUE übrigens hast du einen Link zum Decidim-Code gepostet, aber wir haben ja gar nicht Original-Decidim installiert, sondern die Pokecode Backports 0.29. Deine verlinkte Code-Datei ist zwar dort noch drin, aber einfach damit wir es nicht verwechseln.
Und die Message des letzten Commits auf dem Backports Branch sagt mir, dass Pokecode mit Votes und Cache etwas gemacht haben, auch wenn ich diese Einträge im memcached nicht gefunden habe: openpoke/decidim@a3e489f

[larsUE]

Das ist korrekt. Das war ein erster Fix um den Backport zum laufen zu bringen, weil zuvor die Buttons sich gar nicht geändert hatten, z.B. wenn ich meine maximale Stimmenanzahl ausgeschöpft hatte.

[microstudi]

Hi guys, checking this issue and the backport commit I think I found a case that might be causing this:
The current cache key does not distinct between and unlogged user and a logged user that has not voted. And, in the view, the rendered button is different (that's why some of them present the modal window while others don't).

I've just added the current user id into the cache key, I think this should solve the issue if you can re-deploy updating the 0.29-backports branch (openpoke/decidim@2289c1b)

[carlobeltrame]

Hey @microstudi yes this looks like it. But please could you change it not only in the proposal_g_cell, but also in the proposal_l_cell which is used for the list display? Once you do that I will install it. Thanks!

[microstudi]

completely forgot! sorry, done!