Error in nodereview_list_axes

twohills - August 1, 2008 - 02:36
Project:NodeReview
Version:5.x-1.0
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:active
Description

Line 452 $axes[$node_type][$record->aid] = $record->tag;
should read
$axes[$node_type][$record->aid] = $record->aid;
$axes[$node_type][$record->tag] = $record->tag;

#1

twohills - August 1, 2008 - 03:10

sorry: and...

line 118 of nodereview_node_nodereview.inc, function _nodereview_form_review
    '#default_value' => $votes[$axis->tag] ? $votes[$axis->tag] : 50,
should read
    '#default_value' => $votes[$axis->aid] ? $votes[$axis->aid] : 50,

line 174 of nodereview_node_nodereview.inc, function nodereview_load
$record->score = $votes[$record->tag]->value;
should read
$record->score = $votes[$record->aid]->value;

The confusion arises because voting uses "tag" to refer to the unique identifier of the thing being voted on.
The unique identifier of the axis is the aid not the tag.
In most cases it probably isn't an issue but i am "bending" nodereview and it may be for me (and others).

 
 

Drupal is a registered trademark of Dries Buytaert.