View Issue Details

IDProjectCategoryView StatusLast Update
11576Bug reports[All Projects] Central participant databasepublic2016-08-24 13:53
ReporterDenisChenuAssigned ToDenisChenu 
PrioritynoneSeverityminor 
Status assignedResolutionopen 
Product Version2.50.x 
Target VersionFixed in Version 
Summary11576: Some participants throw jquery error
Description

Sometimes : the Particpant option are not showned.

You need to have a participant_id starting by a . (dot) or maybe other caracter.

Steps To Reproduce

Create a participant,
Add some attribute
Go to SQL DB and update is participant_id to .qdjsh.sqhsdg767 for example (can happen).
Test to see or update attribute of participant.

Additional Information

The participant_id is used for id of tr : this broke if participant_id start by dot (or number, or maybe in other condition).
Think best is to use data-participantid for javascript usage ?

Error shown in console:
Error: Syntax error, unrecognized expression: tr#.IBJ.8EfqUSSLOVxVgQJbA
http://.../tmp/assets/800abd00/jquery-2.2.4.min.js
Line 2

TagsNo tags attached.
Complete LimeSurvey version number (& build)160823
I will donate to the project if issue is resolvedNo
BrowserFF45
Database & DB-Versionmysql 5
Server OS (if known)debian8
Webserver software & version (if known)apache
PHP VersionPHP Version 5.6

Activities

DenisChenu

DenisChenu

2016-08-23 18:20

developer  

c_schmitz

c_schmitz

2016-08-24 10:51

administrator   ~40418

I am sorry but I don't see how gen_uuid() in the Participant model would be able to create an ID starting with a '.' ?

DenisChenu

DenisChenu

2016-08-24 11:30

developer   ~40423

Strange , i have it on a clean install of a 2.50 version : https://github.com/LimeSurvey/LimeSurvey/blob/master/application/models/Participant.php#L134 can produce dot ?

Or maybe we don't use everywhere this function ?
But , i must ask to user if he use an API maybe ?

A question : https://github.com/LimeSurvey/LimeSurvey/blob/master/application/models/Participant.php#L134 produce only id valid caracter ? I didn't realy understand it ;).

PS : the data-partcicipantid fix the solution with all caracter.

c_schmitz

c_schmitz

2016-08-24 12:04

administrator   ~40429

They are probably using their own UIDs.
We could enforce it to use only valid UUIDs but on the other hand I am not sure if that is necessary and would be too restricting.

Can you apply the fix to use the participant table auto ID instead?

DenisChenu

DenisChenu

2016-08-24 12:06

developer   ~40430

There are no autoid in Particpant table. I validate if it was created with external tools. And see if we can update the javascript to allow any particiapnt_id and make a pull request.

c_schmitz

c_schmitz

2016-08-24 12:26

administrator   ~40433

Ok, please do - thank you!

DenisChenu

DenisChenu

2016-08-24 13:53

developer   ~40439

Confirm : not created via LS

Issue History

Date Modified Username Field Change
2016-08-23 18:20 DenisChenu New Issue
2016-08-23 18:20 DenisChenu File Added: Capture du 2016-08-23 18-06-32.png
2016-08-24 10:51 c_schmitz Note Added: 40418
2016-08-24 10:51 c_schmitz Assigned To => c_schmitz
2016-08-24 10:51 c_schmitz Status new => feedback
2016-08-24 11:30 DenisChenu Note Added: 40423
2016-08-24 11:30 DenisChenu Status feedback => assigned
2016-08-24 12:04 c_schmitz Note Added: 40429
2016-08-24 12:06 DenisChenu Note Added: 40430
2016-08-24 12:26 c_schmitz Note Added: 40433
2016-08-24 12:26 c_schmitz Assigned To c_schmitz => DenisChenu
2016-08-24 13:53 DenisChenu Note Added: 40439