CB-Admins von Advance Booking Days ausnehmen

[hansmorb]

Aus #1161 extrahiert

Am besten auf Merge von #1161 warten, da gibt es einige Überlappungen

TODO: Nach Merge von #1161

• Custom Cache anpassen um auch mit privilegierten Rollen zu arbeiten

closes #1418

[codecov]

Codecov Report

Attention: Patch coverage is 92.85714% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 47.57%. Comparing base (f1224fc) to head (dc81272).
Report is 17 commits behind head on master.

Files	Patch %	Lines
src/Repository/UserRepository.php	80.00%	2 Missing ⚠️
src/Service/Cache.php	87.50%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1422      +/-   ##
============================================
+ Coverage     47.31%   47.57%   +0.25%     
- Complexity     2560     2569       +9     
============================================
  Files            93       93              
  Lines         10096    10121      +25     
============================================
+ Hits           4777     4815      +38     
+ Misses         5319     5306      -13     

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

[chriwen]

@hansmorb Ich glaube ich habe die Idee verstanden. Dazu zunächst folgende Fragen: Konzeptionell war es ja mal angedacht, die maxBookingDays basierend auf User-Rollen anpassbar zu machen (evtl. über die Backend-Einstellungen).

Dazu hatte ich ja mal nen ersten Draft gebaut, der jedoch zunächst statt Rollen die Einstellungen für einzelne User definiert (#697)
Fragen dazu:

• Ist diese Funktionalität in deinen Buchungsregeln integriert (also die definition der MaxDays basierend auf User-Rollen)
• Würde mit der Implementierung einer solchen Funktion nicht die Funktionalität in diesem PR obsolet werden, da wir das dann über die jeweiligen Rollen flexibler und individueller definieren könnten?

[hansmorb]

@hansmorb Ich glaube ich habe die Idee verstanden. Dazu zunächst folgende Fragen: Konzeptionell war es ja mal angedacht, die maxBookingDays basierend auf User-Rollen anpassbar zu machen (evtl. über die Backend-Einstellungen).

Hier geht es mir um die advance booking days erstmal. Die maxbookingdays ließe sich damit natürlich auch machen aber dann hast du ja schon fast eine Adminbuchung im Frontend nachgebaut.

Dazu hatte ich ja mal nen ersten Draft gebaut, der jedoch zunächst statt Rollen die Einstellungen für einzelne User definiert (#697)
Fragen dazu:

• Ist diese Funktionalität in deinen Buchungsregeln integriert (also die definition der MaxDays basierend auf User-Rollen)

Nein, das ist nicht definiert. Die maxbookingdays werden nicht angefasst, vielmehr werden über die Buchungsregeln einfach Buchungen nicht zugelassen die über ein in den Regeln definiertes Kontingent gehen (zb. Max 2 Tage pro Woche).

• Würde mit der Implementierung einer solchen Funktion nicht die Funktionalität in diesem PR obsolet werden, da wir das dann über die jeweiligen Rollen flexibler und individueller definieren könnten?

Siehe oben, die Daten die an den Litepicker gegeben werden werden mit keiner Buchungsregel angefasst

[chriwen]

Äh. Ich meinte natürlich die advanceBooking days.

[hansmorb]

Achso ja, die werden von den Buchungsregeln auch nicht angetastet

[chriwen]

ok, ja, dann können wir das hier ja gerne erstmal so implementieren und ggf. später dann anpassbar machen für weitere User-Rollen. Das ist ja dann nur noch ne kleine Änderung.