Closed (fixed)
Project:
Nodewords: D6 Meta Tags
Version:
5.x-1.12
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Jul 2007 at 20:59 UTC
Updated:
14 Jul 2012 at 19:26 UTC
Jump to comment: Most recent file
Comments
Comment #1
summit commentedHi,
I read that panels 2 is ready for Meta Tags (http://drupal.org/node/164768), please make Meta tags work with panels 2.
I need this because the meta tags are still used in google to see if a page is unique.
Thanks in advance!
greetings,
Martijn
Comment #2
johnalbinIn Panels 2.x, the panels_info database table has been renamed to panels_page.
Martijn, if you need a quick fix, you can use the patch I’ve attached. However, the patch isn’t a permanent fix as it would break support for Panels 1.x.
Comment #3
summit commentedHi John,
Thanks for the patch.
I am on holliday in spain.
I think I definitely stay with panels 2.0. It is such a great improvement from panels 1.1.
Earl did a great job for Drupal in my perspective!
Maybe there is a way to make this patch general by building an IF THEN ELSE around the different tables related to the panels version...
Greetings,
Martijn
www.haan.be (I am on holliday here :)
Comment #4
thierry_gd commentedTested the replacement of panels_info by panels_page
==> Works OK, can be applied
Comment #5
najibx commentedI have done the patch. Normally metatag is done on the individual page. In panel/views, where this is done?
Under general setting i.e, admin/build/panel-page/8/edit, I don't see the option there?
Comment #6
summit commentedHi,
I think you are right. The patch is not working.
I think there has to be done more.
The panels_page module should be used instead of panels.module.
I tried to make the changes in nodewords.module but unsuccesfull until now.
I tried to rename:
1) panels_edit_form -> panels_page_edit_form
2) panels_delete_confirm -> panels_page_delete_confirm
but then I am stuck.. I don't know if all the panels should be renamed to panels_page.
I am getting the meta-tags saved to the database, but I am not getting them shown on admin or in the source...
please help!
greetings,
Martijn
Comment #7
summit commentedHi,
I am able to insert the panelpage metatags now, and I am able to save them and get them back on admin page.
I use the PID from the panels_page database-table.
I am still not able to show the filled in metatags in the sourcecode of the panels_page page...and that is what the metatags are ment for right :(
Could somebody please help to build the patch.
My halfway-solution Nodewords-Panels sourcecode is:
please help!
greetings,
Martijn
Comment #8
summit commentedIt looks like it is working with the attached sourcecode.
I do not know how to file a patch, sorry for that.
And I do not know if I am correct to use the PID instead of the DID.
The pieces for panels are the same as the above post.
It worked after setting back pathauto from 2.0-dev, to 1.2. I do not know why..but it works now!
Please confirm working of this code and if so please build a patch.
Thanks in advance,
greetings,
Martijn
Comment #9
iaminawe commentedsubscribe ... looking to do this too
Comment #10
summit commentedThis is implemented in the newest version of Meta tags, differently, but it is working great, so changed to fixed.
greetings,
Martijn
Comment #11
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.