View Issue Details

This bug affects 1 person(s).
 44
IDProjectCategoryView StatusLast Update
10893Bug reportsSurvey editingpublic2016-04-11 11:56
Reporterwebmasterlpo Assigned ToLouisGac 
PriorityimmediateSeveritycrash 
Status closedResolutionfixed 
Product Version2.50.x 
Fixed in Version2.50.x 
Summary10893: KCfinder error when saving survey settings
Description

I can't upload a file or image on a new survey, the file manager say that the folder is in read-only. But it seems in fact that the "root" path of the explorer (kcfinder) is wrong, see screenshot.

TagsNo tags attached.
Attached Files
Bug heat44
Complete LimeSurvey version number (& build)160407
I will donate to the project if issue is resolvedNo
BrowserFirefox 46
Database type & versionMySQL
Server OS (if known)Debian 7
Webserver software & version (if known)Apache 2.2
PHP Versionphp 5.5

Relationships

has duplicate 10895 closedLouisGac Save button opens kfinder tab instead of saving changed preferences 
has duplicate 10897 closedLouisGac New bug just after "comfort" updating to Version 2.50+ Build 160408 
has duplicate 10906 closedc_schmitz Save Survey-Settings don't work anymore 
has duplicate 10910 closedc_schmitz Bounce settings in a survey are not being saved 

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2016-04-08 15:49

developer   ~37149

It's because when you create a new survey : the sid is not known ... then uploader must nbe deactuvated.

Seems the old fix don't work : https://github.com/LimeSurvey/LimeSurvey/blob/master/application/helpers/admin/htmleditor_helper.php#L31

webmasterlpo

webmasterlpo

2016-04-08 15:54

reporter   ~37150

In fact, it wasn't a "new" survey because, I've already saved once the survey. I've updated the limesurvey install with the 160408 build and the issue is gone :)

DenisChenu

DenisChenu

2016-04-08 16:22

developer   ~37151

PS : but there are an issue with "new survey ;)"

jpdayan

jpdayan

2016-04-09 10:28

reporter   ~37155

à Denis Chenu:
Ce bug est arrivé avec la mise à jour (comfort upload) quand je suis passé de la 160404 à la 160408. ici lpo est à la 160407.
Comme lui, je travaillais sur un questionnaire depuis 2 jours et tout marchait très bien, je fais cette mise à jour et désormais, je ne peux plus rien modifier des paramètres généraux et textes du questionnaire
Le simple fait de cliquer sur save ou save and close ouvre cette maudite fenêtre qui n'a rien à voir avec mon questionnaire !
Voir aussi mon rapport de bug 10897.
Cordialement

jelo

jelo

2016-04-09 12:19

partner   ~37156

Last edited: 2016-04-09 12:21

Might be the fix that was reverted in LimeSurvey 2.50+ build 160408
-Fixed issue #10820: kcfinder not working with symlinked /upload (reverted from commit 36c988bc75dcc680d7bf20c6df6f6d5e27ad6c3d) (Carsten Schmitz)
https://bugs.limesurvey.org/view.php?id=10820

There seem to be a issue with kcfinder in templates too.
https://www.limesurvey.org/forum/installation-a-update-issues/104187-saving-template-edits-fails-and-redirects-to-kcfinder

umfundi

umfundi

2016-04-09 12:23

reporter   ~37157

Last edited: 2016-04-09 12:30

I installed a new setup of LS just two weeks ago from the site download. 2.5 buils 60317. I have created a couple of surveys and this morning I upgraded to buils 60408 and now I have similar problems.

  • Trying to change a surveys template crashes out to KCfinder

  • Create a new template and try to use it .... crash out to kcfinder.

I have no symlinked stuff on my install????

umfundi

umfundi

2016-04-09 12:48

reporter   ~37158

I have backed out the update and I'm back to 60317.

KCFinder problem has gone away.

jpdayan

jpdayan

2016-04-09 16:23

reporter   ~37161

@umfundi : How can I cancel an update ?(before 160408 I was 160404 and everything worked perfectly). Problem began with 160407 !!!

umfundi

umfundi

2016-04-09 16:34

reporter   ~37162

I had a copy of 60308 in my downloads file, simply copied it over the top of the directory that was live, worked perfectly.

BUT I'M WORKING IN A DEV ENVIRONMENT so I CAN'T GUARANTEE IT'LL WORK FOR you!

