Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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 | Elin Yordanov | |
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
$handler
that 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.