Update annual vacation days from last year to next year

[honnel]

fix #3623

[honnel]

Kann es Konstellationen geben, bei denen man nicht möchte, dass der jährliche Urlaubsanspruch ins Folgejahr übertragen wird?

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

100.0% Coverage
0.0% Duplication

[derTobsch]

Kann es Konstellationen geben, bei denen man nicht möchte, dass der jährliche Urlaubsanspruch ins Folgejahr übertragen wird?

ja ich glaube das ist ggf der Standard, wenn man was verändert hat. Ich denke wir sollten schauen ob die Daten verändert wurden und wenn ja dann die veränderten Übernehmen...muss mich aber auch nochmal besser reindenken

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

100.0% Coverage
0.0% Duplication

[sonarcloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[honnel]

ja ich glaube das ist ggf der Standard, wenn man was verändert hat. Ich denke wir sollten schauen ob die Daten verändert wurden und wenn ja dann die veränderten Übernehmen...muss mich aber auch nochmal besser reindenken

Ich meinte damit nicht den Standard-Wert aus den globalen Einstellungen, sondern der im Vorjahr konfigurierte jährliche Jahresanspruch im Konto der betreffenden Person

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[derTobsch]

Ist es korrekt, dass dann auch diese Informationen via updateRemainingVacationDays zum Beispiel aus einem cancel eines Urlaubes angepasst wird? Sollte man 'remaining vacation days' Berechnung nicht von den 'annual vacation days', welche einmalig beim Jahreswechsel angepasst werden packen?

Bin ab hin und her überlegen.

Wir sollten mal die Fälle durchdenken.

Angenommen wir sind im Jahr 2020:

1. Account für 2020 vorhanden, Account für 2021 nicht vorhanden, Jährlicher Urlaubsanspruch wird von 30 auf 31 in 2020 angepasst => derzeit: in 2021 werden die 31 übernommen | danach: in 2021 werden die 31 übernommen - finde ich nachvollziehbar
2. Account für 2020 vorhanden, Account für 2021 vorhanden, Jährlicher Urlaubsanspruch wird von 30 auf 31 in 2020 angepasst => derzeit: wird in 2021 nicht übernommen und muss händisch angepasst werden | danach: wird bei neuen Anträgen (apply/cancel) in 2021 upgedated - hier wissen wir nicht ob in 2021 bereits der Urlaubsanspruch händisch vorbereitet wurde auf zum Beispiel 2 Urlaubstage weniger und wir überschreiben dann den korrekten Wert - finde ich kritisch
3. Account für 2020 vorhanden, Account für 2021 vorhanden, Jährlicher Urlaubsanspruch wird von 30 auf 31 in 2021 angepasst => derzeit: der Urlaubsanspruch aus 2021 ist unabhängig von dem aus 2020 | danach: der korrekte 2021 Urlaubsanspruch wird mit dem falschen aus 2020 überschrieben => finde ich kritisch
4. Account für 2020 vorhanden, Account für 2021 vorhanden, Jährlicher Urlaubsanspruch wird von 30 auf 31 in 2021 und von 30 auf 28 in 2021 angepasst => derzeit: der Urlaubsanspruch aus 2021 ist unabhängig von dem aus 2020 und kann unabhängig gepflegt werden | danach: der korrekte neue 2021 Urlaubsanspruch wird mit dem für 2020 korrekten aber in 2021 falschen überschrieben => finde ich kritisch

Kann es sein das wir wissen müssen ob ein Account und ggf direkt er Jährlicher Urlaubsanspruch eines Accounts in der Zukunft bereits händisch angepasst wurde und nur wenn dieser noch nicht händisch angepasst wurde, dann passen wir die Daten an und sobald er händisch angepasst wurde machen wir nichts mehr außer den Resturlaub?