Closed (fixed)
Project:
Nodewords: D6 Meta Tags
Version:
6.x-1.x-dev
Component:
Third-party modules integration
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
5 Nov 2009 at 05:32 UTC
Updated:
20 Nov 2009 at 11:00 UTC
Comments
Comment #1
avpadernoNodewords uses as teaser what Drupal code reports to be the teaser. I don't know what the other module does, but it could be it doesn't call the correct Drupal functions to alter the node teaser.
Comment #2
avpadernoPardon... Nodewords uses code taken from
node_teaser()to get the node teaser.Comment #3
avpadernoI think that the module could simply use the value of
$node->teaser, and verify if doesn't contain any PHP code, as it already does.In this way, it should be compatible with any modules that change the teaser.
Comment #4
avpadernoThe code has been changed, and committed in CVS.
Thanks for your report.
Comment #5
hd commentedI have successfully tested the 6.x-1.x-dev (from November 6). Now it is working together with the excerpt module. Thanks for having fixed this!
Comment #6
avpadernoThe fix was easy.
Why should have the module created the teaser when it was already present in
$node->teaser? I changed the existing code, but I have never thought that the object node contains a property that contain the node teaser; it contains the teaser, whatever it's created from Drupal core, of from a third-party module.Thanks again for your report; not using a third-party to create the teaser, I would not have noticed the problem.