PR State:
merged
PR:
#14
Commit:
8a13ac0115d28074176a5513eac591d9449bb3be
[REF] viin_brain, viin_ai_brain: drop group_brain_user, use base.group_user
group_brain_user was effectively an alias of base.group_user (it just
implied base.group_user and added no behavior). Carrying it around
inflated ACL/ir.rule entries and produced redundant runtime state with
no functional benefit.
- security/res_groups.xml: remove group_brain_user; group_brain_editor
now implies base.group_user directly. Two Brain groups remain:
editor + admin.
- ir.model.access.csv, ir_rules.xml: replace every group_brain_user
reference with base.group_user.
- Drop the auto-enroll machinery that only existed to put internal
users into group_brain_user: remove hooks.py (post_init_hook),
models/res_users.py (_brain_maybe_enroll), the
brain_default_enable_for_all_employees setting field and its
Settings UI block. Every internal user can use Brain by default;
vault membership + role drive editor/admin capabilities.
- Update __manifest__.py description (3 groups -> 2 groups), drop the
post_init_hook key, and update the demo data note.
- Tests: delete test_brain_security_default_enroll.py (covered a
feature that no longer exists and imported the deleted hooks
module), drop the cls.group_brain_user alias from common.py /
test_brain_comment.py / test_brain_tools.py, and switch remaining
references in test_brain_snapshot.py / test_brain_vault.py /
test_brain_acl_multicompany.py to group_internal_user.
- docs/brain/security.md, ai-bridge.md, roadmap.md: align wording
with the 2-group model.
Branch:
17.0