In statistics.module at line 165 the following code

if ($node = node_load(arg(1))) {

should be

>
if ($node = node_load(array(nid => arg(1)))) {

to stop very bad implode errors in node.module when $cond is imploded in node_load at line 363 of node.module in drupal 4.6.3.

The same code is still there in HEAD, so I think the bug applies there too.

CommentFileSizeAuthor
#1 statistics.module.node_load.patch377 bytesdopry

Comments

dopry’s picture

dopry commented
Version: 4.6.3 » 4.7.0-beta3
Assigned: Unassigned » dopry
StatusFileSize
new statistics.module.node_load.patch377 bytes

Verified that it is fixed in 4.6.5...

made a patch against 4.7.0-beta3, seems like it should apply against head as well.

dopry’s picture

dopry commented
Status: Active » Needs review

updating status.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Closed (works as designed)

node_load() now accepts an integer in 4.7. AFAICT, there is no bug here. you seem to be using 4.7 statistics.module with a 4.6.x site?