Aug 12: [FIX] Bug Fixed 'employee_late_check_in'

employee_late_check_in/README.rst

@@ -1,6 +1,6 @@
-.. image:: https://img.shields.io/badge/license-AGPL--3-blue.svg
+.. image:: https://img.shields.io/badge/license-LGPL--3-blue.svg
-    :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
+    :target: https://www.gnu.org/licenses/lgpl-3.0-standalone.html
-    :alt: License: AGPL-3
+    :alt: License: LGPL-3
 
 Employee Late Check-in
 ======================
@@ -16,8 +16,8 @@ Company
 
 License
 -------
-Affero General Public License, Version 3 (AGPL v3).
+* Lesser General Public License, Version 3 (LGPL v3).
-(https://www.gnu.org/licenses/agpl-3.0-standalone.html)
+(https://www.gnu.org/licenses/lgpl-3.0-standalone.html)
 
 Credits
 -------

employee_late_check_in/__init__.py

@@ -1,35 +1,22 @@
 # -*- coding: utf-8 -*-
-###############################################################################
+#############################################################################
 #
 #    Cybrosys Technologies Pvt. Ltd.
 #
 #    Copyright (C) 2024-TODAY Cybrosys Technologies(<https://www.cybrosys.com>)
-#    Author: Cybrosys Techno Solutions (odoo@cybrosys.com)
+#    Author: Cybrosys Techno Solutions(<https://www.cybrosys.com>)
 #
-#    You can modify it under the terms of the GNU AFFERO
+#    You can modify it under the terms of the GNU LESSER
-#    GENERAL PUBLIC LICENSE (AGPL v3), Version 3.
+#    GENERAL PUBLIC LICENSE (LGPL v3), Version 3.
 #
 #    This program is distributed in the hope that it will be useful,
 #    but WITHOUT ANY WARRANTY; without even the implied warranty of
 #    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-#    GNU AFFERO GENERAL PUBLIC LICENSE (AGPL v3) for more details.
+#    GNU LESSER GENERAL PUBLIC LICENSE (LGPL v3) for more details.
 #
-#    You should have received a copy of the GNU AFFERO GENERAL PUBLIC LICENSE
+#    You should have received a copy of the GNU LESSER GENERAL PUBLIC LICENSE
-#    (AGPL v3) along with this program.
+#    (LGPL v3) along with this program.
 #    If not, see <http://www.gnu.org/licenses/>.
 #
-###############################################################################
+############################################################################.
 from . import models
-from odoo import SUPERUSER_ID
-from odoo.api import Environment
-
-def post_init_hook(cr, registry):
-    env = Environment(cr, SUPERUSER_ID, {})
-
-    cron = env.ref('employee_late_check_in.ir_cron_late_check_in',
-                   raise_if_not_found=False)
-    if cron:
-        model = env[cron.model_id.model]
-        method = getattr(model, 'late_check_in_records', None)
-        if method:
-            method()

employee_late_check_in/__manifest__.py

@@ -1,57 +1,56 @@
 # -*- coding: utf-8 -*-
-###############################################################################
+#############################################################################
 #
 #    Cybrosys Technologies Pvt. Ltd.
 #
 #    Copyright (C) 2024-TODAY Cybrosys Technologies(<https://www.cybrosys.com>)
-#    Author: Cybrosys Techno Solutions (odoo@cybrosys.com)
+#    Author: Cybrosys Techno Solutions(<https://www.cybrosys.com>)
 #
-#    You can modify it under the terms of the GNU AFFERO
+#    You can modify it under the terms of the GNU LESSER
-#    GENERAL PUBLIC LICENSE (AGPL v3), Version 3.
+#    GENERAL PUBLIC LICENSE (LGPL v3), Version 3.
 #
 #    This program is distributed in the hope that it will be useful,
 #    but WITHOUT ANY WARRANTY; without even the implied warranty of
 #    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-#    GNU AFFERO GENERAL PUBLIC LICENSE (AGPL v3) for more details.
+#    GNU LESSER GENERAL PUBLIC LICENSE (LGPL v3) for more details.
 #
-#    You should have received a copy of the GNU AFFERO GENERAL PUBLIC LICENSE
+#    You should have received a copy of the GNU LESSER GENERAL PUBLIC LICENSE
-#    (AGPL v3) along with this program.
+#    (LGPL v3) along with this program.
 #    If not, see <http://www.gnu.org/licenses/>.
 #
-###############################################################################
+############################################################################.
 {
-    "name": "Employee Late Check-in",
+    'name': 'Employee Late Check-in',
-    "version": "16.0.1.0.1",
+    'version': '16.0.1.0.0',
-    "category": "Human Resources",
+    'category': 'Human Resources',
-    "summary": "Employee Late Check-in module for tracking and managing late "
+    'summary': 'Employee Late Check-in module for tracking and managing late '
-    "check-ins of employees and may deduct salary from payslip",
+               'check-ins of employees and may deduct salary from payslip',
-    "description": "The module is designed for meticulous tracking and "
+    'description': 'The module is designed for meticulous tracking and '
-    "management of employee punctuality. It enables "
+                   'management of employee punctuality. It enables '
-    "organizations to monitor late check-ins efficiently, "
+                   'organizations to monitor late check-ins efficiently, '
-    "offering insights into tardiness patterns. With a "
+                   'offering insights into tardiness patterns. With a '
-    "user-friendly interface, it provides a comprehensive "
+                   'user-friendly interface, it provides a comprehensive '
-    "overview of individual employee records, facilitating "
+                   'overview of individual employee records, facilitating '
-    "timely interventions. This module contributes to fostering"
+                   'timely interventions. This module contributes to fostering'
-    " a punctual and efficient work environment.",
+                   ' a punctual and efficient work environment.',
-    "author": "Cybrosys Techno Solutions",
+    'author': "Cybrosys Techno Solutions",
-    "company": "Cybrosys Techno Solutions",
+    'company': 'Cybrosys Techno Solutions',
-    "maintainer": "Cybrosys Techno Solutions",
+    'maintainer': 'Cybrosys Techno Solutions',
-    "website": "https://www.cybrosys.com",
+    'website': 'https://www.cybrosys.com',
-    "depends": ["hr_attendance", "hr_payroll_community"],
+    'depends': ['hr_attendance', 'hr_payroll_community', 'hr_contract_types'],
-    "data": [
+    'data': [
-        "security/ir.model.access.csv",
+        'security/ir.model.access.csv',
-        "data/ir_cron_data.xml",
+        'data/ir_cron_data.xml',
-        "data/salary_rule.xml",
+        'data/salary_rule.xml',
-        "views/res_config_settings_views.xml",
+        'views/res_config_settings_views.xml',
-        "views/hr_attendance_views.xml",
+        'views/hr_attendance_views.xml',
-        "views/late_check_in_views.xml",
+        'views/late_check_in_views.xml',
-        "views/hr_employee_views.xml",
+        'views/hr_employee_views.xml',
-        "views/hr_payslip_views.xml",
+        'views/hr_payslip_views.xml',
     ],
-    "images": ["static/description/banner.png"],
+    'images': ['static/description/banner.png'],
-    "license": "AGPL-3",
+    'license': 'LGPL-3',
-    "installable": True,
+    'installable': True,
-    "auto_install": False,
+    'auto_install': False,
-    "application": False,
+    'application': False,
-    "post_init_hook": "post_init_hook"
 }

employee_late_check_in/data/ir_cron_data.xml

@@ -1,17 +1,15 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
-    <!--The daily function late_check_in_records verifies whether employees are tardy.-->
+        <!--The daily function late_check_in_records verifies whether employees are tardy.-->
-    <record forcecreate="True" id="ir_cron_late_check_in" model="ir.cron">
+        <record forcecreate="True" id="ir_cron_late_check_in" model="ir.cron">
-        <field name="name">Attendance: Late Check-in</field>
+            <field name="name">Attendance: Late Check-in</field>
-        <field name="model_id" ref="model_hr_attendance"/>
+            <field name="model_id" ref="model_hr_attendance"/>
-        <field name="state">code</field>
+            <field name="state">code</field>
-        <field name="code">model.late_check_in_records()</field>
+            <field name="code">model.late_check_in_records()</field>
-        <field name="user_id" ref="base.user_root"/>
+            <field name="user_id" ref="base.user_root" />
-        <field name="interval_number">1</field>
+            <field name="interval_number">1</field>
-        <field name="interval_type">days</field>
+            <field name="interval_type">days</field>
-        <field name="numbercall">-1</field>
+            <field name="numbercall">-1</field>
-        <field name="doall" eval="False"/>
+            <field name="doall" eval="False"  />
-    </record>
+        </record>
 </odoo>
-
-

employee_late_check_in/data/salary_rule.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <odoo>
-    <!--Calculate the amount for the 'Late Check-in' rule.-->
+        <!--Calculate the amount for the 'Late Check-in' rule.-->
     <record id="late_check_in" model="hr.salary.rule">
         <field name="name">Late Check-in</field>
         <field name="sequence" eval="6"/>
@@ -14,7 +14,8 @@ try:
         amount = inputs.LC.amount
 except:
     amount = 0
-result = -amount</field>
+result = -amount
+        </field>
     </record>
     <!--Salary structure for late check-in-->
     <record id="late_check_in_salary_structure" model="hr.payroll.structure">

employee_late_check_in/doc/RELEASE_NOTES.md

@@ -10,3 +10,10 @@
 #### Version 16.0.1.0.1
 #### UPDATE
  - Updated scheduled action and ensured amount updates in the late-check-in. Displayed time-off when generating payslip and deducted late check in amount from it.
+
+#### 12.08.2024
+#### Version 16.0.1.0.2
+#### UPDATE
+ - Bug Fix in Employee Late Check-in
+
+

employee_late_check_in/models/__init__.py

@@ -1,24 +1,24 @@
 # -*- coding: utf-8 -*-
-###############################################################################
+#############################################################################
 #
 #    Cybrosys Technologies Pvt. Ltd.
 #
 #    Copyright (C) 2024-TODAY Cybrosys Technologies(<https://www.cybrosys.com>)
-#    Author: Cybrosys Techno Solutions (odoo@cybrosys.com)
+#    Author: Cybrosys Techno Solutions(<https://www.cybrosys.com>)
 #
-#    You can modify it under the terms of the GNU AFFERO
+#    You can modify it under the terms of the GNU LESSER
-#    GENERAL PUBLIC LICENSE (AGPL v3), Version 3.
+#    GENERAL PUBLIC LICENSE (LGPL v3), Version 3.
 #
 #    This program is distributed in the hope that it will be useful,
 #    but WITHOUT ANY WARRANTY; without even the implied warranty of
 #    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-#    GNU AFFERO GENERAL PUBLIC LICENSE (AGPL v3) for more details.
+#    GNU LESSER GENERAL PUBLIC LICENSE (LGPL v3) for more details.
 #
-#    You should have received a copy of the GNU AFFERO GENERAL PUBLIC LICENSE
+#    You should have received a copy of the GNU LESSER GENERAL PUBLIC LICENSE
-#    (AGPL v3) along with this program.
+#    (LGPL v3) along with this program.
 #    If not, see <http://www.gnu.org/licenses/>.
 #
-###############################################################################
+############################################################################.
 from . import hr_attendance
 from . import hr_employee
 from . import hr_employees_public

employee_late_check_in/models/hr_attendance.py

@@ -1,40 +1,39 @@
 # -*- coding: utf-8 -*-
-###############################################################################
+#############################################################################
 #
 #    Cybrosys Technologies Pvt. Ltd.
 #
 #    Copyright (C) 2024-TODAY Cybrosys Technologies(<https://www.cybrosys.com>)
-#    Author: Cybrosys Techno Solutions (odoo@cybrosys.com)
+#    Author: Cybrosys Techno Solutions(<https://www.cybrosys.com>)
 #
-#    You can modify it under the terms of the GNU AFFERO
+#    You can modify it under the terms of the GNU LESSER
-#    GENERAL PUBLIC LICENSE (AGPL v3), Version 3.
+#    GENERAL PUBLIC LICENSE (LGPL v3), Version 3.
 #
 #    This program is distributed in the hope that it will be useful,
 #    but WITHOUT ANY WARRANTY; without even the implied warranty of
 #    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-#    GNU AFFERO GENERAL PUBLIC LICENSE (AGPL v3) for more details.
+#    GNU LESSER GENERAL PUBLIC LICENSE (LGPL v3) for more details.
 #
-#    You should have received a copy of the GNU AFFERO GENERAL PUBLIC LICENSE
+#    You should have received a copy of the GNU LESSER GENERAL PUBLIC LICENSE
-#    (AGPL v3) along with this program.
+#    (LGPL v3) along with this program.
 #    If not, see <http://www.gnu.org/licenses/>.
 #
-###############################################################################
+############################################################################.
 import pytz
 from datetime import datetime, timedelta
-from odoo import fields, models
+from odoo import api,fields, models
 
 
 class HrAttendance(models.Model):
     """Inherit the module to add fields and methods"""
-
+    _inherit = 'hr.attendance'
-    _inherit = "hr.attendance"
 
     late_check_in = fields.Integer(
-        string="Late Check-in(Minutes)",
+        string="Late Check-in(Minutes)", compute="_compute_late_check_in",
-        compute="_compute_late_check_in",
         help="This indicates the duration of the employee's tardiness.",
-    )
+        store=True)
 
+    @api.depends('check_in')
     def _compute_late_check_in(self):
         """Calculate late check-in minutes for each record in the current Odoo
         model.This method iterates through the records and calculates late
@@ -44,77 +43,62 @@ class HrAttendance(models.Model):
         for rec in self:
             rec.late_check_in = 0.0
             if rec.employee_id.contract_id:
-                for schedule in (
+                for schedule in rec.sudo().employee_id.contract_id.resource_calendar_id.sudo().attendance_ids:
-                    rec.sudo()
+                    if (schedule.dayofweek == str(
-                    .employee_id.contract_id.resource_calendar_id.sudo()
+                            rec.sudo().check_in.weekday()) and
-                    .attendance_ids
+                            schedule.day_period == 'morning'):
-                ):
-                    if (
-                        schedule.dayofweek == str(rec.sudo().check_in.weekday())
-                        and schedule.day_period == "morning"
-                    ):
                         dt = rec.check_in
                         if self.env.user.tz in pytz.all_timezones:
-                            old_tz = pytz.timezone("UTC")
+                            old_tz = pytz.timezone('UTC')
                             new_tz = pytz.timezone(self.env.user.tz)
                             dt = old_tz.localize(dt).astimezone(new_tz)
                         str_time = dt.strftime("%H:%M")
-                        check_in_date = datetime.strptime(str_time, "%H:%M").time()
+                        check_in_date = datetime.strptime(
+                            str_time, "%H:%M").time()
                         start_date = datetime.strptime(
-                            "{0:02.0f}:{1:02.0f}".format(
+                            '{0:02.0f}:{1:02.0f}'.format(*divmod(
-                                *divmod(schedule.hour_from * 60, 60)
+                                schedule.hour_from * 60, 60)),
-                            ),
+                            "%H:%M").time()
-                            "%H:%M",
+                        check_in = timedelta(hours=check_in_date.hour,
-                        ).time()
+                                             minutes=check_in_date.minute)
-                        check_in = timedelta(
+                        start_date = timedelta(hours=start_date.hour,
-                            hours=check_in_date.hour, minutes=check_in_date.minute
+                                               minutes=start_date.minute)
-                        )
-                        start_date = timedelta(
-                            hours=start_date.hour, minutes=start_date.minute
-                        )
                         if check_in > start_date:
                             final = check_in - start_date
                             rec.sudo().late_check_in = final.total_seconds() / 60
 
     def late_check_in_records(self):
         """Function creates records in late.check.in model for the employees
-        who were late as a scheduled action"""
+        who were late"""
-        minutes_after = (
+        minutes_after = int(
-            int(
+            self.env['ir.config_parameter'].sudo().get_param(
-                self.env["ir.config_parameter"]
+                'employee_late_check_in.late_check_in_after')) or 0
-                .sudo()
+        max_limit = int(self.env['ir.config_parameter'].sudo().get_param(
-                .get_param("employee_late_check_in.late_check_in_after")
+            'employee_late_check_in.maximum_minutes')) or 0
-            )
+        for rec in self.sudo().search([]):
-            or 0
+            print(rec,minutes_after)
-        )
+            if rec.id not in self.env['late.check.in'].sudo().search(
-        max_limit = (
+                    []).attendance_id.ids:
-            int(
+                if minutes_after < rec.late_check_in < max_limit:
-                self.env["ir.config_parameter"]
+                    print(rec,'rec')
-                .sudo()
+                    self.env['late.check.in'].sudo().create({
-                .get_param("employee_late_check_in.maximum_minutes")
+                        'employee_id': rec.employee_id.id,
-            )
+                        'late_minutes': rec.late_check_in,
-            or 0
+                        'date': rec.check_in.date(),
-        )
+                        'attendance_id': rec.id,
-        for rec in self.sudo().search(
+                    })
-            [
+            else:
-                (
+                self.env['late.check.in'].sudo().search(
-                    "id",
+                    [('attendance_id', '=', rec.id)]).update({
-                    "not in",
+                    'late_minutes': rec.late_check_in,
-                    self.env["late.check.in"].sudo().search([]).attendance_id.ids,
+                    'date': rec.check_in.date(),
-                )
+                    'attendance_id': rec.id,
-            ]
+                })
-        ):
+
-            late_check_in = rec.sudo().late_check_in + 210
+    def unlink(self):
-            if (
+        """Override the unlink method to delete the corresponding records in
-                rec.late_check_in > minutes_after
+        late.check.in model"""
-                and late_check_in > minutes_after
+        for record in self:
-                and late_check_in < max_limit
+            self.env['late.check.in'].sudo().search(
-            ):
+                [('attendance_id', '=', record.id)]).unlink()
-                self.env["late.check.in"].sudo().create(
+            return super(HrAttendance, self).unlink()
-                    {
-                        "employee_id": rec.employee_id.id,
-                        "late_minutes": late_check_in,
-                        "date": rec.check_in.date(),
-                        "attendance_id": rec.id,
-                    }
-                )

employee_late_check_in/models/hr_employee.py

@@ -1,56 +1,51 @@
 # -*- coding: utf-8 -*-
-###############################################################################
+#############################################################################
 #
 #    Cybrosys Technologies Pvt. Ltd.
 #
 #    Copyright (C) 2024-TODAY Cybrosys Technologies(<https://www.cybrosys.com>)
-#    Author: Cybrosys Techno Solutions (odoo@cybrosys.com)
+#    Author: Cybrosys Techno Solutions(<https://www.cybrosys.com>)
 #
-#    You can modify it under the terms of the GNU AFFERO
+#    You can modify it under the terms of the GNU LESSER
-#    GENERAL PUBLIC LICENSE (AGPL v3), Version 3.
+#    GENERAL PUBLIC LICENSE (LGPL v3), Version 3.
 #
 #    This program is distributed in the hope that it will be useful,
 #    but WITHOUT ANY WARRANTY; without even the implied warranty of
 #    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-#    GNU AFFERO GENERAL PUBLIC LICENSE (AGPL v3) for more details.
+#    GNU LESSER GENERAL PUBLIC LICENSE (LGPL v3) for more details.
 #
-#    You should have received a copy of the GNU AFFERO GENERAL PUBLIC LICENSE
+#    You should have received a copy of the GNU LESSER GENERAL PUBLIC LICENSE
-#    (AGPL v3) along with this program.
+#    (LGPL v3) along with this program.
 #    If not, see <http://www.gnu.org/licenses/>.
 #
-###############################################################################
+############################################################################.
-from odoo import fields, models, _
+from odoo import _, fields, models
 
 
 class HrEmployee(models.Model):
     """Inherit the model to add fields and methods"""
-
+    _inherit = 'hr.employee'
-    _inherit = "hr.employee"
 
     late_check_in_count = fields.Integer(
-        string="Late Check-In",
+        string="Late Check-In", compute="_compute_late_check_in_count",
-        compute="_compute_late_check_in_count",
+        help="Count of employee's late checkin")
-        help="Count of employee's late checkin",
-    )
 
     def action_to_open_late_check_in_records(self):
         """
-        :return: dictionary defining the action to open the late check-in
+            :return: dictionary defining the action to open the late check-in
-        records window.
+            records window.
-        :rtype: dict
+            :rtype: dict
         """
         return {
-            "name": _("Employee Late Check-in"),
+            'name': _('Employee Late Check-in'),
-            "domain": [("employee_id", "=", self.id)],
+            'domain': [('employee_id', '=', self.id)],
-            "res_model": "late.check.in",
+            'res_model': 'late.check.in',
-            "type": "ir.actions.act_window",
+            'type': 'ir.actions.act_window',
-            "view_mode": "tree,form",
+            'view_mode': 'tree,form',
-            "limit": 80,
+            'limit': 80}
-        }
 
     def _compute_late_check_in_count(self):
         """Compute the late check-in count"""
         for rec in self:
