Closed (fixed)
Project:
Panels
Version:
4.7.x-1.0
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Jan 2007 at 11:19 UTC
Updated:
29 Mar 2007 at 19:32 UTC
Panels should not use drupal_goto('admin/panels') in the delete confirm form submit handler, it should use return 'admin/panels';.
If you use drupal_goto(); then other modules can not add another submit handler to the form. I'm stumbling on this problem while adding support for panels module in my nodewords module (which adds META tags to pages).
Patch attached.
| Comment | File | Size | Author |
|---|---|---|---|
| panels.drupal_goto.patch.txt | 559 bytes | Robrecht Jacques |
Comments
Comment #1
Robrecht Jacques commentedComment #2
Robrecht Jacques commentedSome extra information from #drupal IRC:
Comment #3
slantview commented+1 Looks good to me.
Comment #4
merlinofchaos commentedCommitted. Thanks for the patch!
Comment #5
(not verified) commented