diff --git a/hr_timesheet_overtime_rate_begin_end/README.rst b/hr_timesheet_overtime_rate_begin_end/README.rst new file mode 100644 index 0000000..ccc8206 --- /dev/null +++ b/hr_timesheet_overtime_rate_begin_end/README.rst @@ -0,0 +1,70 @@ +====================================================== +Timesheet - Overtime and begin/end hours compatibility +====================================================== + +.. + !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + !! This file is generated by oca-gen-addon-readme !! + !! changes will be overwritten. !! + !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + !! source digest: sha256:5ab790e671629b45c0297eceb50037e97ab4827da3d637687db534dd0dd0dde0 + !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + +.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png + :target: https://odoo-community.org/page/development-status + :alt: Beta +.. |badge2| image:: https://img.shields.io/badge/licence-AGPL--3-blue.png + :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html + :alt: License: AGPL-3 +.. |badge3| image:: https://img.shields.io/badge/github-coopiteasy%2Fcie--timesheet-lightgray.png?logo=github + :target: https://github.com/coopiteasy/cie-timesheet/tree/16.0/hr_timesheet_overtime_begin_end + :alt: coopiteasy/cie-timesheet + +|badge1| |badge2| |badge3| + +Make the two modules compatible. + +**Table of contents** + +.. contents:: + :local: + +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 `_. + +Do not contact contributors directly about support or help with technical issues. + +Credits +======= + +Authors +~~~~~~~ + +* Coop IT Easy SC + +Contributors +~~~~~~~~~~~~ + +* `Coop IT Easy SC `_: + + * Carmen Bianca BAKKER + +Maintainers +~~~~~~~~~~~ + +.. |maintainer-carmenbianca| image:: https://github.com/carmenbianca.png?size=40px + :target: https://github.com/carmenbianca + :alt: carmenbianca + +Current maintainer: + +|maintainer-carmenbianca| + +This module is part of the `coopiteasy/cie-timesheet `_ project on GitHub. + +You are welcome to contribute. diff --git a/hr_timesheet_overtime_rate_begin_end/__init__.py b/hr_timesheet_overtime_rate_begin_end/__init__.py new file mode 100644 index 0000000..3eb7887 --- /dev/null +++ b/hr_timesheet_overtime_rate_begin_end/__init__.py @@ -0,0 +1,5 @@ +# SPDX-FileCopyrightText: 2024 Coop IT Easy SC +# +# SPDX-License-Identifier: AGPL-3.0-or-later + +from . import models diff --git a/hr_timesheet_overtime_rate_begin_end/__manifest__.py b/hr_timesheet_overtime_rate_begin_end/__manifest__.py new file mode 100644 index 0000000..110f129 --- /dev/null +++ b/hr_timesheet_overtime_rate_begin_end/__manifest__.py @@ -0,0 +1,21 @@ +# SPDX-FileCopyrightText: 2024 Coop IT Easy SC +# +# SPDX-License-Identifier: AGPL-3.0-or-later + +{ + "name": "Timesheet - Overtime and begin/end hours compatibility", + "summary": """ + Make the two modules compatible.""", + "version": "12.0.1.0.0", + "category": "Human Resources", + "website": "https://coopiteasy.be", + "author": "Coop IT Easy SC", + "maintainers": ["carmenbianca"], + "license": "AGPL-3", + "application": False, + "depends": [ + "hr_timesheet_overtime", + "hr_timesheet_activity_begin_end", + ], + "auto_install": True, +} diff --git a/hr_timesheet_overtime_rate_begin_end/models/__init__.py b/hr_timesheet_overtime_rate_begin_end/models/__init__.py new file mode 100644 index 0000000..84f4adc --- /dev/null +++ b/hr_timesheet_overtime_rate_begin_end/models/__init__.py @@ -0,0 +1,5 @@ +# SPDX-FileCopyrightText: 2024 Coop IT Easy SC +# +# SPDX-License-Identifier: AGPL-3.0-or-later + +from . import account_analytic_line diff --git a/hr_timesheet_overtime_rate_begin_end/models/account_analytic_line.py b/hr_timesheet_overtime_rate_begin_end/models/account_analytic_line.py new file mode 100644 index 0000000..2f5c760 --- /dev/null +++ b/hr_timesheet_overtime_rate_begin_end/models/account_analytic_line.py @@ -0,0 +1,39 @@ +# SPDX-FileCopyrightText: 2024 Coop IT Easy SC +# +# SPDX-License-Identifier: AGPL-3.0-or-later + +from odoo import api, models + + +class AnalyticLine(models.Model): + _inherit = "account.analytic.line" + + # This is a bit of a hack. Normally in hr_timesheet_overtime, the value of + # unit_amount is updated in this method as part of writing or creating a + # record. However, in hr_timesheet_activity_begin_end, unit_amount is + # adjusted during an onchange using unit_amount_from_start_stop. + # + # If BOTH the onchange and the write method adjust the value of unit_amount, + # then the rate is applied twice, which is not good. Therefore, this method + # is 'disabled', and we instead rely on onchange to correctly set the value + # of unit_amount. + # + # The disadvantage is that programmatic usages of hr_timesheet_overtime no + # longer work here. The advantage is that you can now manually override the + # value of unit_amount without the rate being automatically applied. + # + # In version 16, this behaviour is a lot less messy, because it uses compute + # functions instead of the aforementioned methods. + @api.model + def _update_values(self, values): + time_start = values.get("time_start", self.time_start) + time_stop = values.get("time_stop", self.time_stop) + # Do not double-compute if we're using times. + if time_start or time_stop: + return + return super()._update_values(values) + + def unit_amount_from_start_stop(self): + result = super().unit_amount_from_start_stop() + result *= self.rate_for_date(self.date) + return result diff --git a/hr_timesheet_overtime_rate_begin_end/readme/CONTRIBUTORS.rst b/hr_timesheet_overtime_rate_begin_end/readme/CONTRIBUTORS.rst new file mode 100644 index 0000000..f1ac675 --- /dev/null +++ b/hr_timesheet_overtime_rate_begin_end/readme/CONTRIBUTORS.rst @@ -0,0 +1,3 @@ +* `Coop IT Easy SC `_: + + * Carmen Bianca BAKKER diff --git a/hr_timesheet_overtime_rate_begin_end/readme/DESCRIPTION.rst b/hr_timesheet_overtime_rate_begin_end/readme/DESCRIPTION.rst new file mode 100644 index 0000000..58c190c --- /dev/null +++ b/hr_timesheet_overtime_rate_begin_end/readme/DESCRIPTION.rst @@ -0,0 +1 @@ +Make the two modules compatible. diff --git a/hr_timesheet_overtime_rate_begin_end/static/description/index.html b/hr_timesheet_overtime_rate_begin_end/static/description/index.html new file mode 100644 index 0000000..a9ed5b0 --- /dev/null +++ b/hr_timesheet_overtime_rate_begin_end/static/description/index.html @@ -0,0 +1,420 @@ + + + + + +Timesheet - Overtime and begin/end hours compatibility + + + +
+

