In the current version of node_teaser, teasers for nodes having both the PHP filter and at least one occurrence of the XML PI directive (<?
) are always built as the whole node. According to the API site, this is to avoid splitting code when generating the teaser.
Notwithstanding the fact that no sane site maintainer would probably insert <!--break--> at a point where it would break his PHP code, making this test of dubious value, there is a case where a proper teaser can be generated: that is, when the PI directive occurs only AFTER the <!--break--> in which case, generating the shortened teaser does not look like it could break anything.
The attached one-half-line patch allows this behaviour.
Since the HEAD verison of node.module today (1.687) uses the same test, it should also work on it.
Comment | File | Size | Author |
---|---|---|---|
node.module_43.patch | 674 bytes | fgm | |
Comments
Comment #1
killes@www.drop.org CreditAttribution: killes@www.drop.org commentedmoving to cvs to get more reviews. will backport if committed.
Comment #2
drummCommitted to HEAD.
Comment #3
(not verified) CreditAttribution: commentedComment #4
fgmKilles said he would backport if committed...
Comment #5
killes@www.drop.org CreditAttribution: killes@www.drop.org commentedbackported
Comment #6
fgmThanks