I'm not sure if this is really a bug or by design however i seem to have a tab on the admin/build/panels/layouts page for every single layout i've created (see first image). Not sure the purpose of each tab though, when i click on them i go to a page like this: admin/build/panels/layouts/add-flexible:multi_3_column (see second image).

Strangely, if you edit the new layout, it seems like the layout thinks it is a panel itself, with the normal panels interface for adding content to the regions (see the bottom of image 2)?

CommentFileSizeAuthor
#9 877810-add-flexible.patch763 bytesmerlinofchaos
panels-tabs-page.jpg90.1 KBcodewatson
panels-layout-tabs.jpg70.1 KBcodewatson

Comments

codewatson’s picture

codewatson commented

Forgot to say this is with Drupal 6.17/Panels 3.7/Ctools 1.7

codewatson’s picture

codewatson commented

Ack, went to delete the layout and get this:

user warning: Table 'database.panels_layout' doesn't exist query: DELETE FROM panels_layout WHERE name = 'erm' in /home2/globanf0/public_html/drupal6/sites/all/modules/ctools/includes/export.inc on line 189.

merlinofchaos’s picture

merlinofchaos commented

That's crazy. Have you run update.php? Try rerunning update 6308 for Panels which is what creates that table.

codewatson’s picture

codewatson commented

I dont think its a lack of the table, i think that the query itself is not adding the prefix to the table, in this case i have a prefix of "d6_" yet the query is looking for 'database.panels_layout' (instead of database.d6_panels_layout), i ran into this error on the ctools custom content panes as well: #874052: Error on deleting a custom content pane from the ctools interface

codewatson’s picture

codewatson commented

just to confirm, the table exists and has the layouts i've created in it, however the delete query seems to be leaving off the table prefix

merlinofchaos’s picture

merlinofchaos commented

Ok, can you file a separate issue on the delete query? Much easier to deal with when there is one item per issue.

codewatson’s picture

codewatson commented

Done: #878114: DB error on delete a panel layout

codewatson’s picture

codewatson commented

To update this issue, i did a fresh drupal 6.17 install with panels 3.7 and ctools 1.7.

Created 2 panel layouts with a name of test and test2.

No extra tabs appeared

Cleared the cache at admin/settings/performance

2 extra "Add Flexible Layout" tabs appeared, both with ":layoutname" after the normal add flexible layout url (admin/build/panels/layouts/add-flexible)

so that gives:
admin/build/panels/layouts/add-flexible:test
and
admin/build/panels/layouts/add-flexible:test2

merlinofchaos’s picture

merlinofchaos commented
Status: Active » Needs review
StatusFileSize
new 877810-add-flexible.patch763 bytes

This patch is a shot in the dark but I bet does the trick.

merlinofchaos’s picture

merlinofchaos commented

You'll need to make sure menu gets rebuilt after applying the patch.

codewatson’s picture

codewatson commented
Status: Needs review » Reviewed & tested by the community

That indeed did the trick

merlinofchaos’s picture

merlinofchaos commented
Status: Reviewed & tested by the community » Fixed

Committed!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.