I've updated the module to version 7.x-3.6
And when trying to update the Panel's content I was receiving a notice with the following text:
Notice: Undefined property: panels_display::$storage_id in panels_panel_context_edit_content_submit() (line 760
Maybe the root of this is much deeper but from the first glance, the notice is caused because there isn't a check if this property exists or not. I will provide a simple patch that fixes this notice.

CommentFileSizeAuthor
#2 panels_storage-id-notice_2786173_2_7.patch604 bytesthe.tai.pen@gmail.com

Comments

andrei.colesnic created an issue. See original summary.

the.tai.pen@gmail.com’s picture

the.tai.pen@gmail.com commented
Status: Active » Needs review
StatusFileSize
new panels_storage-id-notice_2786173_2_7.patch604 bytes

Providing the patch

the.tai.pen@gmail.com’s picture

the.tai.pen@gmail.com commented
Issue summary: View changes
dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented

Hm, this should always exist assuming you've run the database updates via update.php or drush updb. Is it possible that you forgot to do that?

japerry’s picture

japerry
Location KOMK
commented
Parent issue: » #2786071: Plan for Panels 7.x-3.7 release
capysara’s picture

capysara commented

I was getting this error message after the update to 7.x-3.6. I ran drush updb again, no updates required (again), but the Notice no longer appears.

the.tai.pen@gmail.com’s picture

the.tai.pen@gmail.com commented

Thank you for a quick feedback. I did the updb(once) right after I updated the module - and there were no errors during the update. The panels display table has the new columns( storage_type, storage_id ) but not all rows have values for this new columns.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented

The panels display table has the new columns( storage_type, storage_id ) but not all rows have values for this new columns.

Hm. What Panels modules are you using? We added update hooks to page_manager, panels_node, panels_mini and panelizer that should fill that column. Maybe you have a module we didn't consider? Or, possibly there was an error while running one of the update hooks.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Needs review » Needs work

Marking as "Needs work" since the patch is just preventing the notice, not addressing the underlying probelm

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Needs work » Postponed (maintainer needs more info)

Postponing on needing more info - I don't think we should commit a patch that only addresses the symptoms without steps to reproduce and knowing what the cause is.

vitalym’s picture

vitalym commented

The issue was due to outdated Panelizer module required to be updated along with Panels.

japerry’s picture

japerry
Location KOMK
commented
Status: Postponed (maintainer needs more info) » Closed (outdated)

Drupal 7 is no longer supported, closing.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.