View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
07763 | Bug reports | Other | public | 2013-04-12 14:57 | 2013-05-14 14:38 |
Reporter | digmxl | Assigned To | DenisChenu | ||
Priority | low | Severity | minor | ||
Status | closed | Resolution | fixed | ||
Product Version | 2.00+ | ||||
Target Version | 2.00+ | Fixed in Version | 2.00+ | ||
Summary | 07763: Answers not saved in database | ||||
Description | First set of questions (Q280a,Q290a, ... Q380a) are never saved in database | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
Bug heat | 8 | ||||
Complete LimeSurvey version number (& build) | 130406 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | |||||
Database type & version | mysql 5.5 | ||||
Server OS (if known) | linux | ||||
Webserver software & version (if known) | apache2 | ||||
PHP Version | php5 | ||||
parent of | 07805 | closed | DenisChenu | Comparaison String and Numeric is different in same page and other page |
limesurvey_group_357.lsg : group of questions causing the issue |
|
The attached survey is way too big. Please try to reduce the survey to the smallest size possible where the issue is still reproducable. |
|
limesurvey_survey_256293.lss : added smallest one |
|
This is partially a problem in your survey. The result of a numerical question is a number. So you need to compare for one. Currently you are comparing for a string: ((( ! is_empty(Q280a.NAOK) && (Q280a.NAOK < "1995")))) should be ((( ! is_empty(Q280a.NAOK) && (Q280a.NAOK < 1995)))) instead. Denis, do you see a way to 'unify' the behaviour in PHP and JS? |
|
I try to have a look, but not sure to find a goos solution for every problem. Here : Choose js solution or php solution ? Denis |
|
I think the PHP solution would be the better one. |
|
OK, But: Then think EM make something like ISSTRING/ISNUMBER somewhere. Review this. |
|
Hi, |
|
Hello, It's not a comparaison problem. When you use condition GUI, you use answer code and not answer text. For your problem: You need to update answer code or use Q270.shown ..... But still a minor issue with |
|
Here the problem is: With javascript do Think the equation is bad: See ANSWERCODE ( Answer text) and not Answer text |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=12361 |
|
Fix committed to 2.05 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=12362 |
|
Fix condition GUI editor: more clean. Fix for js/php comparaison can break a lot of survey (look at isnum function at EM core ...) |
|
If it can break alot of surveys we should do it for 2.05 only, not for 2.0 |
|
Maybe, but there are a lot of function for stroing to number in PHP if you look at em core. Here : the code is bad. |
|
em_core_helper.php: Line 277: Some line after: Maybe have a : But what for "A1">1 : it's false in PHP ? But in JS ? Denis New bug : http://bugs.limesurvey.org/view.php?id=7805 but need discussion. |
|
Thank you for your answers, your time and your support ! |
|
Version 2.0 Build 130514 released |
|
LimeSurvey: master b0515e62 2013-04-29 12:43 Details Diff |
Fixed issue 07763: Better information display in condition editor gui Dev: condition editor GUI use code and not answer, then show code Dev: Leave a problem with number/string comparaison : really don't know how to fix it |
Affected Issues 07763 |
|
mod - scripts/jquery/lime-conditions-tabs.js | Diff File | ||
LimeSurvey: 2.05 5607b619 2013-04-29 12:50 Details Diff |
Fixed issue 07763: Better information display in condition editor gui Dev: condition editor GUI use code and not answer, then show code Dev: Leave a problem with number/string comparaison : really don't know how to fix it Dev: strangly Answers array are Answer array in 205 |
Affected Issues 07763 |
|
mod - application/controllers/admin/conditionsaction.php | Diff File | ||
mod - scripts/admin/conditions.js | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2013-04-12 14:57 | digmxl | New Issue | |
2013-04-12 14:57 | digmxl | File Added: limesurvey_survey_525791.lss | |
2013-04-12 16:47 | mfaber | Issue Monitored: mfaber | |
2013-04-12 21:45 | digmxl | File Added: limesurvey_group_357.lsg | |
2013-04-12 21:45 | digmxl | Note Added: 24955 | |
2013-04-12 21:48 | digmxl | Note Edited: 24955 | |
2013-04-15 12:16 | digmxl | File Added: survey.jpg | |
2013-04-15 12:17 | digmxl | File Added: pma.jpg | |
2013-04-15 17:33 | c_schmitz | Note Added: 24957 | |
2013-04-15 17:33 | c_schmitz | Assigned To | => c_schmitz |
2013-04-15 17:33 | c_schmitz | Status | new => feedback |
2013-04-15 17:41 | digmxl | File Added: limesurvey_survey_256293.lss | |
2013-04-15 17:42 | digmxl | Note Added: 24958 | |
2013-04-15 17:42 | digmxl | Status | feedback => assigned |
2013-04-22 14:22 | c_schmitz | File Deleted: limesurvey_survey_525791.lss | |
2013-04-22 14:22 | c_schmitz | File Deleted: limesurvey_group_357.lsg | |
2013-04-22 14:34 | c_schmitz | Assigned To | c_schmitz => DenisChenu |
2013-04-22 14:37 | c_schmitz | Note Added: 25039 | |
2013-04-22 15:07 | DenisChenu | Note Added: 25045 | |
2013-04-24 01:24 | c_schmitz | Note Added: 25101 | |
2013-04-24 10:06 | DenisChenu | Note Added: 25104 | |
2013-04-24 12:06 | digmxl | Note Added: 25114 | |
2013-04-29 12:16 | DenisChenu | Note Added: 25156 | |
2013-04-29 12:32 | DenisChenu | Note Added: 25157 | |
2013-04-29 12:44 | DenisChenu | Changeset attached | => LimeSurvey master b0515e62 |
2013-04-29 12:44 | DenisChenu | Note Added: 25158 | |
2013-04-29 12:44 | DenisChenu | Resolution | open => fixed |
2013-04-29 12:52 | DenisChenu | Changeset attached | => LimeSurvey 2.05 5607b619 |
2013-04-29 12:52 | DenisChenu | Note Added: 25159 | |
2013-04-29 12:53 | DenisChenu | Note Added: 25160 | |
2013-04-29 12:53 | DenisChenu | Status | assigned => resolved |
2013-04-29 12:54 | DenisChenu | Severity | partial_block => minor |
2013-04-29 12:54 | DenisChenu | Fixed in Version | => 2.00+ |
2013-04-29 12:54 | DenisChenu | Target Version | => 2.00+ |
2013-04-29 15:04 | DenisChenu | Priority | high => low |
2013-04-29 15:15 | c_schmitz | Note Added: 25161 | |
2013-04-29 15:17 | DenisChenu | Note Added: 25162 | |
2013-04-29 15:29 | DenisChenu | Note Added: 25163 | |
2013-04-29 16:12 | DenisChenu | Relationship added | parent of 07805 |
2013-04-29 16:19 | DenisChenu | Note Edited: 25163 | |
2013-05-07 11:57 | digmxl | Note Added: 25188 | |
2013-05-14 14:38 | c_schmitz | Note Added: 25258 | |
2013-05-14 14:38 | c_schmitz | Status | resolved => closed |