View Issue Details

This bug affects 1 person(s).
 4
IDProjectCategoryView StatusLast Update
05897Bug reportsOtherpublic2012-03-14 18:14
Reportersergey Assigned ToTMSWhite  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Fixed in Version1.92+ 
Summary05897: 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.

TagsNo tags attached.
Bug heat4
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

Users monitoring this issue

There are no users monitoring this issue.

Activities

TMSWhite

TMSWhite

2012-03-11 03:54

reporter   ~17821

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.

c_schmitz

c_schmitz

2012-03-11 16:31

administrator   ~17822

1.92+ Build 120211 released

Related Changesets

LimeSurvey: master d7cffd0a

2012-03-10 18:54:23

TMSWhite

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

Issue History

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