Add setReason() calls for debugging and consistency

Created on 11 September 2023, about 1 year ago

Updated 4 March 2024, 9 months ago

Problem/Motivation

in[#2936995] we are adding revision UI for taxonomy term, to keep things consistent and for better debugging we should add setReason() calls similar to TermAccessControlHandler.php in \Drupal\block_content\BlockContentAccessControlHandler::checkAccess.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

📌 Task

Status

Postponed: needs info

Version

11.0 🔥

Component

Block content →

Last updated 3 days ago

Maintained by
🇦🇺Australia @acbramley
🇺🇸United States @smustgrave
🇦🇺Australia @larowlan

Created by

🇮🇳India yash.rode pune

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @yash.rode
Status changed to Postponed about 1 year ago5:24am 11 September 2023
Comment about 1 year ago →
🇮🇳India yash.rode pune
Once [#2936995]is in, we should address this.
Comment about 1 year ago →
🇮🇳India yash.rode pune
Status changed to Postponed: needs info 9 months ago4:32am 4 March 2024
Comment 9 months ago →
🇦🇺Australia acbramley
The taxonomy issue is in, however BlockContentAccessControlHandler uses a match and returns AccessResult::allowedIfHasPermission which already sets a reason. I'm not seeing much to do here anymore?

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024