-            rec.late_check_in_count = self.env["late.check.in"].search_count(
+            rec.late_check_in_count = self.env['late.check.in'].search_count(
-                [("employee_id", "=", rec.id)]
+                [('employee_id', '=', rec.id)])
-            )

employee_late_check_in/models/hr_employees_public.py

@@ -1,56 +1,52 @@
 # -*- coding: utf-8 -*-
-###############################################################################
+#############################################################################
 #
 #    Cybrosys Technologies Pvt. Ltd.
 #
 #    Copyright (C) 2024-TODAY Cybrosys Technologies(<https://www.cybrosys.com>)
-#    Author: Cybrosys Techno Solutions (odoo@cybrosys.com)
+#    Author: Cybrosys Techno Solutions(<https://www.cybrosys.com>)
 #
-#    You can modify it under the terms of the GNU AFFERO
+#    You can modify it under the terms of the GNU LESSER
-#    GENERAL PUBLIC LICENSE (AGPL v3), Version 3.
+#    GENERAL PUBLIC LICENSE (LGPL v3), Version 3.
 #
 #    This program is distributed in the hope that it will be useful,
 #    but WITHOUT ANY WARRANTY; without even the implied warranty of
 #    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-#    GNU AFFERO GENERAL PUBLIC LICENSE (AGPL v3) for more details.
