When you setup a pollfield the "starting votes count" allows you to put in non-numerical information without throwing an error or stopping submission. I put "4z" in the "starting votes count" field when creating a poll and the submission went through without an error. It did strip the string down to just the number 4.

Let's write some error code on that bit of content submission and halt the submission with a message if non-numeric data is entered into that field. Does that seem like the right way forward?

Comments

mario_prkos’s picture

mario_prkos commented

Yes, you are right. I am aware of this issue I will put some validation function.
Thank you for testing this module, it is very helpful. It is also important to achive good quality of module and good quality of drupal in general.

dddave’s picture

dddave commented
Version: 6.x-1.x-dev » 6.x-1.11-beta1

Still present. Although the current behavior is quite sane (only displaying numbers and stripping all the rest) a message would be nice. Or is this fixed in the latest dev?

nibblebot’s picture

nibblebot commented

status?

bryan kennedy’s picture

bryan kennedy commented
Version: 6.x-1.11-beta1 » 6.x-1.x-dev
Assigned: Unassigned » bryan kennedy
Priority: Minor » Major
Issue tags: +error handling

Raising priority on this one. Not yet fixed in the latest dev.

bryan kennedy’s picture

bryan kennedy commented
Status: Active » Fixed

I just committed a change to the dev branch that should fix this issue. We need to consider the validation differently once we refactor the module further, but this address the issue from a user perspective now.

Status: Fixed » Closed (fixed)
Issue tags: -form validation, -error handling

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

Issue tags: +form validation, +error handling

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.