From ad805ca7060cc56d091d2822a726646880a337cc Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Duy=20=28=C4=90=E1=BB=97=20Anh=29?= Date: Wed, 2 Oct 2024 15:02:35 +0700 Subject: [PATCH] [MIG] hr_holidays_public: Migration to 18.0 --- hr_holidays_public/README.rst | 13 ++++++----- hr_holidays_public/__manifest__.py | 2 +- hr_holidays_public/models/hr_leave.py | 22 ++----------------- hr_holidays_public/models/hr_leave_type.py | 2 +- hr_holidays_public/readme/CONTRIBUTORS.md | 3 ++- hr_holidays_public/readme/CREDITS.md | 2 +- .../static/description/index.html | 9 ++++---- .../tests/test_holidays_calculation.py | 6 ----- .../views/hr_holidays_public_view.xml | 16 +++++++------- .../holidays_public_next_year_wizard.py | 2 +- .../holidays_public_next_year_wizard.xml | 2 +- 11 files changed, 29 insertions(+), 50 deletions(-) diff --git a/hr_holidays_public/README.rst b/hr_holidays_public/README.rst index 35b4b54a..4b9f7063 100644 --- a/hr_holidays_public/README.rst +++ b/hr_holidays_public/README.rst @@ -17,13 +17,13 @@ HR Holidays Public :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html :alt: License: AGPL-3 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fhr--holidays-lightgray.png?logo=github - :target: https://github.com/OCA/hr-holidays/tree/17.0/hr_holidays_public + :target: https://github.com/OCA/hr-holidays/tree/18.0/hr_holidays_public :alt: OCA/hr-holidays .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png - :target: https://translation.odoo-community.org/projects/hr-holidays-17-0/hr-holidays-17-0-hr_holidays_public + :target: https://translation.odoo-community.org/projects/hr-holidays-18-0/hr-holidays-18-0-hr_holidays_public :alt: Translate me on Weblate .. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png - :target: https://runboat.odoo-community.org/builds?repo=OCA/hr-holidays&target_branch=17.0 + :target: https://runboat.odoo-community.org/builds?repo=OCA/hr-holidays&target_branch=18.0 :alt: Try me on Runboat |badge1| |badge2| |badge3| |badge4| |badge5| @@ -104,7 +104,7 @@ Bug Tracker Bugs are tracked on `GitHub Issues `_. In case of trouble, please check there if your issue has already been reported. If you spotted it first, help us to smash it by providing a detailed and welcomed -`feedback `_. +`feedback `_. Do not contact contributors directly about support or help with technical issues. @@ -154,11 +154,12 @@ Contributors - `Trobz `__: - Thao Le + - Do Anh Duy Other credits ------------- -The migration of this module from 16.0 to 17.0 was financially supported +The migration of this module from 17.0 to 18.0 was financially supported by Camptocamp. Maintainers @@ -174,6 +175,6 @@ OCA, or the Odoo Community Association, is a nonprofit organization whose mission is to support the collaborative development of Odoo features and promote its widespread use. -This module is part of the `OCA/hr-holidays `_ project on GitHub. +This module is part of the `OCA/hr-holidays `_ project on GitHub. You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute. diff --git a/hr_holidays_public/__manifest__.py b/hr_holidays_public/__manifest__.py index 0c8b8e2e..140925bf 100644 --- a/hr_holidays_public/__manifest__.py +++ b/hr_holidays_public/__manifest__.py @@ -4,7 +4,7 @@ { "name": "HR Holidays Public", - "version": "17.0.1.0.3", + "version": "18.0.1.0.0", "license": "AGPL-3", "category": "Human Resources", "author": "Michael Telahun Makonnen, " diff --git a/hr_holidays_public/models/hr_leave.py b/hr_holidays_public/models/hr_leave.py index 0ee803a6..ec6c9a68 100644 --- a/hr_holidays_public/models/hr_leave.py +++ b/hr_holidays_public/models/hr_leave.py @@ -26,35 +26,17 @@ def action_validate(self): super(HrLeave, leave).action_validate() return True - def _get_duration(self, check_leave_type=True, resource_calendar=None): + def _get_durations(self, check_leave_type=True, resource_calendar=None): if self.holiday_status_id.exclude_public_holidays or not self.holiday_status_id: instance = self.with_context( employee_id=self.employee_id.id, exclude_public_holidays=True ) else: instance = self - return super(HrLeave, instance)._get_duration( + return super(HrLeave, instance)._get_durations( check_leave_type, resource_calendar ) - @api.depends("number_of_days") - def _compute_number_of_hours_display(self): - """If the leave is validated, no call to `_get_number_of_days` is done, so we - need to inject the context here for including the public holidays if applicable. - - For such cases, we need to serialize the call to super in fragments. - """ - to_serialize = self.filtered( - lambda x: x.state == "validate" - and x.holiday_status_id.exclude_public_holidays - ) - for leave in to_serialize: - leave = leave.with_context( - exclude_public_holidays=True, employee_id=leave.employee_id.id - ) - super(HrLeave, leave)._compute_number_of_hours_display() - return super(HrLeave, self - to_serialize)._compute_number_of_hours_display() - def _get_domain_from_get_unusual_days(self, date_from, date_to=None): domain = [("date", ">=", date_from)] # Use the employee of the user or the one who has the context diff --git a/hr_holidays_public/models/hr_leave_type.py b/hr_holidays_public/models/hr_leave_type.py index 1c4a6488..679ee7d3 100644 --- a/hr_holidays_public/models/hr_leave_type.py +++ b/hr_holidays_public/models/hr_leave_type.py @@ -10,5 +10,5 @@ class HrLeaveType(models.Model): exclude_public_holidays = fields.Boolean( default=True, - help=("If enabled, public holidays are skipped in leave days calculation."), + help="If enabled, public holidays are skipped in leave days calculation.", ) diff --git a/hr_holidays_public/readme/CONTRIBUTORS.md b/hr_holidays_public/readme/CONTRIBUTORS.md index 82c4161a..2805c94e 100644 --- a/hr_holidays_public/readme/CONTRIBUTORS.md +++ b/hr_holidays_public/readme/CONTRIBUTORS.md @@ -16,5 +16,6 @@ - Iván Todorovich \<\> - [Pesol](https://www.pesol.es): - Pedro Evaristo Gonzalez Sanchez \<\> -- [Trobz](https://trobz.com): +- [Trobz](https://trobz.com): - Thao Le \<\> + - Do Anh Duy \<\> diff --git a/hr_holidays_public/readme/CREDITS.md b/hr_holidays_public/readme/CREDITS.md index 8add88a2..83b3ec91 100644 --- a/hr_holidays_public/readme/CREDITS.md +++ b/hr_holidays_public/readme/CREDITS.md @@ -1 +1 @@ -The migration of this module from 16.0 to 17.0 was financially supported by Camptocamp. +The migration of this module from 17.0 to 18.0 was financially supported by Camptocamp. diff --git a/hr_holidays_public/static/description/index.html b/hr_holidays_public/static/description/index.html index b71e54f7..9e014f60 100644 --- a/hr_holidays_public/static/description/index.html +++ b/hr_holidays_public/static/description/index.html @@ -369,7 +369,7 @@

