View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
05897 | Bug reports | Other | public | 2012-03-11 01:41 | 2012-03-14 18:14 |
Reporter | sergey | Assigned To | TMSWhite | ||
Priority | normal | Severity | minor | ||
Status | closed | Resolution | fixed | ||
Fixed in Version | 1.92+ | ||||
Summary | 05897: Survey ID not set correctly after row insert in servey_NNN table | ||||
Description | People who come to fill out a survey for the first time are greeted with an error message after they try to click Next on the first page. Their answers can't be saved because survey id is not retrieved from the database correctly. | ||||
Additional Information | In the latest code (1.92 build 20120303) the following line is wrong: $srid = $connect->Insert_ID($this->surveyOptions['tablename'],"sid"); It should be "id" not "sid" as the last parameter. The line (in "dev" branch) is 3279: https://github.com/LimeSurvey/LimeSurvey/blob/dev/classes/eval/LimeExpressionManager.php#L3279. | ||||
Tags | No tags attached. | ||||
Bug heat | 4 | ||||
Complete LimeSurvey version number (& build) | 201203 | ||||
I will donate to the project if issue is resolved | |||||
Browser | |||||
Database type & version | |||||
Server OS (if known) | |||||
Webserver software & version (if known) | |||||
PHP Version | |||||
Fixed with commit https://github.com/LimeSurvey/LimeSurvey/commit/d7cffd0a1f97f54f44ed0704a320efe31bf71af4 Interestingly, this is exactly how it has always been done in 1.91+ too. That argument is ignored in all databases except Postgres, so this must be a Postgres-only issue. |
|
1.92+ Build 120211 released |
|
LimeSurvey: master d7cffd0a 2012-03-10 18:54:23 Details Diff |
Fixed issue 05897: Survey ID not set correctly after row insert in servey_NNN table Dev this could only affect Postgres installations, as that argument is ignored by all other database types. |
Affected Issues 05897 |
|
mod - classes/expressions/LimeExpressionManager.php | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2012-03-11 01:41 | sergey | New Issue | |
2012-03-11 03:52 | TMSWhite | Project | @2@ => Bug reports |
2012-03-11 03:52 | TMSWhite | Category | Survey at runtime => Other issues |
2012-03-11 03:54 | TMSWhite | Note Added: 17821 | |
2012-03-11 03:54 | TMSWhite | Status | new => resolved |
2012-03-11 03:54 | TMSWhite | Fixed in Version | => 1.92+ |
2012-03-11 03:54 | TMSWhite | Resolution | open => fixed |
2012-03-11 03:54 | TMSWhite | Assigned To | => TMSWhite |
2012-03-11 16:31 | c_schmitz | Note Added: 17822 | |
2012-03-11 16:31 | c_schmitz | Status | resolved => closed |
2012-03-14 18:14 | TMSWhite | Changeset attached | => Import 2012-03-09 13:30:34 master d7cffd0a |