comparing a uid to nid is fun but won't lead to anything good, i guess.

Edit: I am writing a simpletest. Again and again and again, write those and you will rewarded richly with bugs.

CommentFileSizeAuthor
#3 poll_notice_followup.patch614 byteschx
#1 poll_opsie.patch1.5 KBchx
poll_opsie.patch1.33 KBchx
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chx’s picture

chx CreditAttribution: chx commented
FileSize
poll_opsie.patch1.5 KB

Better patchy. Note that I am fixing notices wherever I find them.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Good catch! Committed to CVS HEAD.

chx’s picture

chx CreditAttribution: chx commented
Title: Poll access check is braindead » Poll notice fix changed behaviour
Status: Fixed » Reviewed & tested by the community
FileSize
poll_notice_followup.patch614 bytes

I am so sorry. An unset array index is NULL not 0. And though 0 is more adequate here but that's not what the original code did and past RC it's not nice to change behaviour. But now the simpletest is working so I do not expect further trouble.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Reviewed & tested by the community » Fixed

Checked the poll bar default value, and it should indeed be NULL here. Thanks, committed.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)

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