[FIX] Bug fixed 'employee_vehicle_request'

employee_vehicle_request/__manifest__.py

@@ -22,7 +22,7 @@
 #############################################################################
 {
     'name': 'Employee Vehicle Request',
-    'version': '12.0.1.0.0',
+    'version': '13.0.1.0.1',
     'summary': """Manage Vehicle Requests From Employee""",
     'description': """This module is used for manage vehicle requests from employee.
                    This module also checking the vehicle availability at the requested time slot.""",

employee_vehicle_request/doc/RELEASE_NOTES.md

@@ -3,3 +3,7 @@
 #### 12.11.2019
 #### Version 13.0.1.0.0
 #### ADD
+
+#### 24.03.2021
+#### Version 13.0.1.0.1
+#### FIX

employee_vehicle_request/models/employee_fleet.py

@@ -62,17 +62,18 @@ class EmployeeFleet(models.Model):
             check_availability = 0
             for i in fleet_obj:
                 for each in i.reserved_time:
-                    if each.date_from <= self.date_from <= each.date_to:
-                        check_availability = 1
-                    elif self.date_from < each.date_from:
-                        if each.date_from <= self.date_to <= each.date_to:
-                            check_availability = 1
-                        elif self.date_to > each.date_to:
+                    if each.date_from and each.date_to:
+                        if each.date_from <= self.date_from <= each.date_to:
                             check_availability = 1
+                        elif self.date_from < each.date_from:
+                            if each.date_from <= self.date_to <= each.date_to:
+                                check_availability = 1
+                            elif self.date_to > each.date_to:
+                                check_availability = 1
+                            else:
+                                check_availability = 0
                         else:
                             check_availability = 0
-                    else:
-                        check_availability = 0
             if check_availability == 0:
                 reserved_id = self.fleet.reserved_time.create({'employee': self.employee.id,
                                                                'date_from': self.date_from,
@@ -100,7 +101,7 @@ class EmployeeFleet(models.Model):
         mail_id.mail_message_id.body = mail_content
         mail_id.send()
         if self.employee.user_id:
-            mail_id.mail_message_id.write({'needaction_partner_ids': [(4, self.employee.user_id.partner_id.id)]})
+            # mail_id.mail_message_id.write({'needaction_partner_ids': [(4, self.employee.user_id.partner_id.id)]})
             mail_id.mail_message_id.write({'partner_ids': [(4, self.employee.user_id.partner_id.id)]})
 
     # @api.multi
@@ -120,7 +121,7 @@ class EmployeeFleet(models.Model):
         mail_id.mail_message_id.body = mail_content
         mail_id.send()
         if self.employee.user_id:
-            mail_id.mail_message_id.write({'needaction_partner_ids': [(4, self.employee.user_id.partner_id.id)]})
+            # mail_id.mail_message_id.write({'needaction_partner_ids': [(4, self.employee.user_id.partner_id.id)]})
             mail_id.mail_message_id.write({'partner_ids': [(4, self.employee.user_id.partner_id.id)]})
 
     # @api.multi
@@ -135,7 +136,7 @@ class EmployeeFleet(models.Model):
         self.returned_date = fields.datetime.now()
         self.state = 'return'
 
-    @api.constrains('date_rom', 'date_to')
+    @api.constrains('date_from', 'date_to')
     def onchange_date_to(self):
         for each in self:
             if each.date_from > each.date_to:
@@ -148,17 +149,18 @@ class EmployeeFleet(models.Model):
             fleet_obj = self.env['fleet.vehicle'].search([])
             for i in fleet_obj:
                 for each in i.reserved_time:
-                    if each.date_from <= self.date_from <= each.date_to:
-                        i.write({'check_availability': False})
-                    elif self.date_from < each.date_from:
-                        if each.date_from <= self.date_to <= each.date_to:
-                            i.write({'check_availability': False})
-                        elif self.date_to > each.date_to:
+                    if each.date_from and each.date_to:
+                        if each.date_from <= self.date_from <= each.date_to:
                             i.write({'check_availability': False})
+                        elif self.date_from < each.date_from:
+                            if each.date_from <= self.date_to <= each.date_to:
+                                i.write({'check_availability': False})
+                            elif self.date_to > each.date_to:
+                                i.write({'check_availability': False})
+                            else:
+                                i.write({'check_availability': True})
                         else:
                             i.write({'check_availability': True})
-                    else:
-                        i.write({'check_availability': True})
 
     reserved_fleet_id = fields.Many2one('fleet.reserved', invisible=1, copy=False)
     name = fields.Char(string='Request Number', copy=False)