View Issue Details

Related ChangesetsJump to NotesJump to History
This bug affects 1 person(s).
 4
IDProjectCategoryView StatusDate SubmittedLast Update
10102Bug reportsOtherpublic2015-12-10 14:102016-01-22 08:54
Reportersieutruc Assigned ToDenisChenu  
PrioritynormalSeverityblock 
Status closedResolutionfixed 
Product Version2.06+ 
Fixed in Version2.06+ 
Summary10102: reuse the master field map in case of activating a survey with at least one randomization group
Description

Create a survey with 2 groups, at least one of them is randomization group.

Deactivating and reactivating this survey with the "Save IP address" changed, causes an error when opening the survey.

Error: surveydynamic.ipaddr is not defined . Check the table of that survey in the database, " describe limesurvey[surveynumber]", this table does not have the field "ipaddr".

Steps To Reproduce

Create a new survey with the "Save IP address" value to "No", create 5 tokens, do one or two survey.

(There is at least one randomization group)

Deactivate that survey and reactive it with the "Save IP address" value to "Yes", restore the initial token table, and do one survey from that table. ----- > Error appears

Additional Information

The bug is also reported on github. Here the link is:

https://github.com/LimeSurvey/LimeSurvey/pull/395

TagsNo tags attached.
Attached Files
limesurvey_survey_918628.lss (15,082 bytes)
Bug heat4
Complete LimeSurvey version number (& build)e9fa25c676
I will donate to the project if issue is resolvedNo
Browser
Database type & versionMariaDB 10.0.20
Server OS (if known)Fedora 20
Webserver software & version (if known)Apache 2.3
PHP VersionPHP 5.5.26

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2015-12-10 14:15

developer   ~33861

Can confirm the issue.

Think the last solution is the better : completely removing ALL session information when deactivate.
DenisChenu

DenisChenu

2015-12-10 14:22

developer   ~33862

@sieutruc : if you can today : can you redo the commit with commit message:
Fixed issue : reactivating a randomized survey can break survey table
or something more clear but with
Fixed issue : blah
or better
Fixed issue 10102 : blah blah
DenisChenu

DenisChenu

2015-12-17 14:25

developer   ~34045

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=16626
sieutruc

sieutruc

2015-12-18 09:30

reporter   ~34054

@DenisChenu : sorry for my late reply, I saw that you merged that change, do i still modify the commit message ?
DenisChenu

DenisChenu

2015-12-18 10:08

developer   ~34055

Not needed now, just for next pull then ;).

Thanks for all

Related Changesets

LimeSurvey: master a89ed2fe

2015-12-17 14:24

DenisChenu


Details Diff		 Merge pull request #395 from sieutruc/wrongFieldMap

Fixed issue 10102, deactivate , update seetings, reactivate a randomized survey can break Database		 Affected Issues
10102
mod - application/controllers/admin/surveyadmin.php Diff File

Issue History

Date Modified Username Field Change
2015-12-10 14:10 sieutruc New Issue
2015-12-10 14:10 sieutruc File Added: limesurvey_survey_918628.lss
2015-12-10 14:15 DenisChenu Note Added: 33861
2015-12-10 14:20 DenisChenu Assigned To => DenisChenu
2015-12-10 14:20 DenisChenu Status new => assigned
2015-12-10 14:22 DenisChenu Note Added: 33862
2015-12-10 14:22 DenisChenu Status assigned => feedback
2015-12-17 14:25 DenisChenu Changeset attached => LimeSurvey master a89ed2fe
2015-12-17 14:25 DenisChenu Note Added: 34045
2015-12-17 14:25 DenisChenu Resolution open => fixed
2015-12-18 08:56 DenisChenu Status feedback => resolved
2015-12-18 08:56 DenisChenu Fixed in Version => 2.06+
2015-12-18 09:30 sieutruc Note Added: 34054
2015-12-18 10:08 DenisChenu Note Added: 34055
2016-01-22 08:54 c_schmitz Status resolved => closed