| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 19617 | Bug reports | RemoteControl | public | 2024-06-18 08:25 | 2024-09-23 17:29 |
| Reporter | bpresentati | Assigned To | DenisChenu | ||
| Priority | none | Severity | partial_block | ||
| Status | closed | Resolution | fixed | ||
| Product Version | 6.5.x | ||||
| Summary | 19617: get_question_properties Return Array Doesn't Have "question" field - you can't see what the question is | ||||
| Description | Hello, To help visualize what I mean, I'm including screenshots of the return array for "get_question_properties" from LS 3.22 and 6.5.4. I am not attaching an LSS since this is not relegated to a specific survey. | ||||
| Steps To Reproduce | Steps to reproducemake call to the Remote Control API get_question_properties Expected resultreturn array has field "question" that contains question text Actual resultreturn array has basically everything but the question field | ||||
| Tags | No tags attached. | ||||
| Bug heat | 14 | ||||
| Complete LimeSurvey version number (& build) | 6.5.4+240422 | ||||
| I will donate to the project if issue is resolved | No | ||||
| Browser | Google Chrome Version 126.0.6478.62 (Official Build) (64-bit) | ||||
| Database type & version | Note sure | ||||
| Server OS (if known) | |||||
| Webserver software & version (if known) | |||||
| PHP Version | Not sure | ||||
 |
3 to 6 : MAJOR version when you make incompatible API changes see https://semver.org/ Then : Not an issue (just to confirm) |
 |
I could not find the question text in the attributes_lang. |
|
|
Then need a solution : i10n/lang array ? |
|
|
I understand that the API may be broken between versions. But wouldn't it be possible to provide a user friendly and backwards compatible fix by adjusting the 6.x API code so we get the required "question" details from the now different table(s)? |
|
|
Yes, i start to work on it, i think i just send question, help and script without extra array. |
|
|
https://github.com/LimeSurvey/LimeSurvey/pull/3949 Doing both |
|
|
Capture d’écran du 2024-08-28 17-45-54.png (26,427 bytes) |
|
|
Looks great! |
|
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=36546 |
|
|
Tested and merged. |
|
|
Fixed in Release 6.6.4+240923 |
