implemented option to configre counting of overbooked non bookable days

[chriwen]

EDIT datengraben: This closes #702

Dazu gehört auch wielebenwir/Litepicker#3

[codecov]

Codecov Report

Merging #1147 (c8edb4d) into master (d2afd3a) will increase coverage by 2.17%.
Report is 20 commits behind head on master.
The diff coverage is 71.92%.

@@             Coverage Diff              @@
##             master    #1147      +/-   ##
============================================
+ Coverage     35.87%   38.04%   +2.17%     
- Complexity     2115     2188      +73     
============================================
  Files            83       84       +1     
  Lines          8592     8879     +287     
============================================
+ Hits           3082     3378     +296     
+ Misses         5510     5501       -9     

Files	Coverage Δ
src/View/Calendar.php	50.40% <86.36%> (+1.67%)	⬆️
src/Plugin.php	33.33% <0.00%> (+4.79%)	⬆️
src/Wordpress/CustomPostType/Location.php	11.23% <75.86%> (+7.93%)	⬆️

... and 20 files with indirect coverage changes

[nelarsen]

Juhuu, danke euch (hier wohl v.a. @hansmorb) für die fortwährende Weiterentwicklung. Wird mit diesem PR neben #702 auch #1113 behoben? Diese Issues gehören für mich den größten noch vorhandenen Problemen mit cb2. Es ist mir in den beiden Issues ein bisschen unklar, wie das gewünschte Verhalten definiert ist. Das gewünschte Verhalten ist für mich wie "cb0 mit dem special-days-Plugin", aber auch das ist keine gute Definition vom gewünschten Verhalten.

[hansmorb]

Also zumindest sollte mit diesem Branch sich das Holiday Field jetzt wie die Schließtage verhalten. Und ihr könnt setzen, wie viele Tage von einem Block MAXIMAL gezählt werden. Also zum Beispiel gibt es hier am Samstag und Sonntag einen Schließtag und der Montag ist Feiertag. Mit diesen Optionen kann jezt nur von Freitag bis Dienstag gebucht werden:

Wenn nicht gezählt wird, dann fügt sich kein Tag zum Maximum hinzu

Wenn die Option nicht angewählt ist, ist es nicht erlaubt über geblockte Tage zu buchen -> WE nicht überbuchbar

[hansmorb]

@chriwen Aus meiner Sicht ist das hier ready, kann dich leider nicht als Reviewer hinzufügen weil das dein PR ist aber vielleicht kannst du mal rüberschauen.

Der PR für das Feature im Litepicker Repo ist hier: wielebenwir/Litepicker#3

[datengraben]

@hansmorb wenn ich mir den diff hier über Github anschaue sehe ich einige Änderungen (hauptsächlich E2E) und die glaube zu den hier herein gemergeden branches gehören. Kann es sein, das es noch andere Branches gibt, die in der Reihenfolge vorher in den master sollten?

[hansmorb]

@hansmorb wenn ich mir den diff hier über Github anschaue sehe ich einige Änderungen (hauptsächlich E2E) und die glaube zu den hier herein gemergeden branches gehören. Kann es sein, das es noch andere Branches gibt, die in der Reihenfolge vorher in den master sollten?

Ne, die Änderungen in der content-example gehören auch zu diesem Branch. Ich habe einen ganzen Haufen Artikel / Standorte und Zeitrahmen hinzugefügt um die Überbuchung mit verschiedenen Timeframe Konfigurationen in den E2E Tests testen zu können.

[datengraben]

Also nur mal um das zusammenzufassen.

Was ist hier umgesetzt:

• Timeframe Einzel-Konfiguration der "Überbuchung" von "Schließtagen" aus dem Ursprungs Issue
• korrektes Verhalten des Kalenders in Litepicker für "Überbuchung" von "Schließtagen" aus dem Ursprungs Issue
• Möglichkeit einer "globalen" Konfiguration (zusätzlich zur Einzel-Konfiguration via Timeframe)

Zusätzlich noch e2e Tests.

@hansmorb was vergessen?

[hansmorb]

@datengraben Genau! Und halt die entsprechenden Änderungen im Litepicker die aus dem Master branch kommen. Da ist auch noch die Korrektur der Übersetzung von days/day zu Tag / Tage mit drin.

[datengraben]

Beim Testen ist mir noch folgendes aufgefallen:

• Wo finde ich die "globalen" Konfigurationen
• Die hier gezeigten Optionen sehe ich nicht. ich sehe nur die "Allow lock days" checkbox, die anderen beiden Felder sehe ich nicht.

[hansmorb]

Beim Testen ist mir noch folgendes aufgefallen:

* Wo finde ich die "globalen" Konfigurationen

Im Reiter "Allgemein" unter den Einstellungen

* Die [hier](https://github.com/wielebenwir/commonsbooking/pull/1147#issuecomment-1656405194) gezeigten Optionen sehe ich nicht. ich sehe nur die "Allow lock days" checkbox, die anderen beiden Felder sehe ich nicht.

• Ui, das ist ungut. Ich kann das leider nicht reproduzieren und wenn die "Allow locked days" Checkbox generiert wird, dann müsst eigentlich auch die getOverbookingSettingsMetaboxes ausgeführt werden und damit die anderen Metaboxen auch mitkommen. Vielleicht ne dumme Annahme aber warst du auf dem richtigen Branch?

[datengraben]

@hansmorb ok hat alles funktioniert wie geplant.