Problem/Motivation

Move Context mapping to be inside a try/catch to catch MissingValueContextException & ContextException

Same as Drupal\block\BlockAccessControlHandler::checkAccess()

Steps to reproduce

Add an ctools block EntityField to a context reaction - see WSOD when not on a node page.

Proposed resolution

Put $block->access() inside try/catch.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 errorcontext.png60.16 KBguilhermevp
#2 catch-missingvaluecontextexception-3202392-2.patch2.82 KBsilverham

Comments

silverham created an issue. See original summary.

silverham’s picture

silverham commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new catch-missingvaluecontextexception-3202392-2.patch2.82 KB
guilhermevp’s picture

guilhermevp commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new errorcontext.png60.16 KB

I was able to reproduce the error, and the patch prevents WSOD.

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented

Nice catch @silverham.
Thanks!

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.