Incorrect access denied messages in branding block

Error refers to pages the user does not have access

When a user does not have permission to do something, we don't usually have an explanation message.

I'd like to propose removing the "You do not have the appropriate permissions to change the ...." message. I have a client that became confused why he was not granted permission to do something, despite being so close to being able to do it.

And remove the "Defined on the XXX page." text if the user does not have permission to access it.

Steps to reproduce

Add a new user and assign a role other than administrator say Content Editor (comes default with Drupal 10)

From the admin permissions page /admin/people/permissions, give permission to "Administer blocks" for the "Content Editor"

Configure Site Branding block /admin/structure/block/manage/olivero_site_branding to see the results in "Toggle branding elements
" section

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Status changed to Needs review about 1 year ago4:49pm 8 April 2023
Comment about 1 year ago →
🇺🇸United States smustgrave
Rerolled #27
error: patch failed: core/modules/block/tests/src/Functional/BlockSystemBrandingTest.php:119
error: core/modules/block/tests/src/Functional/BlockSystemBrandingTest.php: patch does not apply
Comment about 1 year ago →
🇮🇳India TanujJain-TJ
tested and verified patch #38 on claro and olivero themes, patch applied successfully and fixed the 'Incorrect access denied messages in branding block', attaching screenshots for reference. need RTBC+1
Status changed to RTBC about 1 year ago10:44am 10 April 2023
Comment about 1 year ago →
🇮🇳India saket-001
patch #38 is working fine. also mentioned in #39 so moving status to RTBC.
Status changed to Needs work about 1 year ago9:53pm 10 April 2023

Comment about 1 year ago →

🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

+++ b/core/modules/block/tests/src/Functional/BlockSystemBrandingTest.php
@@ -115,6 +115,17 @@ public function testSystemBrandingSettings() {
+    $this->assertSession()->responseNotContains('Defined on the Appearance or Theme Settings page. You do not have the appropriate permissions to change the site logo.');
+    $this->assertSession()->responseNotContains('Defined on the Site Information page. You do not have the appropriate permissions to change the site logo.');
+    $this->assertSession()->responseNotContains('Defined on the Site Information page. You do not have the appropriate permissions to change the site logo.');

We never emit 'You do not have the appropriate permissions' anymore, so these assertions would still pass even if the patch didn't work.

Let's change them to assert the new string isn't emitted.

Let's also add some positive asserts for the user that does have the required permissions

Comment about 1 year ago →
🇮🇳India Akram Khan Cuttack, Odisha
added updated patch and addressed #41

Incorrect access denied messages in branding block

Error description refers to incorrect element

Error refers to pages the user does not have access

Steps to reproduce

Comments & Activities