View Issue Details

This bug affects 1 person(s).
 4
IDProjectCategoryView StatusLast Update
09790Feature requestsRemoteControlpublic2016-04-12 10:48
ReporterJanE Assigned Toc_schmitz  
PrioritynormalSeverityfeature 
Status closedResolutionfixed 
Summary09790: Use conditions to get a subset of list_participants
Description

list_participants does not have the ability to search for specified participants. If you want to check if someone is already registered, you will have to fetch the complete list and loop through the list to see if you have a match. This could take 'ages' if you have a high number of participants. This issue presents a method to search for specified participants, for instance by e-mail address.

The idea is to add $aConditions as extra, optional argument to list_participants() and use those conditions in the findall().

Additional Information

I will post a Pull Request shortly. I my use case (with 11000 participants) the response time of a 'is registered' check diminished by 6 seconds.

TagsNo tags attached.
Bug heat4
Story point estimate
Users affected %

Users monitoring this issue

There are no users monitoring this issue.

Activities

JanE

JanE

2015-07-22 14:53

reporter   ~32760

PR for the master branch:
https://github.com/LimeSurvey/LimeSurvey/pull/349

PR for the develop branch:
https://github.com/LimeSurvey/LimeSurvey/pull/350

JanE

JanE

2015-07-22 14:58

reporter   ~32761

Typical use case:

$aConditions = Array();
$aConditions[] = "email = 'info@example.com'";
$response = $myJSONRPCClient->list_participants(
    $sessionKey, $survey_id, 0, 1, false, false, $aConditions
);

This way you only check if there is one participant with the e-mail address info@example.com.

JanE

JanE

2015-08-05 08:54

reporter   ~32884

A week ago I created new PR's:

https://github.com/LimeSurvey/LimeSurvey/pull/355
https://github.com/LimeSurvey/LimeSurvey/pull/356

See this comment for a brief explanation:
https://github.com/LimeSurvey/LimeSurvey/pull/349#issuecomment-125626765

c_schmitz

c_schmitz

2015-08-14 10:51

administrator   ~32952

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

c_schmitz

c_schmitz

2015-08-14 10:51

administrator   ~32953

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

c_schmitz

c_schmitz

2015-08-14 10:56

administrator   ~32954

Great work, thank you.

c_schmitz

c_schmitz

2016-04-12 10:48

administrator   ~37250

Version 2.50+ Build 140413 released

Related Changesets

LimeSurvey: master 9b2f871b

2015-07-28 14:05:24

Jan-E

Details Diff
Fix 09790: Use conditions to get a subset of list_participants Affected Issues
09790
mod - application/helpers/remotecontrol/remotecontrol_handle.php Diff File

LimeSurvey: develop c21da9d5

2015-07-28 14:06:38

Jan-E

Details Diff
Fix 09790: Use conditions to get a subset of list_participants Affected Issues
09790
mod - application/helpers/remotecontrol/remotecontrol_handle.php Diff File

LimeSurvey: develop 3b9eb485

2015-08-14 08:51:26

c_schmitz

Details Diff
Merge pull request #356 from Jan-E/develop

Fix 09790: Use conditions to get a subset of list_participants
Affected Issues
09790
mod - application/helpers/remotecontrol/remotecontrol_handle.php Diff File

LimeSurvey: master 513110ba

2015-08-14 08:51:32

c_schmitz

Details Diff
Merge pull request #355 from Jan-E/master

Fix 09790: Use conditions to get a subset of list_participants
Affected Issues
09790
mod - application/helpers/remotecontrol/remotecontrol_handle.php Diff File

Issue History

Date Modified Username Field Change
2015-07-22 14:37 JanE New Issue
2015-07-22 14:53 JanE Note Added: 32760
2015-07-22 14:58 JanE Note Added: 32761
2015-08-05 08:54 JanE Note Added: 32884
2015-08-14 10:51 Changeset attached => LimeSurvey develop c21da9d5
2015-08-14 10:51 c_schmitz Changeset attached => LimeSurvey develop 3b9eb485
2015-08-14 10:51 c_schmitz Note Added: 32952
2015-08-14 10:51 c_schmitz Assigned To => c_schmitz
2015-08-14 10:51 c_schmitz Resolution open => fixed
2015-08-14 10:51 Changeset attached => LimeSurvey master 9b2f871b
2015-08-14 10:51 c_schmitz Changeset attached => LimeSurvey master 513110ba
2015-08-14 10:51 c_schmitz Note Added: 32953
2015-08-14 10:56 c_schmitz Note Added: 32954
2015-08-14 10:56 c_schmitz Status new => resolved
2015-08-14 10:56 c_schmitz Fixed in Version => 2.06+
2016-04-12 10:48 c_schmitz Note Added: 37250
2016-04-12 10:48 c_schmitz Status resolved => closed