View Issue Details

This bug affects 1 person(s).
 6
IDProjectCategoryView StatusLast Update
19881Feature requestsRemoteControlpublic2024-12-20 17:47
Reportertammo Assigned Totibor.pacalat  
PrioritynoneSeverityfeature 
Status ready for testingResolutionopen 
Summary19881: Make survey group ID available as output or input for RPC
Description

RPC:

  • when asking for list_surveys, there is no information of survey group ID
  • when asking for survey_properties, there is no information on survey group ID
  • when asking for survey group ID's, there is no information on connected surveys.

In other words: in RPC there is no way of finding out to which survey group a survey belongs.

Additional Information

Please make the following available:

  1. when asking for list_surveys, include the survey_group ID
  2. when asking for list_surveys, please add the possibility to only ask for surveys that belong to a specific survey group.
  3. when asking for survey_properties, please include the survey group that the survey belongs to.
TagsNo tags attached.
Bug heat6
Story point estimate0
Users affected %0

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2024-12-19 13:10

developer   ~81666

when asking for survey_properties, please include the survey group that the survey belongs to.

You have the gsid , you want group name ? in a survey group array or ?

Or maybe better : a get_surveysgroup_properties ?

DenisChenu

DenisChenu

2024-12-19 13:11

developer   ~81667

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

tammo

tammo

2024-12-19 13:42

partner   ~81668

Fastest: include gsid
Better: include gsid and survey group name in array.

But anyway: I am already content with gsid in list_surveys AND gsid in survey_properties

tammo

tammo

2024-12-19 13:43

partner   ~81669

Bonus would be to be able to make something like list_surveys(gsid=2)

DenisChenu

DenisChenu

2024-12-19 15:14

developer   ~81670

Fastest: include gsid

Already here (in survey_properties)

Better: include gsid and survey group name in array.

No : better have a get_surveysgroup_properties then you can ask for gsid and get the gsid one time if you ask for 10 surveys.

All is here now.

DenisChenu

DenisChenu

2024-12-19 15:22

developer   ~81671

Fastest: include gsid

tammo

tammo

2024-12-19 15:32

partner   ~81673

Great! Thank you.

tibor.pacalat

tibor.pacalat

2024-12-20 15:40

administrator   ~81705

@DenisChenu there are a couple of issues:

  1. I am getting an error if I use both optional params $sUsername and $gsid: "CDbCommand failed to execute the SQL statement: SQLSTATE[23000]: Integrity constraint violation: 1052 Column 'gsid' in where clause is ambiguous...."
  2. The documentation at /index.php/admin/remotecontrol is missing "gsid" under "Returns array with"
DenisChenu

DenisChenu

2024-12-20 16:32

developer   ~81709

Good catch !

DenisChenu

DenisChenu

2024-12-20 17:14

developer   ~81714

I am getting an error if I use both optional params $sUsername and $gsid: "CDbCommand failed to execute the SQL statement: SQLSTATE[23000]: Integrity constraint violation: 1052 Column 'gsid' in where clause is ambiguous...."

:( I can not reproduce and no gsid in langagesettings or user …

tibor.pacalat

tibor.pacalat

2024-12-20 17:17

administrator   ~81715

@DenisChenu I just understood that you can not reproduce the error. But what do you mean by "and no gsid in langagesettings or user …"? I will try to see why this happens to me.

DenisChenu

DenisChenu

2024-12-20 17:46

developer   ~81716

I mean : join is done with lime_users and lime_surveys_languagesettings

I don't found a second gsid here.

PS : i have an update version of jsonrpcphp to show the error send by LimeSurvey : you want it ?

DenisChenu

DenisChenu

2024-12-20 17:46

developer   ~81717

https://github.com/LimeSurvey/LimeSurvey/pull/4115/commits/cbd4a09f7a000a6e7709f0a0ba02686d55df520b

DenisChenu

DenisChenu

2024-12-20 17:47

developer   ~81718

Issue History

Date Modified Username Field Change
2024-12-06 19:40 tammo New Issue
2024-12-07 13:07 DenisChenu Assigned To => DenisChenu
2024-12-07 13:07 DenisChenu Status new => assigned
2024-12-11 03:57 violationquaint Bug heat 0 => 2
2024-12-11 08:48 DenisChenu Bug heat 2 => 0
2024-12-19 13:10 DenisChenu Note Added: 81666
2024-12-19 13:10 DenisChenu Bug heat 0 => 2
2024-12-19 13:11 DenisChenu Note Added: 81667
2024-12-19 13:42 tammo Note Added: 81668
2024-12-19 13:42 tammo Bug heat 2 => 4
2024-12-19 13:43 tammo Note Added: 81669
2024-12-19 15:14 DenisChenu Note Added: 81670
2024-12-19 15:15 DenisChenu Assigned To DenisChenu => gabrieljenik
2024-12-19 15:15 DenisChenu Status assigned => ready for code review
2024-12-19 15:22 DenisChenu Note Added: 81671
2024-12-19 15:22 DenisChenu File Added: Capture d’écran du 2024-12-19 15-22-01.png
2024-12-19 15:32 tammo Note Added: 81673
2024-12-19 15:36 gabrieljenik Assigned To gabrieljenik => tibor.pacalat
2024-12-19 15:36 gabrieljenik Status ready for code review => ready for testing
2024-12-20 15:40 tibor.pacalat Note Added: 81705
2024-12-20 15:40 tibor.pacalat File Added: Screenshot 2024-12-20 at 15.36.27.png
2024-12-20 15:40 tibor.pacalat Bug heat 4 => 6
2024-12-20 15:41 tibor.pacalat Assigned To tibor.pacalat => DenisChenu
2024-12-20 16:32 DenisChenu Note Added: 81709
2024-12-20 17:14 DenisChenu Note Added: 81714
2024-12-20 17:17 tibor.pacalat Note Added: 81715
2024-12-20 17:46 DenisChenu Note Added: 81716
2024-12-20 17:46 DenisChenu Note Added: 81717
2024-12-20 17:47 DenisChenu Assigned To DenisChenu => tibor.pacalat
2024-12-20 17:47 DenisChenu Note Added: 81718
2024-12-20 17:47 DenisChenu File Added: Capture d’écran du 2024-12-20 17-45-03.png
2024-12-20 17:47 DenisChenu File Added: Capture d’écran du 2024-12-20 17-45-10.png