I've wondered why $node is not defined anymore in my theme's page.tpl.php.
The reason is a modified expression in phptemplate.engine, which now reads:

if ((arg(0) == 'node') && is_int(arg(1))) {

arg(1) may be the node number, but it is a string variable, and therefore this check is always false.
is_int should be replaced with the code given in the patch.

CommentFileSizeAuthor
#1 phptemplate-is_int.patch739 bytesRobrecht Jacques
phptemplate_node.patch720 byteswiz-1

Comments

Robrecht Jacques’s picture

Robrecht Jacques commented
StatusFileSize
new phptemplate-is_int.patch739 bytes

Rerolled the patch (and changed the coding: eg Drupal uses "&&", not "and"). I think wiz is correct: you should use "is_numeric" here, not "is_int".

Dave Cohen’s picture

Dave Cohen commented

Consider: if (arg(1) && ctype_digit(arg(1)));

reportedly performs better than is_numeric(), and true only for integers.

dries’s picture

dries commented
Status: Needs review » Fixed

Committed to HEAD.

Anonymous’s picture

(not verified) commented
Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)