View Issue Details

This bug affects 1 person(s).
 4
IDProjectCategoryView StatusLast Update
17474Bug reportsRemoteControlpublic2021-10-26 16:35
Reporterarnaudbore Assigned Toc_schmitz  
PrioritynoneSeveritypartial_block 
Status feedbackResolutionopen 
Product Version3.25.20 
Summary17474: Unable to update token value of response with remote control
Description

When using set_participant_properties if token is modified related responses will not be linked to a participant anymore.

Steps To Reproduce

Using remote control

Create a participant
Answer response
Modify token using set_participant_properties
Related answer is now unlinked

TagsNo tags attached.
Bug heat4
Complete LimeSurvey version number (& build)3.27.7+210713
I will donate to the project if issue is resolvedNo
Browser
Database type & version10.6.3-MariaDB-1:10.6.3+maria~focal
Server OS (if known)
Webserver software & version (if known)
PHP Version7.4.16

Users monitoring this issue

There are no users monitoring this issue.

Activities

arnaudbore

arnaudbore

2021-07-25 06:09

reporter   ~65606

I also tried update_response with an array (id of the response and the new token) but the new token does not update.

The idea was to do the propagation in three steps.

  1. get_response_id (using old token)
  2. update_response (using ids from get_response_id and try to update token)
  3. set_participant_properties with new token (query done with old token)
DenisChenu

DenisChenu

2021-07-25 11:51

developer   ~65610

Last edited: 2021-07-29 08:56

Same in GUI : not an issue ? Desired behavior in my opinion.

arnaudbore

arnaudbore

2021-07-25 14:03

reporter   ~65611

Last edited: 2021-07-29 08:56

If it's the desire behavior, Is there a way using the remote control function to update the token of a response ?

arnaudbore

arnaudbore

2021-07-25 14:37

reporter   ~65612

Last edited: 2021-07-29 08:56

Since it does not seem to be an issue a follow-up on this here: https://forums.limesurvey.org/forum/can-i-do-this-with-limesurvey/125048-update-response-token-using-remote-control-functions

DenisChenu

DenisChenu

2021-07-25 17:57

developer   ~65613

Last edited: 2021-07-29 08:56

My opinion : update_response must allow updating «token»
If it's the case : no issue.

arnaudbore

arnaudbore

2021-07-25 18:12

reporter   ~65614

Last edited: 2021-07-29 08:56

So far I was not able to make it run maybe I forget something or I'm doing it wrong. Can somebody validate that it should be working ?

DenisChenu

DenisChenu

2021-07-25 18:29

developer   ~65615

Last edited: 2021-07-29 08:56

  1. update_response (using ids from get_response_id and try to update token)

This seems to work ?
Can you tell us the instruction tested ?

  1. set_participant_properties with new token (query done with old token)

Can you tell us the function tested ?

arnaudbore

arnaudbore

2021-07-25 18:39

reporter   ~65616

Last edited: 2021-07-29 08:56

2 Update response
$aResponseModifyed = array('id' => $id_response, 'token' => $next_multi_token);
$updatedResponse = $lsJSONRPCClient->update_response($sSessionKey, $surveyID, $aResponseModifyed);

  1. set_participant_properties with new token (query done with old token)
    $aTokenData = array('token' => $next_token);
    $aQuery = array('token' => $current_token);
    $updated_participant = $lsJSONRPCClient->set_participant_properties($sSessionKey, $surveyID, $aQuery, $aTokenData);
arnaudbore

arnaudbore

2021-07-25 19:11

reporter   ~65617

Last edited: 2021-07-29 08:56

2 not working
3 working

DenisChenu

DenisChenu

2021-07-26 08:56

developer   ~65618

Last edited: 2021-07-29 08:56

OK :
Confirm the issue : https://github.com/LimeSurvey/LimeSurvey/blob/957bf9a0c18fffb0bfc724e4ba8617046ebbe179/application/helpers/remotecontrol/remotecontrol_handle.php#L2973

OK to rename the title to «Unable to update token value of response with remote control» ?

arnaudbore

arnaudbore

2021-07-26 12:45

reporter   ~65621

Last edited: 2021-07-29 08:56

I don't see how to rename the title. I will suggest a PR. Can we agree that modifying only the token should not be dependant on whether or not we have the permission to update responses ? In my opinion token is not a response to a survey.

arnaudbore

arnaudbore

2021-07-26 13:05

reporter   ~65622

Last edited: 2021-07-29 08:56

PR sent: https://github.com/LimeSurvey/LimeSurvey/pull/1981

Issue History

Date Modified Username Field Change
2021-07-24 20:40 arnaudbore New Issue
2021-07-25 06:09 arnaudbore Note Added: 65606
2021-07-25 11:51 DenisChenu Note Added: 65610
2021-07-25 14:03 arnaudbore Note Added: 65611
2021-07-25 14:37 arnaudbore Note Added: 65612
2021-07-25 17:57 DenisChenu Note Added: 65613
2021-07-25 18:12 arnaudbore Note Added: 65614
2021-07-25 18:29 DenisChenu Note Added: 65615
2021-07-25 18:39 arnaudbore Note Added: 65616
2021-07-25 19:11 arnaudbore Note Added: 65617
2021-07-26 08:56 DenisChenu Note Added: 65618
2021-07-26 12:45 arnaudbore Note Added: 65621
2021-07-26 13:05 arnaudbore Note Added: 65622
2021-07-26 13:30 galads Assigned To => galads
2021-07-26 13:30 galads Status new => ready for testing
2021-07-29 08:43 galads Status ready for testing => confirmed
2021-07-29 08:47 galads Status confirmed => ready for testing
2021-07-29 08:56 galads Sync to Zoho Project => |Yes|
2021-10-26 15:59 c_schmitz Summary set_participant_properties does not propagate to responses => Unable to update token value of response with remote control
2021-10-26 15:59 c_schmitz Zoho Project Synchronization Yes => |Yes|
2021-10-26 16:35 c_schmitz Assigned To galads => c_schmitz
2021-10-26 16:35 c_schmitz Status ready for testing => feedback