Closed (fixed)
Project:
Advanced Poll
Version:
5.x-1.0-beta2
Component:
Miscellaneous
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
11 May 2007 at 00:18 UTC
Updated:
25 May 2007 at 06:03 UTC
Jump to comment: Most recent file
Comments
Comment #1
ChrisKennedy commentedThese are anonymous users or logged in users? And what version of VotingAPI are you using?
Comment #2
RFree190 commentedWOW! You guys are fast...
These are logged in users (My site is closed to non registered users)
and I am using VotingAPI 5.x 1.5
They are using office computers, which share the same IP address.
Thanks for your help!
Comment #3
RFree190 commentedFunny... As I was typing my last message, I noticed that Beta4 was added to the site. I've downloaded and installed, but still same results. One vote and the poll is closed.
Comment #4
ChrisKennedy commentedNormally I'm a lot slower to respond, but I happened to be working on advpoll right now.
I believe this IP level checking was added to logged-in users by http://drupal.org/node/107990
Attached patch would revert this change - does it fix things for you? Run "patch -p0 < advpoll_ipchecking.patch" in the advpoll directory to apply it.
Comment #5
RFree190 commentedI downloaded the attached patch. I'm kinda new to Drupal. How do I run the patch? Do I call it from my browser? By typing my url + /modules/advpoll/advpoll_ipchecking.patch?
Thanks for your quick response on my issue. I really, really appreciate it.
Comment #6
ChrisKennedy commentedOkay try this zipped version instead: http://advpoll-dev.ck37.com/advpoll.zip
Comment #7
RFree190 commentedChris,
Thank you so much! Works great.
I really appreciate how quickly you responded and kept on my issue until it was solved.
I know I made the right decision in choosing Drupal. Customer service means a lot!!!
Thanks again.
Comment #8
ChrisKennedy commentedNo problem, although response times do vary a lot, in this project and others.
I went ahead and committed the fix since it's an important bug. http://drupal.org/cvs?commit=67131
It looks like eligibility checking could use a more thorough examination and probably a configurable option or two.
Comment #9
(not verified) commented