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.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | decisions-undefined-423432.patch | 3.89 KB | ezra-g |
| #3 | decisions-undefined-423432.patch | 2.23 KB | ezra-g |
Comments
Comment #1
anarcat commentedComment #2
anarcat commentedComment #3
ezra-g commentedI 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 :)
Comment #4
ezra-g commentedActually, there are a bunch of other ones I just noticed after submitting decision nodes.
Comment #5
ezra-g commentedFound a few more. I feel like we can fix these as more come up.
Comment #6
anarcat commentedI can't review this today, I'll try to take a look next week, thanks!
Comment #7
anarcat commentedFix committed. Reopen if new ones are found (even without a patch).