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

[FIX]hr_holidays: set correct days when creating new leave request #28661

Closed

Conversation

pka-odoo
Copy link
Contributor

  • here, there is need to call '_onchange_leave_dates' method because this method computes number of days according to date_from and date_to fields which are set in method '_onchange_request_parameters'.
  • so, call '_onchange_leave_dates' after setting date_from and date_to parameters.

Related to Issue: 1895291

Description of the issue/feature this PR addresses:
When trying to create leaves(from a calendar) the first time it shows the wrong duration but click on a half day and unclick, it shows 1 day. in duration.

Current behavior before PR:
When trying to create leaves(from a calendar) the first time it shows the wrong duration but click on a half day and unclick, it shows 1 day. in duration.

Desired behavior after PR is merged:
It shows the correct duration while creating leave request from a calendar.

Issue:
https://www.odoo.com/web#id=1895291&action=327&model=project.task&view_type=form&menu_id=4720

Pad:
https://pad.odoo.com/p/r.79862f847d6b2334a59838d759dd152d

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

- here, there is need to call '_onchange_leave_dates' method because this method computes number of days according to date_from and date_to fields which are set in method '_onchange_request_parameters'.
- so, call '_onchange_leave_dates' after setting date_from and date_to parameters.

Related to Issue: 1895291
@C3POdoo C3POdoo added the RD research & development, internal work label Nov 14, 2018
@mart-e mart-e added OE the report is linked to a support ticket (opw-...) and removed RD research & development, internal work labels Nov 14, 2018
@mart-e

This comment has been minimized.

@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Nov 14, 2018
@lathama
Copy link
Contributor

lathama commented Apr 15, 2024

@pka-odoo knowing that 12 is no longer updated, is this PR still needed?

@pka-odoo
Copy link
Contributor Author

@pka-odoo knowing that 12 is no longer updated, is this PR still needed?

Not needed.

@pka-odoo pka-odoo closed this Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses OE the report is linked to a support ticket (opw-...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants