wielebenwir / commonsbooking

CommonsBooking is an open source Wordpress plugin for sharing items with users. This is the NEW CommonsBooking (starting at version v2.0.0). Please install plugin via Wordpress plugin directory.
https://commonsbooking.org
GNU General Public License v2.0
47 stars 15 forks source link

Totalausfall benachrichtigt nicht über Lösung #1346

Open hansmorb opened 1 year ago

hansmorb commented 1 year ago

Zum reproduzieren: Artikel buchen -> Totalausfall erstellen : Buchender erhält zwei E-Mails, eine zum Totalausfall und eine zur Stornierung JETZT Totalausfall auf gelöst: Buchender erhält keine Benachrichtigung, dass das Problem mittlerweile gelöst ist.

Erwartetes Verhalten: Nutzender wird darüber benachrichtigt, dass das Problem gelöst ist. Sollte am besten so formuliert sein, dass der Nutzende weiss, dass er nochmal neu buchen muss.

chriwen commented 1 year ago

kann es sein, dass nach dem ändern des Status nochmal erneut der "Senden"-Button geklickt werden muss? Hattest du das so probiert?

hansmorb commented 1 year ago

Ja, habe ich probiert. Ich glaube das Problem liegt in Zeile 262 vom Restriction Model, dort werden zwar cancelled Bookings abgerufen, aber mit der Funktion \CommonsBooking\Repository\Booking::getByRestriction . Und die default Werte von der Funktion sucht nur nach Confirmed Buchungen

image

Wäre es erwartetes Verhalten, dass dort auch alle cancelled Buchungen benachrichtigt werden?

chriwen commented 1 year ago

Achso. Das war so by Design gewünscht, dass die nicht benachrichtigt werden. Wir haben gesagt, wenn Buchungen storniert sind, müssen sich die Personen anderweitig kümmern. Die Software braucht dann nicht nochmal informieren. Die Annahme hier hier ist dass diese Info nur für einen sehr geringen Teil interessant wäre. Alle anderen können entsprechend ihrer Pläne sich nach alternativen Rädern umschauen. Nur bei vorübergehenden Einschränkungen kann das evtl. Sinn machen.

hansmorb commented 1 year ago

Jup, deshalb habe ich das auch erstmal als Feature markiert. Wahrscheinlich bräuchte man dafür wenn dann ein separates Template. Aber das ist vielleicht was für die lange Bank. Wollte es nur mal dokumentiert haben weil ich da schon ein paar Mal drüber gestolpert bin. Und ehrlich gesagt ist das im Code etwas missverständlich formuliert, deshalb bin ich kurz davon ausgegangen, dass das erwartetes Verhalten war.