Closed (fixed)
Project:
Panels
Version:
7.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Aug 2016 at 06:23 UTC
Updated:
30 Oct 2016 at 01:24 UTC
Jump to comment: Most recent
In the function panels_panel_context_save() there is an if condition checking the existance of $page->conf['temp_layout'] which would always return FALSE since the variable $page is never defined in the scope of the function. I think it was meant to be $handler instead of $page.
Please review my attached patch.
| Comment | File | Size | Author |
|---|---|---|---|
| undefined-variable-page-in-panels-panel-context-save.patch | 562 bytes | ey |
Comments
Comment #2
japerryComment #3
dsnopekWow, this bug is old! The commit that originally added this is from 2010. I looked at the code and can confirm that it's really
$handlerthat has this, so marking RTBC.Comment #4
dsnopekMoving to 3.8 so we can get 3.7 out quicker!
Comment #6
japerryThanks for raising the visibility on this, its fixed now.