Problem/Motivation

We are passing by reference the admin_toolbar_tools.flush menu object, but in restricted permissions this might not actually exist, which results in an unexpected operand failure.

Proposed resolution

Conditionally check if admin_toolbar_tools.flush exists.

Remaining tasks

N/A

User interface changes

N/A

API changes

Data model changes

CommentFileSizeAuthor
#2 unexpected-operand-2923580.patch2.8 KBsylus

Comments

sylus created an issue. See original summary.

sylus’s picture

sylus commented
StatusFileSize
new unexpected-operand-2923580.patch2.8 KB
adriancid’s picture

adriancid
Primary language Spanish
Location Qc, Canada 🇨🇦
commented
Title: Unexpected operand + restricted permissions + reference to admin_toolbar_tools.flush » Unsupported operand types in lazyBuilder()

Thanks @sylus

adriancid’s picture

adriancid
Primary language Spanish
Location Qc, Canada 🇨🇦
commented
Title: Unsupported operand types in lazyBuilder() » Unsupported operand types in ToolbarHandler::lazyBuilder()

  • adriancid committed b8272b7 on 8.x-1.x authored by sylus 
    Issue #2923580 by sylus, adriancid: Unsupported operand types in...
adriancid’s picture

adriancid
Primary language Spanish
Location Qc, Canada 🇨🇦
commented
Status: Active » Fixed

  • adriancid committed c1e5133 on 8.x-1.x 
    Issue #2923580 by sylus, adriancid: Unsupported operand types in...

Status: Fixed » Closed (fixed)

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