Due to the check for poll results the menu_get_object() gets called. When in a particular use case menu_set_item(NULL,$node) is set, for example to change the active menu trail, the menu_get_object() the nid that is returned after checking for the poll results is not the nid from the $arg[1] but the $node->nid from $node = menu_get_object().

I think it will be better to return $arg[1] instead of $node->nid

CommentFileSizeAuthor
#3 nodewords-n1961104-3.patch450 bytesdamienmckenna
#1 nodewords-n1961104.patch359 bytesralphvdhoudt

Comments

ralphvdhoudt’s picture

ralphvdhoudt commented
StatusFileSize
new nodewords-n1961104.patch359 bytes

My proposed path to fix the issue

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review

In that case, shouldn't it instead change to the following?

    if (isset($arg[1]) && is_numeric($arg[1]) && ($node = node_load($arg[1]))) {
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 6.x-1.14 » 6.x-1.x-dev
Issue summary: View changes
StatusFileSize
new nodewords-n1961104-3.patch450 bytes

A patch based upon the code from #2.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Committed.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Parent issue: » #2533956: Plan for Nodewords v6.x-1.15 release

  • DamienMcKenna committed b591c37 on 6.x-1.x 
    Issue #1961104 by DamienMcKenna: Use arg(1) when loading the node on...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.