Gepardec / mega

Make the End-of-the-month-process Great Again! - A Gepardec Learning Friday Project
GNU General Public License v3.0
8 stars 2 forks source link

Urlaub/HO/ZA wird falsch berechnet #416

Closed WernerBruckmueller closed 2 years ago

WernerBruckmueller commented 3 years ago

image

image

WernerBruckmueller commented 3 years ago

Homeoffice, ZA und Urlaub soll überprüft werden

LeitnerMi commented 3 years ago

@WernerBruckmueller ich hab im test-zep ein paar Daten eingetragen und bei mir scheint das Ganze zu funktionieren: image image

Vielleicht durch das Freistelldatum im falschen Monat? Mir wäre auch nichts allzu ungewöhnliches im Code aufgefallen.

WernerBruckmueller commented 3 years ago

Urlaub wurde auf Durchgängig geändert, nicht einzlene Tage.

LeitnerMi commented 3 years ago

Wurde gefixt! Doch:

Urlaub, welcher durchgängig eingetragen wird und über Monatsgrenzen hinausgeht, kann sehr schwierig berechnet werden. Die Zep-Test-Umgebung lässt keinen Urlaub an Samstage/Sonntagen zu: image

Anscheinend kann man trotzdem (?) Fehlzeiten am Wochenende eintragen, deswegen konnte ich das noch nicht testen. Ansonsten funktioniert die Berechnung!

Es gibt aber auch noch die Möglichkeit in ZEP Halbtagsurlaub zu buchen. Diese werden noch nicht berücksichtigt!

phiwu-gpdec commented 3 years ago

Es soll der konsumierte Urlaub/ZA/HO des aktuellen Monats angezeigt werden. Es muss überprüft werden, ob der Urlaub über ein Wochenende oder einen Feiertag geht.

@LeitnerMi @insinnox

WernerBruckmueller commented 3 years ago

Siehe Referenzticket

LeitnerMi commented 3 years ago

Einträge werden im ZEP am Starttag + Dauer des Urlaubs/HO/ZA eingetragen. -> Sprich am 28. Juni 5 Tage Urlaub

Check auf Monatsübertritt: nur 2 Tage Urlaub (bis 30.)

Ollitod commented 3 years ago

@phiwu-gpdec Bug ist grundsätzlich gefixt, folgendes muss ich noch abklären: Urlaub und Zeitausgleich werden beim Sync Personio -> ZEP an Wochenenden und Feiertagen unterbrochen, Homeoffice kann aber immer über Wochenenden und Feiertage hinweg eingetragen werden. Wenn das seitens ZEP möglich ist, Homeoffice nach selbigen Kriterien zu unterbrechen, müssen wir uns in MEGA darüber keine Gedanken machen.

WernerBruckmueller commented 3 years ago

Bitte mit HR klären, ob eine solche "Unterbrechung" notwendig ist. Es sollte niemand HO am Wochenende eintragen (zumindest wäre das meine Annahme).

Ollitod commented 3 years ago

@WernerBruckmueller Ich habe Alexandra diesbezüglich gefragt und sie hat mir gesagt, dass dieses Verhalten von ZEP auch für die Lohnverrechnung einen Mehraufwand bedeutet.

WernerBruckmueller commented 3 years ago

Homeoffice ist keine Abwesenheit, somit kann man es auch bewusst an Feiertagen/WE eintragen (zB für Rufbereitschaft, ...)

WernerBruckmueller commented 2 years ago

Die angezeigten Werte (HO, Urlaub, ...) betrifft immer das angezeigte monat (zB wieviel Urlaub wurde genommen?)

WernerBruckmueller commented 2 years ago

@phiwu-gpdec bitte um aktuellen Status

phiwu-gpdec commented 2 years ago

@Ruhsi @andreas-repo hier ist im Backend zu klären, ob der aktuelle Stand pro Mitarbeiter und pro Monat abgerufen werden kann, oder nur die Daten pro Jahr. Weiters ist zu klären, ob die Daten von ZEP schon richtig geliefert werden oder ob hier noch ein Abgleich mit einem Kalender notwendig ist. Dh. 2 Wochen Urlaub durchgehend eingetragen - werden Sa/So/Feiertage berücksichtig und nur z.B. 9 Tage (2 Mal Sa/So und 1 Feiertag dabei) richtig berechnet?

andreas-repo commented 2 years ago

Habe Tests für MonthlyReportServiceImpl.java hinzugefügt welche für HO/ZA/UB prüfen:

-- wenn ein Eintrag über ein Wochenende geht werden nur Einträge mit einem Datum für den Monatsreport akzeptiert welches kein Wochenende ist -- wenn ein Eintrag über das Monat hinaus geht werden nur die Tage im korrekten Monat akzeptiert und für den Monatsreport verwendet.

Im ZEP sind schon Regeln implementiert wie zb. keine Urlaubseinträge an Wochenenden, etc welche durchgesetzt werden und deshalb scheint der Input von ZEP immer korrekt zu sein. Trotzdem werden die Daten noch im Backend abgeglichen und wenn nötig bearbeitet. Sollte es für bestimmte Mitarbeiter unterschiedliche Regeln geben, könnte es in der Zukunft nötig sein dies im Backend mit zu implementieren da im Moment alle gleich bearbeitet wird und die selbe Regeln gelten (keine Einträge am Wochenende usw).