+#    GNU LESSER GENERAL PUBLIC LICENSE (LGPL v3) for more details.
 #
-#    You should have received a copy of the GNU AFFERO GENERAL PUBLIC LICENSE
+#    You should have received a copy of the GNU LESSER GENERAL PUBLIC LICENSE
-#    (AGPL v3) along with this program.
+#    (LGPL v3) along with this program.
 #    If not, see <http://www.gnu.org/licenses/>.
 #
-###############################################################################
+############################################################################.
-from odoo import fields, models, _
+from odoo import _, fields, models
 
 
 class HrEmployees(models.Model):
     """Inherit the model to add fields and functions"""
-
+    _inherit = 'hr.employee.public'
-    _inherit = "hr.employee.public"
 
     late_check_in_count = fields.Integer(
-        string="Late Check-In",
+        string="Late Check-In", compute="_compute_late_check_in_count",
-        compute="_compute_late_check_in_count",
+        help="Count of employee's late checkin")
-        help="Count of employee's late checkin",
-    )
 
     def action_to_open_late_check_in_records(self):
         """
-        :return: dictionary defining the action to open the late check-in
+            :return: dictionary defining the action to open the late check-in
-        records window.
+            records window.
-        :rtype: dict
+            :rtype: dict
         """
         return {
-            "name": _("Employee Late Check-in"),
+            'name': _('Employee Late Check-in'),
-            "domain": [("employee_id", "=", self.id)],
+            'domain': [('employee_id', '=', self.id)],
-            "res_model": "late.check.in",
+            'res_model': 'late.check.in',
-            "type": "ir.actions.act_window",
+            'type': 'ir.actions.act_window',
-            "view_mode": "tree,form",
+            'view_mode': 'tree,form',
-            "limit": 80,
+            'limit': 80,
         }
 
     def _compute_late_check_in_count(self):
         """Compute the late check-in count"""
         for rec in self:
