I have spent some time working on making panels thinking that it was something other than code. When I would create panels/pages for any of my children domain it would not actually save anything. I noticed in your "panels_save_display" function you utilize drupal_write_record. My recommendation is to wrap that in an if statement so that if there is error, you also write it to the screen as well as dblog. (As some people might not use dblog). Just a suggestion. It could be a simple:

if(!drupal_write_record('panels_display', $display, $update)){
drupal_set_message(/*error message here*/, 'error');
}

Just a suggestion. Thanks.

Comments

Letharion’s picture

Version: 6.x-3.7 » 7.x-3.x-dev
Assigned: Unassigned » merlinofchaos

This seems reasonable to me, and it looks like this still applies to the latest version. @merlinofchaos, do you have any comments? :)

japerry’s picture

Issue summary: View changes
Status: Active » 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.