Closed (fixed)
Project:
Nodewords Page Title (Views, Panels, Non-nodes)
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
24 Jun 2010 at 22:38 UTC
Updated:
30 Mar 2011 at 02:30 UTC
Jump to comment: Most recent file
Comments
Comment #1
Brian294 commentedHi Martijn,
Thanks for submitting an issue in the nodewords queue. I have submitted a patch for nodewords.module that should fix the issue. It's against the most recent -dev version. FYI - you need to use the -dev version of nodeword_pagetitle if you want to use the -dev version of nodewords. There is an API change in nodewords.
http://drupal.org/node/837108#comment-3140234
Comment #2
Brian294 commentedComment #3
Brian294 commentedFor anyone using nodewords-beta9, here is the nodewords.module file you'll want to copy to your nodewords directory.
Comment #4
summit commentedHi Brian,
Will test this this week. Great! I am on Nodewords Beta9. Nodewords further on the line have still issues regarding showing the metatags.
greetings,
Martijn
Comment #5
Brian294 commentedThanks Martijn! Do me a favor, use this file instead. kiam brought up a good point about determining if an arg id belongs to a node, taxonomy, or user. This will ensure that the node/user/taxonomy tokens are not loaded if we our outside of their respective context.
Comment #6
summit commentedHi Brian,
I tested the patch. But I do not see the other tokens arising on the Custom pages meta tags page.
I thought they would arise, just as on the regular Page title page would.
May be related to: http://drupal.org/node/781440#comment-3150522
EDIT: I build this TokenArgument module, see: http://drupal.org/node/273893#comment-3151404 now the arguments are working! Only other tokens still not shown on Custom pages meta tags
greetings,
Martijn
Comment #7
Brian294 commentedyeah, I think you're right. I threw this together really quick for you to try. Let me know if that resolves the issue in beta9.
Peace,
Brian
Comment #8
summit commentedHi Brian,
Yes the nodewords.module in #7 is giving the other tokens also for beta9!
Would be great if you would file a patch for kiam to get this in.
Thanks a lot for the great support.
greetings,
Martijn
Comment #9
Brian294 commentedMartijn, thank YOU for being a great user! I'll whip up the dev patch tonight.
Comment #10
summit commentedHi Brian,
I have arg() now working great. And all tokens are showing in nodewords custom pages.
But when I try to use, say a cck field token, It is not showing.
Is what tokens are shown, related to the url? I have a selfmade url on which I would love to have termnames in the title, metatags description, and metatags keyword.
Would this be doable, my url is for instance. http://www.campings-europa.com/land/Nederland
Greetings,
Martijn
Comment #11
Brian294 commentedHi Martijn,
I submitted a patch to add additional tokens to the -dev version:
http://drupal.org/node/781440#comment-3163690
This fix does pretty much everything that the beta9 patch does, but takes into consideration a few more things. I tested CCK with this patch and it worked, although I did not make any changes to nodewords.module to address CCK.
I have attached the full nodewords.module file that can be applied overtop the current -dev version.
Peace,
Brian
Comment #12
PI_Ron commentedThanks for this patch guys, works great and much appreciated.
Comment #13
PI_Ron commentedHi Again,
Would there be a way to use tokens from http://drupal.org/project/term_fields within a custom pages meta tags?
Ive installed the dev version of this module, which I thought generated tokens based on the custom term fields, but they do not show up in the custom pages edit screen.
I thought it would be great to be able to setup custom term fields for 'Page Title', 'Description' etc. then use their tokens within a custom page, that way when setting up a term you could also set the meta tags directly from the term edit screen, and use one custom page meta tags setup for a vocab, instead of heaps.
What do you think?
Comment #14
dave reidThe upstream issue with nodewords (#781440: Missing tokens) has been fixed. Do we need to keep this issue open at all?
Comment #15
summit commentedHi Tim, I think you have to file a new issue to get attention on it.
greetings, Martijn
Comment #16
Brian294 commentedClosed per David Reid's comment in #14.