View Issue Details

This bug affects 1 person(s).
 6
IDProjectCategoryView StatusLast Update
17827Bug reportsPluginspublic2022-01-11 12:48
Reporterdschirge Assigned Toollehar  
PrioritynoneSeverityminor 
Status resolvedResolutionfixed 
Product Version5.2.x 
Summary17827: missing functions and missing EXT constant in console environment
Description

Hi guys,

as described in https://bugs.limesurvey.org/view.php?id=17541 there is some missing code in LS 5.

I am using the export function of LS and it requires the const EXT in the file application\commands\console.php
and the functions

  • getAssetManager()
  • getClientScript()
    in the ConsoleApplication class.

Should I provide a merge request like the last time?

Steps To Reproduce

Call the internal class ExportSurveyResultsService
with the following parameters: $exportResponses($survey->sid, $survey->language, 'pdf, $formattingOptions, '').
It results in the attached error message.

TagsNo tags attached.
Bug heat6
Complete LimeSurvey version number (& build)5.2.8+220103
I will donate to the project if issue is resolvedNo
Browser
Database type & versionMySQL 8.0.21
Server OS (if known)
Webserver software & version (if known)
PHP Version7.4.9

Users monitoring this issue

User List There are no users monitoring this issue.

Activities

Mazi

Mazi

2022-01-10 12:14

updater   ~67938

@ollehar, are you fine with just sending a merge request for latest LS5 version like we did previoulsy?

ollehar

ollehar

2022-01-10 12:16

administrator   ~67939

This was already fixed in LS3? If yes, sure, just port the PR to LS5 master branch and I'll merge.

dschirge

dschirge

2022-01-10 12:22

reporter   ~67940

Ok. I will do so!

dschirge

dschirge

2022-01-11 11:52

reporter   ~67952

Pull request is created. Please check and accept https://github.com/LimeSurvey/LimeSurvey/pull/2205

Mazi

Mazi

2022-01-11 12:02

updater   ~67956

@ollehar, we'd appreciate if this cna be added to the next release since a customer needs this fix for their plugins. They switched from LS3 which already has that fix to LS5 where the fix is currently missing.

ollehar

ollehar

2022-01-11 12:27

administrator   ~67957

No probs, merged

dschirge

dschirge

2022-01-11 12:48

reporter   ~67958

Thank you!

Issue History

Date Modified Username Field Change
2022-01-10 12:13 dschirge New Issue
2022-01-10 12:14 Mazi Note Added: 67938
2022-01-10 12:14 Mazi Bug heat 0 => 2
2022-01-10 12:16 ollehar Note Added: 67939
2022-01-10 12:16 ollehar Bug heat 2 => 4
2022-01-10 12:22 dschirge Note Added: 67940
2022-01-10 12:22 dschirge Bug heat 4 => 6
2022-01-11 11:52 dschirge Note Added: 67952
2022-01-11 12:02 Mazi Note Added: 67956
2022-01-11 12:27 ollehar Note Added: 67957
2022-01-11 12:28 ollehar Assigned To => ollehar
2022-01-11 12:28 ollehar Status new => resolved
2022-01-11 12:28 ollehar Resolution open => fixed
2022-01-11 12:48 dschirge Note Added: 67958