Closed (fixed)
Project:
Nodewords: D6 Meta Tags
Version:
6.x-1.0
Component:
Code
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
19 Oct 2008 at 05:55 UTC
Updated:
17 Jul 2009 at 16:10 UTC
Entering values into TD.title didn't actually change the page title. I found, when examining nodewords/metatags/DC.Title.inc (why its capitalized, I have no idea) that the function nodewords_dc_title_prepeare only accounted for what to do when a DC.Title was NOT supplied by the user. The first if was never followed up by an else:
if (!isset($value) || empty($value)) {
I added the following else, and moved the return $value below it.
else {
// what if it was actually provided?
$value .= ' - ' . variable_get('site_name', 'Drupal');
}
return $value;
patch attached.
Jen
| Comment | File | Size | Author |
|---|---|---|---|
| dc_title_else.patch | 503 bytes | jenlampton |
Comments
Comment #1
smitty commentedThis works fine for me too! Please commit it.
Comment #2
avpadernoAs the Drupal 5 version is not supported anymore, I am changing the status of this report.
Comment #3
smitty commentedIs this solved in D6 already?
Comment #4
avpadernoSettings the DC.Title doesn't change the page title, because the DC.Title meta tag is not though to replace the page title (which is the one set in the title tag), but to suggest an alternative title to the search engines that actually use it.
Comment #5
avpaderno