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)?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

codewatson’s picture

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

codewatson’s picture

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

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

codewatson’s picture

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

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

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’s picture

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

Status: Active » Needs review
FileSize
763 bytes

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

merlinofchaos’s picture

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

codewatson’s picture

Status: Needs review » Reviewed & tested by the community

That indeed did the trick

merlinofchaos’s picture

Status: Reviewed & tested by the community » Fixed

Committed!

Status: Fixed » Closed (fixed)

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