Timesheet - Overtime and begin/end hours compatibility

+ + +

Beta License: AGPL-3 coopiteasy/cie-timesheet

+

Make the two modules compatible.

+

Table of contents

+ +
+

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.

+

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

+
+
+

Credits

+
+

Authors

+
    +
  • Coop IT Easy SC
    • +
+
+
+

Contributors

+ +
+
+

Maintainers

+

Current maintainer:

+

carmenbianca

+

This module is part of the coopiteasy/cie-timesheet project on GitHub.

+

You are welcome to contribute.

+
+
+
+ + diff --git a/hr_timesheet_overtime_rate_begin_end/tests/__init__.py b/hr_timesheet_overtime_rate_begin_end/tests/__init__.py new file mode 100644 index 0000000..926bcd5 --- /dev/null +++ b/hr_timesheet_overtime_rate_begin_end/tests/__init__.py @@ -0,0 +1,5 @@ +# SPDX-FileCopyrightText: 2024 Coop IT Easy SC +# +# SPDX-License-Identifier: AGPL-3.0-or-later + +from . import test_analytic_line diff --git a/hr_timesheet_overtime_rate_begin_end/tests/test_analytic_line.py b/hr_timesheet_overtime_rate_begin_end/tests/test_analytic_line.py new file mode 100644 index 0000000..af141ff --- /dev/null +++ b/hr_timesheet_overtime_rate_begin_end/tests/test_analytic_line.py @@ -0,0 +1,74 @@ +# SPDX-FileCopyrightText: 2024 Coop IT Easy SC +# +# SPDX-License-Identifier: AGPL-3.0-or-later + +from odoo.tests.common import SavepointCase + + +class TestAnalyticLine(SavepointCase): + @classmethod + def setUpClass(cls): + super().setUpClass() + cls.project = cls.env.ref("project.project_project_1") + cls.employee = cls.env.ref("hr.employee_qdp") + # I'm not sure what this record does in this context. + cls.overtime = cls.env["resource.overtime"].create({"name": "test"}) + cls.rate = cls.env["resource.overtime.rate"].create( + { + "name": "test", + "dayofweek": "0", # Monday + "rate": 2.0, + "overtime_id": cls.overtime.id, + } + ) + + def base_line(self): + return { + "name": "test", + "date": "2024-01-01", # Monday + "time_start": 10.0, + "time_stop": 12.0, + "project_id": self.project.id, + "employee_id": self.employee.id, + } + + def test_rate_applied(self): + line = self.base_line() + line_record = self.env["account.analytic.line"].new(line) + line_record.onchange_hours_start_stop() + self.assertEqual(line_record.unit_amount, 4.0) + + def test_rate_not_double_applied(self): + line = self.base_line() + del line["time_start"] + del line["time_stop"] + + # Emulating a transient record before it is actually created. + line_new = self.env["account.analytic.line"].new(line) + line_new.time_start = 10.0 + line_new.time_stop = 12.0 + line_new.onchange_hours_start_stop() + self.assertEqual(line_new.unit_amount, 4.0) + + # Prepare the transient data for writing to a new record. + # + # Annoyingly, the value for unit_amount is not in _cache, so we have to + # add it here. + # + # Ideally I would emulate this with a Form(), but the necessary fields + # are not available in a form. This is the closest emulation. + vals = line_new._convert_to_write(line_new._cache) + vals.setdefault("unit_amount", line_new.unit_amount) + line_record = self.env["account.analytic.line"].create(vals) + + # The rate was already applied on the transient record. Don't also apply + # it on creation. + self.assertEqual(line_record.unit_amount, 4.0) + + def test_rate_applied_if_no_times(self): + line = self.base_line() + del line["time_start"] + del line["time_stop"] + line["unit_amount"] = 1 + line_record = self.env["account.analytic.line"].create(line) + self.assertEqual(line_record.unit_amount, 2) diff --git a/setup/hr_timesheet_overtime_rate_begin_end/odoo/addons/hr_timesheet_overtime_rate_begin_end b/setup/hr_timesheet_overtime_rate_begin_end/odoo/addons/hr_timesheet_overtime_rate_begin_end new file mode 120000 index 0000000..348076d --- /dev/null +++ b/setup/hr_timesheet_overtime_rate_begin_end/odoo/addons/hr_timesheet_overtime_rate_begin_end @@ -0,0 +1 @@ +../../../../hr_timesheet_overtime_rate_begin_end \ No newline at end of file diff --git a/setup/hr_timesheet_overtime_rate_begin_end/odoo/addons/hr_timesheet_rate_overtime_begin_end b/setup/hr_timesheet_overtime_rate_begin_end/odoo/addons/hr_timesheet_rate_overtime_begin_end new file mode 120000 index 0000000..2704a92 --- /dev/null +++ b/setup/hr_timesheet_overtime_rate_begin_end/odoo/addons/hr_timesheet_rate_overtime_begin_end @@ -0,0 +1 @@ +../../../../hr_timesheet_rate_overtime_begin_end \ No newline at end of file diff --git a/setup/hr_timesheet_overtime_rate_begin_end/setup.py b/setup/hr_timesheet_overtime_rate_begin_end/setup.py new file mode 100644 index 0000000..28c57bb --- /dev/null +++ b/setup/hr_timesheet_overtime_rate_begin_end/setup.py @@ -0,0 +1,6 @@ +import setuptools + +setuptools.setup( + setup_requires=['setuptools-odoo'], + odoo_addon=True, +) diff --git a/test-requirements.txt b/test-requirements.txt index cf1acdf..5a43aed 100644 --- a/test-requirements.txt +++ b/test-requirements.txt @@ -1,2 +1,4 @@ +freezegun +git+https://github.com/coopiteasy/timesheet.git@12.0-backport-refactor-begin-end#subdirectory=setup/hr_timesheet_activity_begin_end git+https://github.com/coopiteasy/addons@12.0#subdirectory=setup/resource_work_time_from_contracts git+https://github.com/coopiteasy/addons@12.0#subdirectory=setup/company_today