When I have ds enabled it starts throwing errors about panel layouts having a "content" region.
For example when using the adaptive theme for panels everywhere (http://drupal.org/project/at_panels_everywhere) I get following error on every page:
Following layouts have a "content" region key which is invalid: panels-atpe_site_template.
Problem is that it also happens for other panel layouts that have a content region.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 1468576-6.patch | 737 bytes | andypost |
| #5 | 1468576-5.patch | 580 bytes | swentel |
| #4 | 1468576-content.patch | 1.21 KB | andypost |
Comments
Comment #1
ramlev commentedI have the same issue with the content region, but if i change the region name to ex. main_content in the .inc file, and referer to that in the main-site.tpl.php file, everything works like a charm.
Seems to be an issue by calling the region key 'content'. Haven't looked in the source to locate the issue.
Comment #2
hnln commentedyep, for my own layouts it's "easy" to change the panel, but sometimes modules come with their own layouts and they sometimes use a 'content' region as well.
Comment #3
hnln commentedOrigin of the problem lies here: http://drupal.org/node/1382132
Maybe the check in ds_get_layout_info should check for $info['module'] != 'panels' (as ds_ds_layout_info also includes panels).
Comment #4
andypostpatch against at_panels_everywhere
Suppose we this problem needs more investigation about why region cant has a name "content"
Comment #5
swentel commentedHere's a patch against ds, could you test this one ?
Comment #6
andypostMake sense, but I extend a comment from #1382132: hook_ds_layout_info(): Some region names are forbidden.
Comment #7
swentel commentedYup, great. Committed and pushed, thanks!