Skip to content

Redundant default on warnings on models implementing resource_mixin #311

@robinkeunen

Description

@robinkeunen

class ResourceMixin(models.AbstractModel):
_inherit = "resource.mixin"
# make this field read-only.
resource_calendar_id = fields.Many2one("resource.calendar", readonly=True)

This line has the side effect of issuing a warning for all models that inherit this mixin and set defaults to resource_calendar_id

2024-01-26 15:09:10,166 69244 WARNING sixteen-odoo odoo.fields: Redundant default on resource.resource.calendar_id 
2024-01-26 15:09:10,170 69244 WARNING sixteen-odoo odoo.fields: Redundant default on hr.employee.resource_calendar_id 
2024-01-26 15:09:10,175 69244 WARNING sixteen-odoo odoo.fields: Redundant default on resource.mixin.resource_calendar_id 
2024-01-26 15:09:10,362 69244 INFO sixteen-odoo odoo.modules.loading: loading spreadsheet_dashboard_hr_timesheet/data/dashboards.xml 
2024-01-26 15:09:10,555 69244 INFO sixteen-odoo odoo.modules.loading: Module spreadsheet_dashboard_hr_timesheet loaded in 0.52s, 26 queries (+26 other) 
2024-01-26 15:09:10,555 69244 INFO sixteen-odoo odoo.modules.loading: Loading module sale_management (54/62) 
2024-01-26 15:09:10,726 69244 WARNING sixteen-odoo odoo.fields: Redundant default on resource.resource.calendar_id 
2024-01-26 15:09:10,730 69244 WARNING sixteen-odoo odoo.fields: Redundant default on hr.employee.resource_calendar_id 
2024-01-26 15:09:10,736 69244 WARNING sixteen-odoo odoo.fields: Redundant default on resource.mixin.resource_calendar_id 
2024-01-26 15:09:11,037 69244 WARNING sixteen-odoo odoo.fields: Redundant default on resource.resource.calendar_id 
2024-01-26 15:09:11,038 69244 WARNING sixteen-odoo odoo.fields: Redundant default on hr.employee.resource_calendar_id 
2024-01-26 15:09:11,102 69244 WARNING sixteen-odoo odoo.fields: Redundant default on resource.mixin.resource_calendar_id 
... and more

More information on the warning :

  # A readonly related field without an inverse method should not have a
  # default value, as it does not make sense.
  if self.default and self.readonly and not self.inverse:
      _logger.warning("Redundant default on %s", self)

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions