Closed (outdated)
Project:
Panels
Version:
7.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
2 Oct 2010 at 19:12 UTC
Updated:
14 Jul 2026 at 02:28 UTC
Jump to comment: Most recent
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
Comment #1
Letharion commentedThis seems reasonable to me, and it looks like this still applies to the latest version. @merlinofchaos, do you have any comments? :)
Comment #2
japerryDrupal 7 is no longer supported, closing.