When used with Exclude node title module http://drupal.org/project/exclude_node_title/, [node:title] is empty, which I guess is kind of expected really.
It would be useful if that wasn't the case though.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

Status: Active » Needs review

Per antonyanimator in #1721026: Undefined index errors and Page Title not displaying in the browser, try using [current-page:title] as the value for the tag.

DamienMcKenna’s picture

Title: Exclude node title module » Compatibility problem with Exclude Node Title module
Status: Needs review » Active

Clarifying the title.

hixster’s picture

sorry I never came back on this [current-page:title] worked. thanks

DamienMcKenna’s picture

Component: Code » Documentation
Category: bug » task

Thanks for the update, lets leave this open as a documentation task.

DamienMcKenna’s picture

DamienMcKenna’s picture

Status: Active » Needs review
FileSize
641 bytes

Is something like this sufficient?

Status: Needs review » Needs work

The last submitted patch, metatag-n1722564-6.patch, failed testing.

DamienMcKenna’s picture

Version: 7.x-1.0-alpha7 » 7.x-1.x-dev
Status: Needs work » Needs review
DamienMcKenna’s picture

#6: metatag-n1722564-6.patch queued for re-testing.

greggles’s picture

IMO it would be better to have a hook_enable or hook_requirements that detects the installation of both modules and complains about it.

DamienMcKenna’s picture

FileSize
1.79 KB

@greggles: We happen to have something similar to that for the Page Title module already. How does the included message work - too verbose?

DamienMcKenna’s picture

FileSize
1.77 KB

Improved/fixed the hook_requirements message.

DamienMcKenna’s picture

Status: Needs review » Fixed

Committed, thanks for the help everyone.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.