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

Soll-Arbeitszeit Anwesenheitsbericht fehlerhaft, wenn ist == soll #65

Closed
albig opened this issue Nov 23, 2023 · 5 comments
Closed

Soll-Arbeitszeit Anwesenheitsbericht fehlerhaft, wenn ist == soll #65

albig opened this issue Nov 23, 2023 · 5 comments
Assignees
Labels
bug Something isn't working Prio 2

Comments

@albig
Copy link
Member

albig commented Nov 23, 2023

Im Bericht "Anwesenheiten" -> "Berichtswesen" stimmt die Spalte "Soll-Zeit" nicht, wenn genau die Soll-Arbeitszeit eingetragen wurde.

Beispiel:

MA arbeitet regulär 6:24h pro Tag. Er bucht am 21. und 22.11.2023 jeweils genau 6:24h.

  • am 21.11.2023 ist regulärer Arbeitstag. Hier erscheint als Soll-Arbeitszeit "0:00" (falsch). Auf das Gleitzeitkonto gehen "0:00" (das ist korrekt) (1+2)
  • am 22.11.2023 ist Feiertag, die Soll-Arbeitszeit wird (korrekt) mit "0:00" angezeigt. Auf das Gleitzeitkonto gehen "6:24". (3+4)
  • bucht der MA nur 6:23h, dann steht am regulären Arbeitstag eine Soll-Arbeitszeit von "6:24".

2023-11-23_09-14

@albig albig added Prio 2 bug Something isn't working labels Nov 23, 2023
@albig
Copy link
Member Author

albig commented Nov 30, 2023

Ich musste #67 leider mit #74 reverten. Normale Mitarbeiter*innen können sonst keine Attendance-Records mehr anlegen.

Fehlermeldung:

Clipboard - November 30, 2023 8_03

@hbrunn
Copy link
Contributor

hbrunn commented Nov 30, 2023

oh nein, da fehlt ein sudo() in https://github.com/verdigado/odoo-customize/pull/67/files#diff-14b295672daec20809ffd281a7f649c723c81c2bfe09d6f2ae4f4f105642c0b8R36 falls Du das selbst reparieren willst, ich bin gerade anderweitig eingespannt

@hbrunn
Copy link
Contributor

hbrunn commented Nov 30, 2023

Also es muss:

self.env["hr.attendance.overtime"].sudo().create(missing_vals)

sein. Wir sollten echt bald test cases fuer alle Rollen bauen, dann kann sowas nicht mehr passieren, weil die CI schon geschrien haette.

@albig
Copy link
Member Author

albig commented Nov 30, 2023

Danke, das probier ich aus. Morgen. Heute muss ich leider zu anderen Dingen...

@albig
Copy link
Member Author

albig commented Dec 1, 2023

Closed with #76

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Prio 2
Projects
None yet
Development

No branches or pull requests

2 participants