Totalausfall: Stornierung optional

[hansmorb]

Behebt issue #866

TODO:

• Erwartetes Verhalten klären was Buchungserinnerungen für User & Standorte angeht (sollen bei Totalausfall aber noch existierender Buchung Nutzende und Station immer noch an die Buchung erinnert werden?)
  • Erwartetes Verhalten ist: Erinnerungen gehen nicht raus, weder an Station, noch an Buchende, in Commit 0bb5234 umgesetzt.
• Einstellung oberhalb von Buchungsregeln schieben

closes #866

[codecov]

Codecov Report

Attention: Patch coverage is 80.95238% with 4 lines in your changes missing coverage. Please review.

Project coverage is 50.41%. Comparing base (852c969) to head (e4a8275).
Report is 86 commits behind head on master.

Files with missing lines	Patch %	Lines
src/Model/Restriction.php	0.00%	2 Missing ⚠️
src/Service/Booking.php	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff            @@
##             master    #1347   +/-   ##
=========================================
  Coverage     50.41%   50.41%           
- Complexity     2719     2733   +14     
=========================================
  Files            99       99           
  Lines         11296    11343   +47     
=========================================
+ Hits           5695     5719   +24     
- Misses         5601     5624   +23     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[datengraben]

Den Code habe ich reviewed.

Funktional sollte das passen.

Die PHPdocs der Repo/Models für Restriction und Booking sollten wir auch nochmal anschauen. War gerade nicht ganz einfach das zu verstehen.

Ein anderes Thema wäre noch die Auswertung ("Statistik") der Buchungs-Zeitrahmen über den Zeitrahmen-Export. Da sie jetzt nicht mehr cancel-ed werden, sind Sie weiterhin ungekennzeichnet vorhanden. Ich weiß noch nicht ob es eine andere Lösung benötigt, aber aktuell würde ich für die Zeitrahmen-Auswertung so machen, das ich mir Buchungs-Zeitrahmen und Restrictions-Zeitrahmen exportiere und so die Breakdown-Buchungen herausrechne.

Nach so Änderungen wie der hier, tendiere ich noch mehr für ein Hosting der Dokumentation/Handbuch Quellen auch hier im Repo (und nicht in der WP-Instanz). Dann könnte man das direkt ändern und im Nachinein besser über die Commits tracken.

Das hat aber jetzt weniger mit dem Review zu tun.