Skip to content

FIX / Admin locks himself out of a container's configuration - #1255

Open
Mary-Clb wants to merge 2 commits into
mainfrom
fix/container-config-access-right
Open

FIX / Admin locks himself out of a container's configuration#1255
Mary-Clb wants to merge 2 commits into
mainfrom
fix/container-config-access-right

Conversation

@Mary-Clb

@Mary-Clb Mary-Clb commented Sep 3, 2026

Copy link
Copy Markdown
Contributor
  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes !46060
  • On the container's Profiles tab, setting the currently active profile's
    right to No access made the whole container configuration form
    (front/container.form.php?id=X) return "Access denied" — not just the block
    on the target itemtype.
    Gate the display branch with $container->check(id, READ) (fallback -1 for
    the add form), like front/container.php and the other sub-forms. Since
    $rightname = 'config', this enforces the standard config right + entity
    scoping. Removed the now-unused AccessDeniedHttpException import.

@Mary-Clb Mary-Clb self-assigned this Sep 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant