| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 06080 | Bug reports | Survey taking | public | 2012-05-10 11:50 | 2012-05-17 10:27 |
| Reporter | Mazi | Assigned To | DenisChenu | ||
| Priority | normal | Severity | minor | ||
| Status | closed | Resolution | fixed | ||
| Product Version | 1.92+ | ||||
| Target Version | 1.92+ | Fixed in Version | 1.92+ | ||
| Summary | 06080: Wrong validation for multiple numeric questions | ||||
| Description | When setting the following question attribute at a multiple numeric question, EM marks them as valid on page load (which is wrong), while other fields are correctly color coded red:
Once you enter a value, the clor coding is corrected. | ||||
| Steps To Reproduce | Import the attached survey, activate and run it | ||||
| Additional Information | See attached screen shots | ||||
| Tags | No tags attached. | ||||
| Bug heat | 6 | ||||
| Complete LimeSurvey version number (& build) | 120510 | ||||
| I will donate to the project if issue is resolved | No | ||||
| Browser | Firefox 12 | ||||
| Database type & version | MySQL 5 | ||||
| Server OS (if known) | Win 7 | ||||
| Webserver software & version (if known) | Apache 2.2 | ||||
| PHP Version | 5.3.1 | ||||
 |
Tom, not sure if you are still available for some little bugfixing. So if not, please assign this one to Carsten. |
 |
Mazi, For 2 multiNumInput2 Multiple numerical input 2 : It's OK because question isn't mandatory: Yes ? Then put error on dynamic_sum only if question is mandatory is right : OK ? Denis |
|
|
Hmm, we need to think about how to handle mandatory questions. Even if a question is NOT mandatory, I think we shouldn't mark the text green. Instead of marking it red, we could just leave it black while for mandatory questions we should mark ti red because the user has to answer the question so validation will always be taken into account. |
|
|
minimum answers => question is "pseudo mandatory" question validation equation : we can't say ... depend on the equation. I add questionInfo['mandatory'] on this part, we can work after on other "bug". But some "bug" can very difficult to manage: exception of exception of exception .... etc. |
|
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=8394 |
|
|
Fix committed to Yii branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=8395 |
 |
New 1.92+ version released. |
