View Issue Details

Related ChangesetsJump to NotesJump to History
This bug affects 1 person(s).
 4
IDProjectCategoryView StatusDate SubmittedLast Update
06827Bug reportsSurvey participants (Tokens)public2012-11-04 19:432012-11-06 12:52
ReporterDenisChenu Assigned ToDenisChenu  
PrioritynormalSeveritypartial_block 
Status closedResolutionfixed 
Product Version2.00+ 
Fixed in Version2.00+ 
Summary06827: Can not link between survey response and token
Description

If a survey have existing token table before activation, then we can not link between survey response and survey token

Steps To Reproduce

Create a survey,
Create a token table for this survey
Activate survey
Create a token and launch the token (more than 5 caracter for token)

The link between token and survey can not be done.

In survey table the token are varchar(5)

Additional Information

in activate_helper: function activateSurvey
f there are no token table : token colum are created before with $createsurvey['token'] = "VARCHAR(36)"; (line 267)

But if the survey table exist then token are in $fieldmap and default is set to VARCHAR(5).

TagsNo tags attached.
Bug heat4
Complete LimeSurvey version number (& build)121104
I will donate to the project if issue is resolvedNo
BrowserFF16
Database type & versionMysql 5
Server OS (if known)debian/linux
Webserver software & version (if known)apache
PHP VersionPHP Version 5.3

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2012-11-04 19:44

developer   ~21942

Think it's best to change some functionnality.

  • allways add token column in survey table even if survey is anonymous or not !
  • allways set token column at VARCHAR(36).

Can take it but tomorrow
DenisChenu

DenisChenu

2012-11-04 19:58

developer   ~21943

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

c_schmitz

2012-11-04 20:52

administrator   ~21948

Sigh, I reviewed this patch - it is okay. Denis, thank you!
Will do another release then :-/
c_schmitz

c_schmitz

2012-11-04 21:02

administrator   ~21950

New version released.

Related Changesets

LimeSurvey: master 51441555

2012-11-04 19:57

DenisChenu


Details Diff		 Fixed issue 06827: Can not link between survey response and token
Dev: token are already included in fieldmap, then don't set to default VARCHAR(5)
Dev: add token at end if it not already exist, maybe remove anonymized test is a good idea !		 Affected Issues
06827
mod - application/helpers/admin/activate_helper.php Diff File

Issue History

Date Modified Username Field Change
2012-11-04 19:43 DenisChenu New Issue
2012-11-04 19:44 DenisChenu Note Added: 21942
2012-11-04 19:58 DenisChenu Changeset attached => LimeSurvey master 51441555
2012-11-04 19:58 DenisChenu Note Added: 21943
2012-11-04 19:58 DenisChenu Assigned To => DenisChenu
2012-11-04 19:58 DenisChenu Resolution open => fixed
2012-11-04 20:20 c_schmitz Status new => feedback
2012-11-04 20:22 c_schmitz Assigned To DenisChenu => mdekker
2012-11-04 20:22 c_schmitz Status feedback => assigned
2012-11-04 20:52 c_schmitz Note Added: 21948
2012-11-04 20:52 c_schmitz Status assigned => resolved
2012-11-04 20:52 c_schmitz Fixed in Version => 2.00+
2012-11-04 21:02 c_schmitz Note Added: 21950
2012-11-04 21:02 c_schmitz Status resolved => closed
2012-11-04 23:12 mdekker Assigned To mdekker => c_schmitz
2012-11-04 23:12 mdekker Status closed => feedback
2012-11-04 23:12 mdekker Resolution fixed => reopened
2012-11-06 12:48 DenisChenu Status feedback => resolved
2012-11-06 12:48 DenisChenu Resolution reopened => fixed
2012-11-06 12:48 DenisChenu Assigned To c_schmitz => DenisChenu
2012-11-06 12:52 c_schmitz Status resolved => closed
2016-12-08 10:39 c_schmitz Category Tokens => Survey participants (Tokens)