When error reporting is set to E_ALL, the decisions module returns the following notices when editing a node.

# notice: Undefined property: stdClass::$uselist in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 96.
# notice: Undefined offset: 2 in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 104.
# notice: Undefined property: stdClass::$quorum_abs in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 122.
# notice: Undefined property: stdClass::$quorum_percent in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 122.
# notice: Undefined property: stdClass::$active in /var/www/drupal6.sites/all/modules/decisions/decisions_node.inc on line 232.
# notice: Undefined property: stdClass::$active in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 43.
# notice: Undefined property: stdClass::$uselist in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 96.
# notice: Undefined offset: 2 in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 104.
# notice: Undefined property: stdClass::$quorum_abs in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 122.
# notice: Undefined property: stdClass::$quorum_percent in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 122.
# notice: Undefined property: stdClass::$active in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 43.
# notice: Undefined property: stdClass::$algorithm in /var/www/drupal6.sites/all/modules/decisions/modes/ranking.module on line 303.
# notice: Undefined property: stdClass::$startdate in /var/www/drupal6.sites/all/modules/decisions/decisions.module on line 260.
# notice: Undefined property: stdClass::$runtime in /var/www/drupal6.sites/all/modules/decisions/decisions.module on line 261.
# notice: Undefined property: stdClass::$uselist in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 96.
# notice: Undefined offset: 2 in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 104.
# notice: Undefined property: stdClass::$uselist in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 96.
# notice: Undefined offset: 2 in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 104.
# notice: Undefined property: stdClass::$quorum_abs in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 122.
# notice: Undefined property: stdClass::$quorum_percent in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 122.
# notice: Undefined offset: 2 in /var/www/drupal6.sites/all/modules/decisions/decisions.inc on line 104.

The issue http://drupal.org/node/399952#comment-1434276 also reports such notices on lines 189, 190 and 196 of decisions_node.inc.

This problem shouldn't affect production sites unless E_NOTICE events are reported.

It would be nice to have this fixed and reviewed, though for the next stable release.

Comments

anarcat’s picture

Priority: Minor » Critical
anarcat’s picture

Priority: Critical » Minor
ezra-g’s picture

Status: Active » Needs review
StatusFileSize
new2.23 KB

I think some of the notices in the original report have been fixed, but I was getting several on the node/add/decisions-selection page. Patch attached :)

ezra-g’s picture

Status: Needs review » Needs work

Actually, there are a bunch of other ones I just noticed after submitting decision nodes.

ezra-g’s picture

Status: Needs work » Needs review
StatusFileSize
new3.89 KB

Found a few more. I feel like we can fix these as more come up.

anarcat’s picture

Assigned: Unassigned » anarcat

I can't review this today, I'll try to take a look next week, thanks!

anarcat’s picture

Status: Needs review » Fixed

Fix committed. Reopen if new ones are found (even without a patch).

Status: Fixed » Closed (fixed)

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