Got pink PHP messages:
Notice: Trying to get property of non-object in _exclude_node_title() (line 177 of C:\wamp\www\drtest\sites\all\modules\exclude_node_title\exclude_node_title.module).
Notice: Undefined index: in _exclude_node_title() (line 191 of C:\wamp\www\drtest\sites\all\modules\exclude_node_title\exclude_node_title.module).
when a requested page is not found and logged in as administrator.

CommentFileSizeAuthor
#3 exclude_node_title-check-for-node-existence-1661838-3.patch494 bytesthatjustin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dshirkalin’s picture

dshirkalin CreditAttribution: dshirkalin commented

A simple fix is to add
if (!$node || !$node->type) return FALSE;
before
$node_type = $node->type;
in exclude_node_title.module.

thatjustin’s picture

thatjustin CreditAttribution: thatjustin commented

Can we get this rolled into an update?

thatjustin’s picture

thatjustin CreditAttribution: thatjustin commented
FileSize
exclude_node_title-check-for-node-existence-1661838-3.patch494 bytes

Here's a patch.

gabrielu’s picture

gabrielu CreditAttribution: gabrielu commented
Status: Active » Closed (fixed)

Added to next release. Thanks @thatjustin for the patch. It seems the node is not loaded correctly.

Gabriel