Bugfix booking creation

phlobbi / prog3

Repository für die Projektarbeit

1 stars 1 forks source link

Bugfix booking creation #116

Closed FriendlyDiamond closed 1 year ago

FriendlyDiamond commented 1 year ago

Damit sollte der Issue #102 gelöst sein

Hinzufügen der Zeit in die Booking-Klasse und deren Daten
Anpassen des UI, sodass Zeiten eingetragen werden können
Rework des UI mit GridPane
neue Validierung für PickUpDateBeforeDropOffDate
neue Berechnung für calculateDaysBetween

FriendlyDiamond commented 1 year ago

Okay, Buildfehler hab ich gefunden, irgendwas in setPickUpDate.. Vielleicht liegt da irgendwo das Problem, dass da geprüft wird, ob das pickUpDate immernoch vorm dropOffDate ist.

Edit: Aktuell müssen wir ein DropOffDate haben, um ein neues PickUpDate zu setzen. Ich könnte einfach in den Test ein setDropOffDate vor das setPickUpDate setzen, damit wärs vermutlich behoben, aber ich weiß nicht ob das dann irgendeinen Stress mit den Persists / der DB gibt und ob wir das vielleicht nicht anders lösen sollten

phlobbi commented 1 year ago

Edit: Aktuell müssen wir ein DropOffDate haben, um ein neues PickUpDate zu setzen. Ich könnte einfach in den Test ein setDropOffDate vor das setPickUpDate setzen, damit wärs vermutlich behoben, aber ich weiß nicht ob das dann irgendeinen Stress mit den Persists / der DB gibt und ob wir das vielleicht nicht anders lösen sollten

Der Test würde nicht mehr funktionieren, da man pickUpDate und dropOffDate ja nur über den Konstruktor "gleichzeitig" setzen kann. Ich hab ihn einfach entfernt.

FriendlyDiamond commented 1 year ago

Der Test würde nicht mehr funktionieren, da man pickUpDate und dropOffDate ja nur über den Konstruktor "gleichzeitig" setzen kann. Ich hab ihn einfach entfernt.

Okay 👍🏼