HR Holidays Public

!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! !! source digest: sha256:9e48583a15773c28f7337bac2983bbe2febc5b6775d3f410139232f20056ec56 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! --> -

Beta License: AGPL-3 OCA/hr-holidays Translate me on Weblate Try me on Runboat

+

Beta License: AGPL-3 OCA/hr-holidays Translate me on Weblate Try me on Runboat

This module handles public holidays.

The calculation of each leave can exclude rest public holiday, depending on the leave type configuration.

@@ -449,7 +449,7 @@

Bug Tracker

Bugs are tracked on GitHub Issues. In case of trouble, please check there if your issue has already been reported. If you spotted it first, help us to smash it by providing a detailed and welcomed -feedback.

+feedback.

Do not contact contributors directly about support or help with technical issues.

@@ -497,13 +497,14 @@

Contributors

  • Trobz:

    • Other credits

    -

    The migration of this module from 16.0 to 17.0 was financially supported +

    The migration of this module from 17.0 to 18.0 was financially supported by Camptocamp.

    @@ -515,7 +516,7 @@

    Maintainers

    OCA, or the Odoo Community Association, is a nonprofit organization whose mission is to support the collaborative development of Odoo features and promote its widespread use.

    -

    This module is part of the OCA/hr-holidays project on GitHub.

    +

    This module is part of the OCA/hr-holidays project on GitHub.

    You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

    diff --git a/hr_holidays_public/tests/test_holidays_calculation.py b/hr_holidays_public/tests/test_holidays_calculation.py index 3a1072dc..f1c1ea3b 100644 --- a/hr_holidays_public/tests/test_holidays_calculation.py +++ b/hr_holidays_public/tests/test_holidays_calculation.py @@ -124,7 +124,6 @@ def test_number_days_excluding_employee_1(self): "employee_id": self.employee_1.id, } ) - leave_request._compute_duration() self.assertEqual(leave_request.number_of_days, 4) def _test_number_days_excluding_employee_2(self): @@ -136,7 +135,6 @@ def _test_number_days_excluding_employee_2(self): "employee_id": self.employee_2.id, } ) - leave_request._compute_duration() self.assertEqual(leave_request.number_of_days, 2) def test_number_days_not_excluding(self): @@ -148,7 +146,6 @@ def test_number_days_not_excluding(self): "employee_id": self.employee_1.id, } ) - leave_request._compute_duration() self.assertEqual(leave_request.number_of_days, 5) def test_number_days_across_year(self): @@ -160,7 +157,6 @@ def test_number_days_across_year(self): "employee_id": self.employee_1.id, } ) - leave_request._compute_duration() self.assertEqual(leave_request.number_of_days, 7) def test_number_days_across_year_2(self): @@ -172,7 +168,6 @@ def test_number_days_across_year_2(self): "employee_id": self.employee_2.id, } ) - leave_request._compute_duration() self.assertEqual(leave_request.number_of_days, 5) def test_number_of_hours_excluding_employee_2(self): @@ -187,4 +182,3 @@ def test_number_of_hours_excluding_employee_2(self): ) self.assertEqual(leave_request.number_of_days, 2) - self.assertEqual(leave_request.number_of_hours_display, 16) diff --git a/hr_holidays_public/views/hr_holidays_public_view.xml b/hr_holidays_public/views/hr_holidays_public_view.xml index a4bc8579..39ddbc2c 100644 --- a/hr_holidays_public/views/hr_holidays_public_view.xml +++ b/hr_holidays_public/views/hr_holidays_public_view.xml @@ -1,13 +1,13 @@ - - hr.holidays.public.tree + + hr.holidays.public.list hr.holidays.public - + - + @@ -17,7 +17,7 @@
    - + @@ -26,7 +26,7 @@ - + - + @@ -47,7 +47,7 @@ Public Holidays hr.holidays.public - tree,form + list,form - +