Skip to content
Snippets Groups Projects
Commit 43bf5a5f authored by Thibault Delavallée's avatar Thibault Delavallée
Browse files

[FIX] calendar_sms, hr_presence: fix broken acls update on sms.template


odoo/odoo#64626 attempt at improving sms.template rules seems quite broken

  * do not restrict system group to calendar.event: this makes no sense.
    With this fix admins can now only use templates being linked to calendar
    event if calendar_sms is installed which is not really smart. Not sure
    what was the exact original purpose;
  * ir.rule in hr_presence has no use on hr.manager group if they do not
    have any access through access rules. Indeed they are currently
    restricted to read access as all standard internal users;

Task ID-2191254
COM PR odoo/odoo#68445
ENT PR odoo/enterprise#17340

closes odoo/odoo#68465

X-original-commit: 7c4c20a6
Related: odoo/enterprise#17348
Signed-off-by: default avatarThibault Delavallee (tde) <tde@openerp.com>
parent 157eee07
No related branches found
No related tags found
No related merge requests found
<?xml version="1.0" encoding="utf-8"?>
<odoo>
<record id="ir_rule_sms_template_system" model="ir.rule">
<field name="name">SMS Template: system administrator CUD on calendar event templates</field>
<field name="name">SMS Template: unnecessary rule for system group</field>
<field name="model_id" ref="sms.model_sms_template"/>
<field name="groups" eval="[(4, ref('base.group_system'))]"/>
<field name="domain_force">[('model_id.model', '=', 'calendar.event')]</field>
<!-- TDE NOTE: remove me in master, broken fix from odoo/odoo#64626 -->
<field name="domain_force">[(1, '=', 1)]</field>
<field name="perm_read" eval="False"/>
</record>
</odoo>
......@@ -19,6 +19,7 @@ Allows to contact directly the employee in case of unjustified absence.
'depends': ['hr', 'hr_holidays', 'sms'],
'data': [
'security/sms_security.xml',
'security/ir.model.access.csv',
'data/ir_actions_server.xml',
'views/hr_employee_views.xml',
'data/sms_data.xml',
......
id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
access_sms_template_hr_manager,access.sms.template.hr.manager,sms.model_sms_template,hr.group_hr_manager,1,1,1,1
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment