i am using nodewords module..
i have created a website which is having pages generated from view module...
i am able to get the meta tags for my normal static pages but not for the page generated from view module..
wht might be the reason...
need serious help..... pleaseeeee..
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 216262_nodewords_fix_for_views.patch | 2.31 KB | mrfelton |
| #4 | 216262_nodewords_fix_for_views.patch | 1.93 KB | mrfelton |
Comments
Comment #1
wmnnd commentedWhat about fixing this issue?
Comment #2
jhaggenburg commentedSame here. Any info on this please?
Comment #3
jhaggenburg commentedNevermind, i've fixed this with drupal_set_html_head()
Comment #4
mrfelton commentedI found that the meta tags are set for some views, but not all. I think the problem is due to the fact that the views detection code is run from nodewords_init(), and is based on a direct database query to look for views in the {views_display} table. However, this table doesn't contain information about ALL views, only those that have been manually created. This does not include those views that are provided by other modules.
I fixed this by using nodewords_views_pre_render() to to add nodewords to the page:
patch attached:
Comment #5
mrfelton commentedComment #6
mrfelton commentedDamn project.module didn't same my text, tag alterations!!... ^^^ Updated patch for a much more solid views support. Fixes some slopy regex in my previous patch, and works for all kinds of views.
Comment #7
avpadernoThe proposed patch contains a call to a third-party module private function (
_custom_breadcrumbs_paths_page_match()); Meta tags cannot rely on the presence of a third-party module, not depend from it.It's then a private function, and we should avoid to call it.
Comment #8
avpadernoI looked at the code of that function, and created a similar function in the project code.
The changes have been committed in CVS.
Thanks for your help.
Comment #10
powery commentedThere are no metatags on pages generated with views, on front page, on no-node pages.
Comment #11
Anonymous (not verified) commentedThis is an old issue. As the code is completely different from the one developed one year ago, it is better to open a different issue.