I suggest you set up a dev/test server copy of your live site ( if you haven't already ) , test the rollback and then roll that out.

c_schmitz

c_schmitz

2016-04-11 10:00

administrator   ~37173

Introduced by commit https://github.com/LimeSurvey/LimeSurvey/commit/dc9ef8808deae82426b4dbec548fb3af121ee028

jpdayan

jpdayan

2016-04-11 10:46

reporter   ~37178

@c.schmitz Thanks for your note ... but ...
It's te third time I see this github page, and I can see that there is somewhere a </form> which should not be there. I agree it's a mistake.

But this doesn't give us the solution to fix the bug described in 10895 10897 and 10893.

webmasterlpo

webmasterlpo

2016-04-11 10:50

reporter   ~37180

Release 160408 with this 2 commits seems OK :

https://github.com/LimeSurvey/LimeSurvey/commit/85d9ba5d556b7253bc4a00d5ada0a93b43b589d9

https://github.com/LimeSurvey/LimeSurvey/commit/c785ec9e1437ca130c61df80fcb926415d3f943f

webmasterlpo

webmasterlpo

2016-04-11 10:57

reporter   ~37181

In fact, no, KCFinder doesn't open anymore but the settings aren't saved.

c_schmitz

c_schmitz

2016-04-11 11:06

administrator   ~37183

Please be patient, we will release a new version, soon.

LouisGac

LouisGac

2016-04-11 11:15

developer   ~37185

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

c_schmitz

c_schmitz

2016-04-11 11:53

administrator   ~37196

Version 2.50+ Build 160411 released

Related Changesets

LimeSurvey: master fd582b48

2016-04-11 09:14:51

LouisGac

Details Diff
Fixed issue 10893: Error when saving survey settings Affected Issues
10893
mod - application/views/admin/survey/editLocalSettings_main_view.php Diff File
mod - application/views/admin/survey/subview/accordion/_accordion_container.php Diff File
mod - application/views/admin/survey/subview/accordion/_resources_panel.php Diff File
add - application/views/admin/survey/subview/import_ressources_modal.php Diff File

Issue History

Date Modified Username Field Change
2016-04-08 15:14 webmasterlpo New Issue
2016-04-08 15:14 webmasterlpo File Added: Capture d’écran 2016-04-08 à 15.00.15.png
2016-04-08 15:49 DenisChenu Note Added: 37149
2016-04-08 15:54 webmasterlpo Note Added: 37150
2016-04-08 16:22 DenisChenu Note Added: 37151
2016-04-09 10:28 jpdayan Note Added: 37155
2016-04-09 12:19 jelo Note Added: 37156
2016-04-09 12:21 jelo Note Edited: 37156
2016-04-09 12:23 umfundi Note Added: 37157
2016-04-09 12:26 umfundi Note Edited: 37157
2016-04-09 12:30 umfundi Note Edited: 37157
2016-04-09 12:48 umfundi Note Added: 37158
2016-04-09 16:23 jpdayan Note Added: 37161
2016-04-09 16:34 umfundi Note Added: 37162
2016-04-11 09:45 c_schmitz Priority none => immediate
2016-04-11 10:00 c_schmitz Note Added: 37173
2016-04-11 10:01 c_schmitz Assigned To => LouisGac
2016-04-11 10:01 c_schmitz Status new => assigned
2016-04-11 10:01 c_schmitz Severity partial_block => crash
2016-04-11 10:01 c_schmitz Relationship added has duplicate 10895
2016-04-11 10:02 c_schmitz Summary KCfinder error => KCfinder error when saving survey settings
2016-04-11 10:02 c_schmitz Relationship added has duplicate 10897
2016-04-11 10:46 jpdayan Note Added: 37178
2016-04-11 10:48 c_schmitz Relationship added has duplicate 10906
2016-04-11 10:50 webmasterlpo Note Added: 37180
2016-04-11 10:57 webmasterlpo Note Added: 37181
2016-04-11 11:06 c_schmitz Note Added: 37183
2016-04-11 11:15 LouisGac Changeset attached => LimeSurvey master fd582b48
2016-04-11 11:15 LouisGac Note Added: 37185
2016-04-11 11:15 LouisGac Resolution open => fixed
2016-04-11 11:16 c_schmitz Status assigned => resolved
2016-04-11 11:16 c_schmitz Fixed in Version => 2.5
2016-04-11 11:53 c_schmitz Note Added: 37196
2016-04-11 11:53 c_schmitz Status resolved => closed
2016-04-11 11:56 c_schmitz Relationship added has duplicate 10910
2019-11-01 17:25 c_schmitz Category Survey design => Survey editing