Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ADD] #34 expected hours to attendance report #37

Merged
merged 1 commit into from
Oct 18, 2023

Conversation

hbrunn
Copy link
Contributor

@hbrunn hbrunn commented Oct 2, 2023

No description provided.

@albig
Copy link
Member

albig commented Oct 9, 2023

Ich bin grad im Urlaub und hatte den PR nur kurz getestet. Die theoretischen Zeiten tauchen im Report auf. Soweit super. In einem Fall standen für einen Feiertag aber die volle Sollzeit, obwohl das in den Overtime richtig berücksichtigt wurde. Bei einem anderen Feiertag schien es zu stimmen. Das muss ich noch mal genauer untersuchen.

Werden denn Feiertage berücksichtigt?

@hbrunn
Copy link
Contributor Author

hbrunn commented Oct 11, 2023

@albig ich arbeite diese Woche etwas außerhalb des Turnus. Eigentlich sollten alle Fälle (Urlaub, Feiertag) berücksichtigt sein, habe soeben die Tests entsprechend angepasst und für Ostern funktioniert es.

Habe bei der Gelegenheit jedoch Trigger zur compute Funktion zugefügt, um auch auf von Hand angepasste Überstunden-Records (da speichere ich die Soll-Zeit weil wir da ohnehin einen Eintrag pro Tag haben) richtig zu berechnen, vielleicht war das das Problem?

@albig
Copy link
Member

albig commented Oct 16, 2023

@hbrunn Ich hab jetzt lange versucht dahinter zu kommen. Aktuell scheint es so zu sein, dass wir ein Feature "verloren" haben und Arbeiten an Feiertagen nicht als Overtime berechnet wird. Das hat aber nichts mit diesem PR zu tun, denn der Fehler ist jetzt schon "live".

Hier fällt mir nur auf, dass die manuell erstellten Overtimes (adjustment == True) nicht (mehr) im Bericht auftauchen. Obwohl sie es ja seit #23 müssten und ich das ja auch gesehen und gemerged habe.

Ich werde morgen noch mal einen Versuch starten und das Problem hoffentlich besser beschreiben.

@albig
Copy link
Member

albig commented Oct 18, 2023

Ich bin gestern lokal in der Versionierung auf den Stand von #23 zurück gegangen und habe vieles (unmögliche) probiert. Alles ohne Erfolg. Arbeiten am Feiertag wurde nicht als Überstunden berechnet.

Dann bin ich anders herangegangen und habe eine neue, leere Demo-Odoo-Datenbank angelegt, verdigado_attendance installiert, Feiertage konfiguriert und da funktionierte es.

Mein Verdacht ist momentan das Modul hr_attendance_overtime_negative_hours (https://github.com/Mint-System/Odoo-Apps-HR/tree/15.0/hr_attendance_overtime_negative_hours), was ich bei uns noch installiert habe, damit man die negativen Overtimes im Time-Off Modul sieht. Kannst Du bestätigen, dass sich das "beißt" mit der Änderung von #23?

Wenn ja, dann wäre das Verhalten erklärt, ich ärgere mich dann ne Runde und kann weiter machen...

@hbrunn hbrunn force-pushed the 15.0-34-expected-hours-report branch from cce461e to 7056d6b Compare October 18, 2023 10:54
Copy link
Member

@albig albig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Es scheint jetzt zu funktionieren. Hurra!

@albig albig merged commit 67bf5d7 into verdigado:15.0 Oct 18, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants