Compare commits
3 Commits
f3aa161807
...
7a0cf23df9
| Author | SHA1 | Date | |
|---|---|---|---|
| 7a0cf23df9 | |||
| 4eee218534 | |||
| 88494104cc |
37
.gitignore
vendored
Normal file
37
.gitignore
vendored
Normal file
@ -0,0 +1,37 @@
|
|||||||
|
# Python
|
||||||
|
__pycache__/
|
||||||
|
*.py[cod]
|
||||||
|
*$py.class
|
||||||
|
|
||||||
|
# Distribution / packaging
|
||||||
|
.Python
|
||||||
|
build/
|
||||||
|
develop-eggs/
|
||||||
|
dist/
|
||||||
|
downloads/
|
||||||
|
eggs/
|
||||||
|
.eggs/
|
||||||
|
lib/
|
||||||
|
lib64/
|
||||||
|
parts/
|
||||||
|
sdist/
|
||||||
|
var/
|
||||||
|
wheels/
|
||||||
|
*.egg-info/
|
||||||
|
.installed.cfg
|
||||||
|
*.egg
|
||||||
|
|
||||||
|
# Odoo
|
||||||
|
*.log
|
||||||
|
.pylintrc
|
||||||
|
.pylint.d/
|
||||||
|
|
||||||
|
# VS Code
|
||||||
|
.vscode/
|
||||||
|
|
||||||
|
# PyCharm
|
||||||
|
.idea/
|
||||||
|
|
||||||
|
# Translations
|
||||||
|
*.pot
|
||||||
|
*.po
|
||||||
11
README.md
11
README.md
@ -10,6 +10,7 @@ Restrict visibility of the following records based on User configuration:
|
|||||||
* **Locations** (`stock.location`)
|
* **Locations** (`stock.location`)
|
||||||
* **Work Centers** (`mrp.workcenter`)
|
* **Work Centers** (`mrp.workcenter`)
|
||||||
* **Approval Categories** (`approval.category`)
|
* **Approval Categories** (`approval.category`)
|
||||||
|
* **Quality Checks Button** (on Manufacturing Orders and Inventory Transfers)
|
||||||
|
|
||||||
## Configuration
|
## Configuration
|
||||||
|
|
||||||
@ -29,6 +30,16 @@ Restrict visibility of the following records based on User configuration:
|
|||||||
* **Populated List = Restricted**: If one or more records are added, the user will **ONLY** see those specific records.
|
* **Populated List = Restricted**: If one or more records are added, the user will **ONLY** see those specific records.
|
||||||
* **Superuser**: The Superuser (OdooBot) and administrators bypassing access rights are not affected by these restrictions.
|
* **Superuser**: The Superuser (OdooBot) and administrators bypassing access rights are not affected by these restrictions.
|
||||||
|
|
||||||
|
## Quality Checks Button Restriction
|
||||||
|
|
||||||
|
The visibility of the "Quality Checks" and "Quality Alert" buttons on **Manufacturing Orders** and **Inventory Transfers** is controlled by a specific user setting.
|
||||||
|
|
||||||
|
To allow a user to see these buttons:
|
||||||
|
1. Navigate to **Settings > Users & Companies > Users**.
|
||||||
|
2. Enable the checkbox **"Is Allowed todo Quality Checks?"** in the **Access Restrictions** tab.
|
||||||
|
|
||||||
|
By default, this setting is unchecked, meaning users will **NOT** see these buttons unless explicitly allowed.
|
||||||
|
|
||||||
## Technical Details
|
## Technical Details
|
||||||
|
|
||||||
This module overrides the `_search` method on the target models to apply a domain filter based on the current user's allowed list. This ensures consistency across views (list, kanban, many2one dropdowns) and avoids common issues associated with Record Rules.
|
This module overrides the `_search` method on the target models to apply a domain filter based on the current user's allowed list. This ensures consistency across views (list, kanban, many2one dropdowns) and avoids common issues associated with Record Rules.
|
||||||
|
|||||||
@ -16,13 +16,12 @@
|
|||||||
""",
|
""",
|
||||||
'category': 'Extra Tools',
|
'category': 'Extra Tools',
|
||||||
'author': 'Suherdy Yacob',
|
'author': 'Suherdy Yacob',
|
||||||
'depends': ['base', 'stock', 'mrp', 'approvals', 'stock_account', 'sale', 'quality_control', 'quality_mrp'],
|
'depends': ['base', 'stock', 'mrp', 'approvals', 'stock_account', 'sale', 'quality_mrp'],
|
||||||
'data': [
|
'data': [
|
||||||
'security/ir.model.access.csv',
|
'security/ir.model.access.csv',
|
||||||
'security/ir_rule.xml',
|
'security/ir_rule.xml',
|
||||||
'security/ir_actions_act_window.xml',
|
'security/ir_actions_act_window.xml',
|
||||||
'views/res_users_views.xml',
|
'views/res_users_views.xml',
|
||||||
'views/stock_picking_views.xml',
|
|
||||||
'views/mrp_production_views.xml',
|
'views/mrp_production_views.xml',
|
||||||
],
|
],
|
||||||
'installable': True,
|
'installable': True,
|
||||||
|
|||||||
@ -1,6 +1,10 @@
|
|||||||
from . import res_users
|
from . import res_users
|
||||||
from . import restricted_models
|
from . import restricted_models
|
||||||
from . import sale_order
|
from . import sale_order
|
||||||
|
<<<<<<< HEAD
|
||||||
from . import stock_picking
|
from . import stock_picking
|
||||||
|
=======
|
||||||
|
>>>>>>> 3a96e90 (feat: Hide quality check buttons on manufacturing orders for specific user groups with exceptions for quality roles.)
|
||||||
from . import mrp_production
|
from . import mrp_production
|
||||||
|
from . import stock_picking
|
||||||
|
|
||||||
|
|||||||
Binary file not shown.
@ -3,6 +3,7 @@ from odoo import models, fields, api
|
|||||||
class MrpProduction(models.Model):
|
class MrpProduction(models.Model):
|
||||||
_inherit = 'mrp.production'
|
_inherit = 'mrp.production'
|
||||||
|
|
||||||
|
<<<<<<< HEAD
|
||||||
restrict_quality_check_button = fields.Boolean(compute='_compute_restrict_quality_check_button')
|
restrict_quality_check_button = fields.Boolean(compute='_compute_restrict_quality_check_button')
|
||||||
|
|
||||||
@api.depends_context('uid')
|
@api.depends_context('uid')
|
||||||
@ -22,3 +23,71 @@ class MrpProduction(models.Model):
|
|||||||
production.restrict_quality_check_button = True
|
production.restrict_quality_check_button = True
|
||||||
else:
|
else:
|
||||||
production.restrict_quality_check_button = False
|
production.restrict_quality_check_button = False
|
||||||
|
=======
|
||||||
|
hide_quality_check_button = fields.Boolean(compute='_compute_hide_quality_check_button')
|
||||||
|
|
||||||
|
@api.depends('product_id')
|
||||||
|
@api.depends_context('uid')
|
||||||
|
def _compute_hide_quality_check_button(self):
|
||||||
|
for record in self:
|
||||||
|
<<<<<<< HEAD
|
||||||
|
user = self.env.user
|
||||||
|
|
||||||
|
# Define the restricted groups
|
||||||
|
# 1. Inventory User (stock.group_stock_user)
|
||||||
|
# 2. Manufacturing User (mrp.group_mrp_user)
|
||||||
|
# 3. MPS User (mrp_mps.group_mrp_mps) - handled safely
|
||||||
|
|
||||||
|
is_restricted = False
|
||||||
|
|
||||||
|
if user.has_group('stock.group_stock_user') or user.has_group('mrp.group_mrp_user'):
|
||||||
|
is_restricted = True
|
||||||
|
|
||||||
|
# Check MPS User group safely as it might not be installed or ID might differ
|
||||||
|
if not is_restricted:
|
||||||
|
if user.has_group('mrp_mps.group_mrp_mps'):
|
||||||
|
is_restricted = True
|
||||||
|
else:
|
||||||
|
# Fallback search by name if XML ID not found or module not standard in this env
|
||||||
|
if self.env['res.groups'].search_count([('name', '=', 'MPS User'), ('id', 'in', user.groups_id.ids)]):
|
||||||
|
is_restricted = True
|
||||||
|
|
||||||
|
# Logic: If user is in ANY of the restricted groups, we hide the button.
|
||||||
|
# However, usually "Manager" groups inherit "User" groups.
|
||||||
|
# So a Manager would also be a User.
|
||||||
|
# We must Ensure that we DO NOT hide it if the user is a Quality Manager or Quality User?
|
||||||
|
# The request says: "only hide ... from user in inventory user group, manufacturing user group and MPS user group"
|
||||||
|
# It implies if I am ONLY one of those, I shouldn't see it.
|
||||||
|
# But if I am ALSO a Quality User, should I see it?
|
||||||
|
# Usually, Quality User > Inventory User regarding Quality checks.
|
||||||
|
# If I hide it for Inventory User, and I am both Inventory User AND Quality User, I will simply NOT see it if I check "if has_group(Inventory)".
|
||||||
|
# So I should probably check if the user is NOT a Quality User/Manager.
|
||||||
|
|
||||||
|
# But the user request is specific: "change the logic... to only hide ... from user in [groups]"
|
||||||
|
# If I am an Inventory User, I shouldn't see it.
|
||||||
|
# If I am also a Quality User, do I see it? Standard Odoo: Quality Users see it.
|
||||||
|
# If the user wants to restriction, likely they want these specific functional users NOT to do quality checks
|
||||||
|
# UNLESS they are explicitly Quality Users?
|
||||||
|
# Or maybe they want to hide it EVEN IF they are Quality Users?
|
||||||
|
# "Only hide ... from user in ..." suggests targeting these specific roles.
|
||||||
|
|
||||||
|
# Let's refine the logic:
|
||||||
|
# Hide IF (User is Inventory OR Mfg OR MPS) AND (User is NOT Quality Manager/User?)
|
||||||
|
# Or is it a hard hide? "Only hide ... from [list]"
|
||||||
|
# If I assume the user implies "People who are just Inventory/Mfg/MPS users shouldn't see this",
|
||||||
|
# then if someone is ALSO a Quality Manager, they should probably see it.
|
||||||
|
# So I will add an exception: If user is Quality User+, they see it.
|
||||||
|
|
||||||
|
is_quality_user = user.has_group('quality.group_quality_user') or user.has_group('quality.group_quality_manager')
|
||||||
|
|
||||||
|
if is_restricted and not is_quality_user:
|
||||||
|
record.hide_quality_check_button = True
|
||||||
|
else:
|
||||||
|
record.hide_quality_check_button = False
|
||||||
|
>>>>>>> 3a96e90 (feat: Hide quality check buttons on manufacturing orders for specific user groups with exceptions for quality roles.)
|
||||||
|
=======
|
||||||
|
# Logic: Hide logic is inverse of "is allowed"
|
||||||
|
# If allowed_quality_checks is True, hide = False
|
||||||
|
# If allowed_quality_checks is False, hide = True
|
||||||
|
record.hide_quality_check_button = not self.env.user.allowed_quality_checks
|
||||||
|
>>>>>>> 4a049d1 (feat: Implement user-specific control for quality check button visibility on manufacturing orders and inventory transfers via a new `allowed_quality_checks` field on users.)
|
||||||
|
|||||||
@ -47,3 +47,10 @@ class ResUsers(models.Model):
|
|||||||
string="Allowed Approvals",
|
string="Allowed Approvals",
|
||||||
help="Approval Categories this user is allowed to access. Leave empty to restrict access to none."
|
help="Approval Categories this user is allowed to access. Leave empty to restrict access to none."
|
||||||
)
|
)
|
||||||
|
|
||||||
|
allowed_quality_checks = fields.Boolean(
|
||||||
|
string="Is Allowed todo Quality Checks?",
|
||||||
|
default=False,
|
||||||
|
help="If checked, this user can see the Quality Checks button on Manufacturing Orders.",
|
||||||
|
prefetch=False,
|
||||||
|
)
|
||||||
|
|||||||
@ -3,6 +3,7 @@ from odoo import models, fields, api
|
|||||||
class StockPicking(models.Model):
|
class StockPicking(models.Model):
|
||||||
_inherit = 'stock.picking'
|
_inherit = 'stock.picking'
|
||||||
|
|
||||||
|
<<<<<<< HEAD
|
||||||
restrict_quality_check_button = fields.Boolean(compute='_compute_restrict_quality_check_button')
|
restrict_quality_check_button = fields.Boolean(compute='_compute_restrict_quality_check_button')
|
||||||
|
|
||||||
@api.depends_context('uid')
|
@api.depends_context('uid')
|
||||||
@ -32,3 +33,12 @@ class StockPicking(models.Model):
|
|||||||
picking.restrict_quality_check_button = True
|
picking.restrict_quality_check_button = True
|
||||||
else:
|
else:
|
||||||
picking.restrict_quality_check_button = False
|
picking.restrict_quality_check_button = False
|
||||||
|
=======
|
||||||
|
hide_quality_check_button = fields.Boolean(compute='_compute_hide_quality_check_button')
|
||||||
|
|
||||||
|
@api.depends('picking_type_id')
|
||||||
|
@api.depends_context('uid')
|
||||||
|
def _compute_hide_quality_check_button(self):
|
||||||
|
for record in self:
|
||||||
|
record.hide_quality_check_button = not self.env.user.allowed_quality_checks
|
||||||
|
>>>>>>> 4a049d1 (feat: Implement user-specific control for quality check button visibility on manufacturing orders and inventory transfers via a new `allowed_quality_checks` field on users.)
|
||||||
|
|||||||
60
views/mrp_production_v19.xml
Normal file
60
views/mrp_production_v19.xml
Normal file
@ -0,0 +1,60 @@
|
|||||||
|
<?xml version="1.0" encoding="utf-8"?>
|
||||||
|
<odoo>
|
||||||
|
<<<<<<< HEAD:views/mrp_production_views.xml
|
||||||
|
<<<<<<< HEAD
|
||||||
|
<!-- Inherit from quality_mrp view -->
|
||||||
|
<record id="view_mrp_production_form_inherit_access_restriction" model="ir.ui.view">
|
||||||
|
=======
|
||||||
|
<record id="mrp_production_view_form_inherit_access_restriction" model="ir.ui.view">
|
||||||
|
>>>>>>> 3a96e90 (feat: Hide quality check buttons on manufacturing orders for specific user groups with exceptions for quality roles.)
|
||||||
|
=======
|
||||||
|
<record id="mrp_production_view_form_inherit_access_restriction_v19" model="ir.ui.view">
|
||||||
|
>>>>>>> 5091c7f (feat: Update module to Odoo 19 by bumping version, adapting MRP and Stock Picking views, and removing stock valuation layer access rules.):views/mrp_production_v19.xml
|
||||||
|
<field name="name">mrp.production.view.form.inherit.access.restriction</field>
|
||||||
|
<field name="model">mrp.production</field>
|
||||||
|
<field name="priority">99999</field>
|
||||||
|
<field name="inherit_id" ref="mrp.mrp_production_form_view"/>
|
||||||
|
<field name="arch" type="xml">
|
||||||
|
<<<<<<< HEAD:views/mrp_production_views.xml
|
||||||
|
<<<<<<< HEAD
|
||||||
|
<<<<<<< HEAD
|
||||||
|
<field name="state" position="before">
|
||||||
|
<field name="restrict_quality_check_button" invisible="1"/>
|
||||||
|
</field>
|
||||||
|
|
||||||
|
<!-- Header Button -->
|
||||||
|
<xpath expr="//button[@name='check_quality']" position="attributes">
|
||||||
|
<attribute name="invisible">not quality_check_todo or restrict_quality_check_button</attribute>
|
||||||
|
</xpath>
|
||||||
|
|
||||||
|
<!-- Smart Buttons -->
|
||||||
|
<xpath expr="//button[@name='%(quality_mrp.quality_check_action_mo)d'][1]" position="attributes">
|
||||||
|
<attribute name="invisible">not check_ids or quality_check_fail or not quality_check_todo or restrict_quality_check_button</attribute>
|
||||||
|
</xpath>
|
||||||
|
<xpath expr="//button[@name='%(quality_mrp.quality_check_action_mo)d'][2]" position="attributes">
|
||||||
|
<attribute name="invisible">not check_ids or quality_check_fail or quality_check_todo or restrict_quality_check_button</attribute>
|
||||||
|
</xpath>
|
||||||
|
<xpath expr="//button[@name='%(quality_mrp.quality_check_action_mo)d'][3]" position="attributes">
|
||||||
|
<attribute name="invisible">not check_ids or not quality_check_fail or restrict_quality_check_button</attribute>
|
||||||
|
</xpath>
|
||||||
|
|
||||||
|
=======
|
||||||
|
<xpath expr="//header" position="inside">
|
||||||
|
=======
|
||||||
|
<xpath expr="//sheet" position="inside">
|
||||||
|
>>>>>>> 4a049d1 (feat: Implement user-specific control for quality check button visibility on manufacturing orders and inventory transfers via a new `allowed_quality_checks` field on users.)
|
||||||
|
=======
|
||||||
|
<sheet position="inside">
|
||||||
|
>>>>>>> 5091c7f (feat: Update module to Odoo 19 by bumping version, adapting MRP and Stock Picking views, and removing stock valuation layer access rules.):views/mrp_production_v19.xml
|
||||||
|
<field name="hide_quality_check_button" invisible="1"/>
|
||||||
|
</sheet>
|
||||||
|
<xpath expr="//button[@name='check_quality']" position="attributes">
|
||||||
|
<attribute name="invisible">hide_quality_check_button</attribute>
|
||||||
|
</xpath>
|
||||||
|
<xpath expr="//button[@name='button_quality_alert']" position="attributes">
|
||||||
|
<attribute name="invisible">hide_quality_check_button</attribute>
|
||||||
|
</xpath>
|
||||||
|
>>>>>>> 3a96e90 (feat: Hide quality check buttons on manufacturing orders for specific user groups with exceptions for quality roles.)
|
||||||
|
</field>
|
||||||
|
</record>
|
||||||
|
</odoo>
|
||||||
@ -1,31 +0,0 @@
|
|||||||
<?xml version="1.0" encoding="utf-8"?>
|
|
||||||
<odoo>
|
|
||||||
<!-- Inherit from quality_mrp view -->
|
|
||||||
<record id="view_mrp_production_form_inherit_access_restriction" model="ir.ui.view">
|
|
||||||
<field name="name">mrp.production.view.form.inherit.access.restriction</field>
|
|
||||||
<field name="model">mrp.production</field>
|
|
||||||
<field name="inherit_id" ref="quality_mrp.mrp_production_view_form_inherit_quality"/>
|
|
||||||
<field name="arch" type="xml">
|
|
||||||
<field name="state" position="before">
|
|
||||||
<field name="restrict_quality_check_button" invisible="1"/>
|
|
||||||
</field>
|
|
||||||
|
|
||||||
<!-- Header Button -->
|
|
||||||
<xpath expr="//button[@name='check_quality']" position="attributes">
|
|
||||||
<attribute name="invisible">not quality_check_todo or restrict_quality_check_button</attribute>
|
|
||||||
</xpath>
|
|
||||||
|
|
||||||
<!-- Smart Buttons -->
|
|
||||||
<xpath expr="//button[@name='%(quality_mrp.quality_check_action_mo)d'][1]" position="attributes">
|
|
||||||
<attribute name="invisible">not check_ids or quality_check_fail or not quality_check_todo or restrict_quality_check_button</attribute>
|
|
||||||
</xpath>
|
|
||||||
<xpath expr="//button[@name='%(quality_mrp.quality_check_action_mo)d'][2]" position="attributes">
|
|
||||||
<attribute name="invisible">not check_ids or quality_check_fail or quality_check_todo or restrict_quality_check_button</attribute>
|
|
||||||
</xpath>
|
|
||||||
<xpath expr="//button[@name='%(quality_mrp.quality_check_action_mo)d'][3]" position="attributes">
|
|
||||||
<attribute name="invisible">not check_ids or not quality_check_fail or restrict_quality_check_button</attribute>
|
|
||||||
</xpath>
|
|
||||||
|
|
||||||
</field>
|
|
||||||
</record>
|
|
||||||
</odoo>
|
|
||||||
@ -16,6 +16,7 @@
|
|||||||
<group string="Manufacturing & others">
|
<group string="Manufacturing & others">
|
||||||
<field name="allowed_workcenter_ids" widget="many2many_tags"/>
|
<field name="allowed_workcenter_ids" widget="many2many_tags"/>
|
||||||
<field name="allowed_approval_category_ids" widget="many2many_tags"/>
|
<field name="allowed_approval_category_ids" widget="many2many_tags"/>
|
||||||
|
<field name="allowed_quality_checks"/>
|
||||||
</group>
|
</group>
|
||||||
</group>
|
</group>
|
||||||
</page>
|
</page>
|
||||||
|
|||||||
@ -1,5 +1,7 @@
|
|||||||
<?xml version="1.0" encoding="utf-8"?>
|
<?xml version="1.0" encoding="utf-8"?>
|
||||||
<odoo>
|
<odoo>
|
||||||
|
<<<<<<< HEAD:views/stock_picking_views.xml
|
||||||
|
<<<<<<< HEAD
|
||||||
<!-- Inherit from the quality control view that adds the button -->
|
<!-- Inherit from the quality control view that adds the button -->
|
||||||
<!-- We need to ensure we target the right buttons. The Quality Checks button is added by quality_control/views/stock_picking_views.xml -->
|
<!-- We need to ensure we target the right buttons. The Quality Checks button is added by quality_control/views/stock_picking_views.xml -->
|
||||||
<!-- The button names are 'check_quality', 'open_quality_alert_picking', 'action_open_quality_check_picking' -->
|
<!-- The button names are 'check_quality', 'open_quality_alert_picking', 'action_open_quality_check_picking' -->
|
||||||
@ -32,6 +34,26 @@
|
|||||||
<attribute name="invisible">not check_ids or not quality_check_fail or restrict_quality_check_button</attribute>
|
<attribute name="invisible">not check_ids or not quality_check_fail or restrict_quality_check_button</attribute>
|
||||||
</xpath>
|
</xpath>
|
||||||
|
|
||||||
|
=======
|
||||||
|
<record id="stock_picking_view_form_inherit_access_restriction" model="ir.ui.view">
|
||||||
|
=======
|
||||||
|
<record id="stock_picking_view_form_inherit_access_restriction_v19" model="ir.ui.view">
|
||||||
|
>>>>>>> 5091c7f (feat: Update module to Odoo 19 by bumping version, adapting MRP and Stock Picking views, and removing stock valuation layer access rules.):views/stock_picking_v19.xml
|
||||||
|
<field name="name">stock.picking.view.form.inherit.access.restriction</field>
|
||||||
|
<field name="model">stock.picking</field>
|
||||||
|
<field name="priority">1000</field>
|
||||||
|
<field name="inherit_id" ref="stock.view_picking_form"/>
|
||||||
|
<field name="arch" type="xml">
|
||||||
|
<xpath expr="//sheet" position="inside">
|
||||||
|
<field name="hide_quality_check_button" invisible="1"/>
|
||||||
|
</xpath>
|
||||||
|
<xpath expr="//button[@name='check_quality']" position="attributes">
|
||||||
|
<attribute name="invisible">hide_quality_check_button</attribute>
|
||||||
|
</xpath>
|
||||||
|
<xpath expr="//button[@name='button_quality_alert']" position="attributes">
|
||||||
|
<attribute name="invisible">hide_quality_check_button</attribute>
|
||||||
|
</xpath>
|
||||||
|
>>>>>>> 4a049d1 (feat: Implement user-specific control for quality check button visibility on manufacturing orders and inventory transfers via a new `allowed_quality_checks` field on users.)
|
||||||
</field>
|
</field>
|
||||||
</record>
|
</record>
|
||||||
</odoo>
|
</odoo>
|
||||||
Loading…
Reference in New Issue
Block a user