-            rec.late_check_in_count = self.env["late.check.in"].search_count(
+            rec.late_check_in_count = self.env['late.check.in'].search_count(
-                [("employee_id", "=", rec.id)]
+                [('employee_id', '=', rec.id)])
-            )

employee_late_check_in/models/hr_payslip.py

@@ -1,59 +1,54 @@
 # -*- coding: utf-8 -*-
-###############################################################################
+#############################################################################
 #
 #    Cybrosys Technologies Pvt. Ltd.
 #
 #    Copyright (C) 2024-TODAY Cybrosys Technologies(<https://www.cybrosys.com>)
-#    Author: Cybrosys Techno Solutions (odoo@cybrosys.com)
+#    Author: Cybrosys Techno Solutions(<https://www.cybrosys.com>)
 #
-#    You can modify it under the terms of the GNU AFFERO
+#    You can modify it under the terms of the GNU LESSER
-#    GENERAL PUBLIC LICENSE (AGPL v3), Version 3.
+#    GENERAL PUBLIC LICENSE (LGPL v3), Version 3.
 #
 #    This program is distributed in the hope that it will be useful,
 #    but WITHOUT ANY WARRANTY; without even the implied warranty of
 #    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-#    GNU AFFERO GENERAL PUBLIC LICENSE (AGPL v3) for more details.
+#    GNU LESSER GENERAL PUBLIC LICENSE (LGPL v3) for more details.
 #
-#    You should have received a copy of the GNU AFFERO GENERAL PUBLIC LICENSE
+#    You should have received a copy of the GNU LESSER GENERAL PUBLIC LICENSE
-#    (AGPL v3) along with this program.
+#    (LGPL v3) along with this program.
 #    If not, see <http://www.gnu.org/licenses/>.
 #
-###############################################################################
+############################################################################.
 from odoo import api, fields, models
 
 
 class PayslipLateCheckIn(models.Model):
     """Inherit the model to add fields and functions"""
-
+    _inherit = 'hr.payslip'
-    _inherit = "hr.payslip"
 
     late_check_in_ids = fields.Many2many(
-        "late.check.in",
+        'late.check.in', string='Late Check-in',
-        string="Late Check-in",
+        help='Late check-in records of the employee')
-        help="Late check-in records of the employee",
-    )
 
     @api.model
     def get_inputs(self, contracts, date_from, date_to):
         """Function used for writing late check-in record in the payslip input
-        tree."""
+         tree."""
-        res = super(PayslipLateCheckIn, self).get_inputs(contracts, date_to, date_from)
+        res = super(PayslipLateCheckIn, self).get_inputs(contracts, date_to,
-        late_check_in_type = self.env.ref("employee_late_check_in.late_check_in")
+                                                         date_from)
-        late_check_in_id = self.env["late.check.in"].search(
+        late_check_in_type = self.env.ref(
-            [
+            'employee_late_check_in.late_check_in')
-                ("employee_id", "=", self.employee_id.id),
+        late_check_in_id = self.env['late.check.in'].search(
-                ("date", "<=", self.date_to),
+            [('employee_id', '=', self.employee_id.id),
-                ("date", ">=", self.date_from),
+             ('date', '<=', self.date_to), ('date', '>=', self.date_from),
-                ("state", "=", "approved"),
+             ('state', '=', 'approved')])
-            ]
-        )
         if late_check_in_id:
             self.late_check_in_ids = late_check_in_id
             input_data = {
-                "name": late_check_in_type.name,
+                'name': late_check_in_type.name,
-                "code": late_check_in_type.code,
+                'code': late_check_in_type.code,
-                "amount": sum(late_check_in_id.mapped("penalty_amount")),
+                'amount': sum(late_check_in_id.mapped('penalty_amount')),
-                "contract_id": self.contract_id.id,
+                'contract_id': self.contract_id.id,
             }
             res.append(input_data)
         return res
@@ -61,5 +56,5 @@ class PayslipLateCheckIn(models.Model):
     def action_payslip_done(self):
         """Function used for marking deducted Late check-in request."""
         for rec in self.late_check_in_ids:
-            rec.state = "deducted"
+            rec.state = 'deducted'
         return super(PayslipLateCheckIn, self).action_payslip_done()

employee_late_check_in/models/late_check_in.py

@@ -1,87 +1,73 @@
 # -*- coding: utf-8 -*-
-###############################################################################
+#############################################################################
 #
 #    Cybrosys Technologies Pvt. Ltd.
 #
 #    Copyright (C) 2024-TODAY Cybrosys Technologies(<https://www.cybrosys.com>)
-#    Author: Cybrosys Techno Solutions (odoo@cybrosys.com)
+#    Author: Cybrosys Techno Solutions(<https://www.cybrosys.com>)
 #
-#    You can modify it under the terms of the GNU AFFERO
+#    You can modify it under the terms of the GNU LESSER
-#    GENERAL PUBLIC LICENSE (AGPL v3), Version 3.
+#    GENERAL PUBLIC LICENSE (LGPL v3), Version 3.
 #
 #    This program is distributed in the hope that it will be useful,
 #    but WITHOUT ANY WARRANTY; without even the implied warranty of
 #    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-#    GNU AFFERO GENERAL PUBLIC LICENSE (AGPL v3) for more details.
+#    GNU LESSER GENERAL PUBLIC LICENSE (LGPL v3) for more details.
 #
-#    You should have received a copy of the GNU AFFERO GENERAL PUBLIC LICENSE
+#    You should have received a copy of the GNU LESSER GENERAL PUBLIC LICENSE
-#    (AGPL v3) along with this program.
+#    (LGPL v3) along with this program.
 #    If not, see <http://www.gnu.org/licenses/>.
 #
-###############################################################################
+############################################################################.
 from odoo import api, fields, models
 
 
 class LateCheckIn(models.Model):
     """Model to store late check-in records"""
-
+    _name = 'late.check.in'
-    _name = "late.check.in"
+    _description = 'Late Check In'
-    _description = "Late Check In"
 
     name = fields.Char(
-        readonly=True, string="Name", help="Reference number of the record"
+        readonly=True, string='Name', help="Reference number of the record")
-    )
+    employee_id = fields.Many2one('hr.employee', string="Employee",
-    employee_id = fields.Many2one(
+                                  help='Late employee')
-        "hr.employee", string="Employee", help="Late employee"
+    late_minutes = fields.Integer(string="Late Minutes",
-    )
+                                  help='The field indicates the number of '
-    late_minutes = fields.Integer(
+                                       'minutes the worker is late.')
-        string="Late Minutes",
+    date = fields.Date(string="Date", help='Current date')
-        help="The field indicates the number of minutes the worker is late.",
+    penalty_amount = fields.Float(compute="_compute_penalty_amount",
-    )
+                                  help='Amount needs to be deducted',
-    date = fields.Date(string="Date", help="Current date")
+                                  string="Amount",)
-    penalty_amount = fields.Float(
+    state = fields.Selection(selection=[('draft', 'Draft'),
-        compute="_compute_penalty_amount",
+                                        ('approved', 'Approved'),
-        help="Amount needs to be deducted",
+                                        ('refused', 'Refused'),
-        string="Amount",
+                                        ('deducted', 'Deducted')],
-    )
+                             string="State", default="draft",
-    state = fields.Selection(
+                             help='State of the record')
-        selection=[
+    attendance_id = fields.Many2one('hr.attendance', string='Attendance',
-            ("draft", "Draft"),
+                                    help='Attendance of the employee')
-            ("approved", "Approved"),
-            ("refused", "Refused"),
-            ("deducted", "Deducted"),
-        ],
-        string="State",
-        default="draft",
-        help="State of the record",
-    )
-    attendance_id = fields.Many2one(
-        "hr.attendance", string="Attendance", help="Attendance of the employee"
-    )
 
     @api.model
     def create(self, vals_list):
         """Create a sequence for the model"""
-        vals_list["name"] = self.env["ir.sequence"].next_by_code("late.check.in") or "/"
+        vals_list['name'] = self.env['ir.sequence'].next_by_code(
+            'late.check.in') or '/'
         return super(LateCheckIn, self.sudo()).create(vals_list)
 
     def _compute_penalty_amount(self):
         """Compute the penalty amount if the employee was late"""
         for rec in self:
-            amount = float(
+            amount = float(self.env['ir.config_parameter'].sudo().get_param(
-                self.env["ir.config_parameter"].sudo().get_param("employee_late_check_in.deduction_amount")
+                'employee_late_check_in.deduction_amount'))
-            )
             rec.penalty_amount = amount
-            if (
+            if self.env['ir.config_parameter'].sudo().get_param(
-                self.env["ir.config_parameter"].sudo().get_param("employee_late_check_in.deduction_type")
+                    'employee_late_check_in.deduction_type') == 'minutes':
-                == "minutes"
-            ):
                 rec.penalty_amount = amount * rec.late_minutes
 
     def approve(self):
         """Change state to approved when approve button clicks"""
-        self.state = "approved"
+        self.state = 'approved'
 
     def reject(self):
         """Change state refused when refuse button clicks"""
-        self.state = "refused"
+        self.state = 'refused'

employee_late_check_in/models/res_company.py

@@ -1,52 +1,45 @@
 # -*- coding: utf-8 -*-
-###############################################################################
+#############################################################################
 #
 #    Cybrosys Technologies Pvt. Ltd.
 #
 #    Copyright (C) 2024-TODAY Cybrosys Technologies(<https://www.cybrosys.com>)
-#    Author: Cybrosys Techno Solutions (odoo@cybrosys.com)
+#    Author: Cybrosys Techno Solutions(<https://www.cybrosys.com>)
 #
-#    You can modify it under the terms of the GNU AFFERO
+#    You can modify it under the terms of the GNU LESSER
-#    GENERAL PUBLIC LICENSE (AGPL v3), Version 3.
+#    GENERAL PUBLIC LICENSE (LGPL v3), Version 3.
 #
 #    This program is distributed in the hope that it will be useful,
 #    but WITHOUT ANY WARRANTY; without even the implied warranty of
 #    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-#    GNU AFFERO GENERAL PUBLIC LICENSE (AGPL v3) for more details.
+#    GNU LESSER GENERAL PUBLIC LICENSE (LGPL v3) for more details.
 #
-#    You should have received a copy of the GNU AFFERO GENERAL PUBLIC LICENSE
+#    You should have received a copy of the GNU LESSER GENERAL PUBLIC LICENSE
-#    (AGPL v3) along with this program.
+#    (LGPL v3) along with this program.
 #    If not, see <http://www.gnu.org/licenses/>.
 #
-###############################################################################
+############################################################################.
 from odoo import fields, models
 
 
 class ResCompany(models.Model):
     """Inherit the model to add fields"""
-
+    _inherit = 'res.company'
-    _inherit = "res.company"
 
     deduction_amount = fields.Float(
-        help="How much amount need to be deducted if a employee was late",
+        help='How much amount need to be deducted if a employee was late',
-        string="Deduction Amount",
+        string="Deduction Amount",)
-    )
     currency_id = fields.Many2one(
-        "res.currency", default=lambda self: self.env.company.currency_id.id
+        'res.currency', default=lambda self: self.env.company.currency_id.id)
-    )
     maximum_minutes = fields.Char(
         help="Maximum time limit a employee was considered as late",
-        string="Maximum Late Minute",
+        string="Maximum Late Minute")
-    )
     late_check_in_after = fields.Char(
-        help="When should the late check-in count down starts.",
+        help='When should the late check-in count down starts.',
-        string="Late Check-in Starts After",
+        string="Late Check-in Starts After",)
-    )
     deduction_type = fields.Selection(
-        selection=[("minutes", "Per Minutes"), ("total", "Per Total")],
+        selection=[('minutes', 'Per Minutes'), ('total', 'Per Total')],
-        default="minutes",
+        default="minutes", string='Deduction Type',
-        string="Deduction Type",
+        help='Type of deduction, (If Per Minutes is chosen then for each '
-        help="Type of deduction, (If Per Minutes is chosen then for each "
+             'minutes given amount is deducted, if Per Total is chosen then '
-        "minutes given amount is deducted, if Per Total is chosen then "
+             'given amount is deducted from the total salary)')
-        "given amount is deducted from the total salary)",
-    )

employee_late_check_in/models/res_config_settings.py

@@ -1,56 +1,49 @@
 # -*- coding: utf-8 -*-
-###############################################################################
+#############################################################################
 #
 #    Cybrosys Technologies Pvt. Ltd.
 #
 #    Copyright (C) 2024-TODAY Cybrosys Technologies(<https://www.cybrosys.com>)
-#    Author: Cybrosys Techno Solutions (odoo@cybrosys.com)
+#    Author: Cybrosys Techno Solutions(<https://www.cybrosys.com>)
 #
-#    You can modify it under the terms of the GNU AFFERO
+#    You can modify it under the terms of the GNU LESSER
-#    GENERAL PUBLIC LICENSE (AGPL v3), Version 3.
+#    GENERAL PUBLIC LICENSE (LGPL v3), Version 3.
 #
 #    This program is distributed in the hope that it will be useful,
 #    but WITHOUT ANY WARRANTY; without even the implied warranty of
 #    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-#    GNU AFFERO GENERAL PUBLIC LICENSE (AGPL v3) for more details.
+#    GNU LESSER GENERAL PUBLIC LICENSE (LGPL v3) for more details.
 #
-#    You should have received a copy of the GNU AFFERO GENERAL PUBLIC LICENSE
+#    You should have received a copy of the GNU LESSER GENERAL PUBLIC LICENSE
-#    (AGPL v3) along with this program.
+#    (LGPL v3) along with this program.
 #    If not, see <http://www.gnu.org/licenses/>.
 #
-###############################################################################
+############################################################################.
 from odoo import fields, models
 
 
 class LateCheckinSettings(models.TransientModel):
     """Inherit the model to add fields"""
-
+    _inherit = 'res.config.settings'
-    _inherit = "res.config.settings"
 
     deduction_amount = fields.Float(
-        config_parameter="employee_late_check_in.deduction_amount",
+        config_parameter='employee_late_check_in.deduction_amount',
-        help="How much amount need to be deducted if a employee was late",
+        help='How much amount need to be deducted if a employee was late',
-        string="Deduction Amount",
+        string="Deduction Amount",)
-    )
     maximum_minutes = fields.Char(
-        config_parameter="employee_late_check_in.maximum_minutes",
+        config_parameter='employee_late_check_in.maximum_minutes',
         help="Maximum time limit a employee was considered as late",
-        string="Maximum Late Minute",
+        string="Maximum Late Minute")
-    )
     late_check_in_after = fields.Char(
-        config_parameter="employee_late_check_in.late_check_in_after",
+        config_parameter='employee_late_check_in.late_check_in_after',
-        help="When should the late check-in count down starts.",
+        help='When should the late check-in count down starts.',
-        string="Late Check-in Starts After",
+        string="Late Check-in Starts After",)
-    )
     deduction_type = fields.Selection(
-        selection=[("minutes", "Per Minutes"), ("total", "Per Total")],
+        selection=[('minutes', 'Per Minutes'), ('total', 'Per Total')],
-        config_parameter="employee_late_check_in.deduction_type",
+        config_parameter='employee_late_check_in.deduction_type',
-        default="minutes",
+        default="minutes", string='Deduction Type',
-        string="Deduction Type",
+        help='Type of deduction, (If Per Minutes is chosen then for each '
-        help="Type of deduction, (If Per Minutes is chosen then for each "
+             'minutes given amount is deducted, if Per Total is chosen then '
-        "minutes given amount is deducted, if Per Total is chosen then "
+             'given amount is deducted from the total salary)')
-        "given amount is deducted from the total salary)",
-    )
     currency_id = fields.Many2one(
-        "res.currency", default=lambda self: self.env.company.currency_id.id
+        'res.currency', default=lambda self: self.env.company.currency_id.id)
-    )

employee_late_check_in/views/late_check_in_views.xml

@@ -74,7 +74,6 @@
                 <field name="late_minutes"/>
                 <field name="date"/>
                 <field name="penalty_amount"/>
-                <field name="state" widget="badge" decoration-info="state == 'draft'" decoration-success="state == 'approved'"/>
             </tree>
         </field>
     </record>

employee_late_check_in/views/res_config_settings_views.xml

@@ -1,20 +1,27 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
-<!--    Adding fields in res.config.settings-->
+    Adding fields in res.config.settings
     <record id="res_config_settings_view_form" model="ir.ui.view">
-        <field name="name">res.config.settings.view.form.inherit.employee.late.check.in</field>
+        <field name="name">
+            res.config.settings.view.form.inherit.employee.late.check.in
+        </field>
         <field name="model">res.config.settings</field>
         <field name="inherit_id"
                ref="hr_attendance.res_config_settings_view_form"/>
         <field name="arch" type="xml">
-            <xpath expr="//div[hasclass('app_settings_block')][@data-key='hr_attendance']" position="inside">
+            <xpath expr="//div[hasclass('app_settings_block')][@data-key='hr_attendance']"
+                   position="inside">
                 <h2>Late Check-in</h2>
                 <div class="row mt16 o_settings_container" name="late_check_in">
                     <div class="col-12 col-lg-6 o_setting_box">
                         <div class="o_setting_right_pane">
-                            <label for="deduction_amount" class="o_form_label">Deduction Amount</label>
+                            <label for="deduction_amount" class="o_form_label">
-                            <span class="fa fa-lg fa-building-o" title="Values set here are company-specific."
+                                Deduction Amount
-                                  role="img" aria-label="Values set here are company-specific."
+                            </label>
+                            <span class="fa fa-lg fa-building-o"
+                                  title="Values set here are company-specific."
+                                  role="img"
+                                  aria-label="Values set here are company-specific."
                                   groups="base.group_multi_company"/>
                             <div class="text-muted">
                                 Amount to be deducted from payslip.
@@ -25,27 +32,41 @@
                             </div>
                             <div class="mt16">
                                 <div class="mt16 row">
-                                    <field name="deduction_amount" widget="monetary"/>
+                                    <field name="deduction_amount"
+                                           widget="monetary"/>
                                     <field name="deduction_type"/>
                                 </div>
                                 <br/>
-                                <label for="late_check_in_after" class="o_form_label">
+                                <label for="late_check_in_after"
+                                       class="o_form_label">
                                     Late Check In Starts After
                                 </label>
                                 <div class="text-muted">
-                                    When should the late check-in count down starts.
+                                    When should the late check-in count down
+                                    starts.
                                 </div>
                                 <field name="late_check_in_after"/>
                                 <span>Minutes</span>
                                 <br/>
                                 <br/>
-                                <label for="maximum_minutes" class="o_form_label">
+                                <label for="maximum_minutes"
+                                       class="o_form_label">
                                     Maximum Late Minutes
                                 </label>
                                 <div class="text-muted">
-                                    Maximum time limit a employee was considered as late.
+                                    Specifies the maximum threshold for
+                                    recording late check-ins.
+                                    (Enter the number of minutes after which a
+                                    check-in is treated as an absence or another
+                                    type of leave,etc.
+                                    For example, if set to 100, a check-in that
+                                    is 99 minutes late will be recorded as a
+                                    late check-in, but a check-in that is 101
+                                    minutes late will not be recorded as a late
+                                    check-in.)
                                 </div>
-                               <field name="maximum_minutes"/><span>Minutes</span>
+                                <field name="maximum_minutes"/>
+                                <span>Minutes</span>
                             </div>
                         </div>
                     </div>