Fix shift overlap bug: update logic to handle shifts starting and ending at the same time

[Michiel-Vanzeir]

Fix for Shift Overlap Bug: Identical Start and End Times Not Detected

Problem:

The current shift overlap detection logic fails to detect cases where two shifts start and end at the exact same time. This results in an incorrect outcome where such shifts are considered non-overlapping, even though they should be.

Solution:

This fix updates the logic to handle shifts that have identical start and end times. The new comparison ensures that shifts with matching start and end times are properly detected as overlapping.

Please review the changes and consider merging them to resolve the bug.

[Tiddons]

Als fix op deze youtrack issue

(Uit de reactie in youtrack)
Anderzijds, het probleem is dat er soms dezelfde persoon 2 keer op dezelfde shift staat, de code in je request lijkt dit niet op te lossen. Wel nice dat je al de litus hebt kunnen verkennen:)

Een shifter die stopt om 12u30 mag een shift die begint om 12u30 opnemen, dus dat mag je weglaten.

[wiskiv1]

Hey man, super tof dat je der tijd en moeite in hebt gestoken

maar ik ben vrij zeker dat je in "registrationshift" aan het werken bent, dit is de code voor de tijdsloten (niet van toepassing op deze issue)

Je moet in de 'normale' shift code werken, deze is voor de shiften.

De bestanden hebben wel verwarrende namen als je niet weet wat wat is.

grtjs
Witse

[Michiel-Vanzeir]

Weet niet zeker wat ik hier allemaal heb zitten doen met Git haha, wilde gewoon terug gaan naar oude versie zonder de commits maar kon dan niet pushen naar master.

Normaal gezien zou het nu in orde moeten zijn, was dezelfde logische fout maar in een ander bestand.
Hoe kan ik eigenlijk zelf testen of aanpassingen werken? Ik kan niks doen zonder in te loggen, maar kan de authenticator ook niet gebruiken precies.

[wiskiv1]

Je kan de authenticatie inderdaad niet gebruiken in je lokale versie van de Litus. Maar er zijn wat test accounts aangemaakt voor dit soort zaken.
Door te surfen naar "localhost/auth" kom je op een username / wachtwoord login pagina. Hier zijn er verschillende accounts die je kan gebruiken voor allerlei zaken te testen.

De accounts zijn:
naam (login) wachtwoord
Werkgroep IT (werkgroepit) password

• Admin

John Doe (johndoe) password

• Man
• Burgie
• niet preasidium
• r1000000

Jane Doe (janedoe) password

• Vrouw
• Archie
• niet preasidium
• r2000000

Papa Tiddo (papatiddo) password

• Man
• Burgie
• Preasidium
• r3000000

professor gobelijn (gobelijn) password

• Man
• Professor

@Tiddons

[Michiel-Vanzeir]

Perfect, bedankt voor de uitleg!
Net mijn changes getest, users krijgen nu een foutmelding indien ze twee shifts van hetzelfde tijdslot willen opnemen.