Skip to content

Style: Fix permission checks in Adminstration > Layout#10711

Merged
BettyFromHH merged 1 commit into
ILIAS-eLearning:release_11from
fneumann:fix11-mantis46419
Feb 2, 2026
Merged

Style: Fix permission checks in Adminstration > Layout#10711
BettyFromHH merged 1 commit into
ILIAS-eLearning:release_11from
fneumann:fix11-mantis46419

Conversation

@fneumann
Copy link
Copy Markdown
Contributor

See https://mantis.ilias.de/view.php?id=46419

This PR fixes the permission checks in Administration > Layout

  • The "Visible" is no longer checked because it doesn't exist
  • The "Read" permission is sufficient to view the kitchen sink documentation and the overview of the system styles
  • The "Edit System Styles" permission is needed to activate/deactivate system styles
  • The "Edit Content Styles" is needed to view the content styles and to show their tab
  • The "Edit Page Layouts" is needed to view the page layouts and to show their tab

This PR does not yet unbundle the access to the kitchen sink documentation from read access to the system styles. That is proposed in the FR More Visibility for Kitchen Sink

@fneumann fneumann self-assigned this Dec 15, 2025
@fneumann fneumann added bugfix php Pull requests that update Php code Looking for Shepherd labels Dec 15, 2025
@fneumann fneumann assigned BettyFromHH and alex40724 and unassigned fneumann Jan 22, 2026
@fneumann
Copy link
Copy Markdown
Contributor Author

Removed LookingForShepherd and assigned the responsibles for System and Content Styles, as suggested by the PM.

@alex40724 alex40724 self-requested a review January 31, 2026 08:57
Copy link
Copy Markdown
Member

@alex40724 alex40724 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fneumann Thanks, looks good for the content style part.

@BettyFromHH You may merge this, if your changes are ok, too.

@alex40724 alex40724 removed their assignment Jan 31, 2026
@BettyFromHH BettyFromHH merged commit 71364bb into ILIAS-eLearning:release_11 Feb 2, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix php Pull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants