If I try to upload images here on survey creation I get the message that I don't have permission to do this (which is the intended behaviour)

Please update to the latest version and check again.

Intended behaviour? So user should guess that he should save the survey first before he will be able to insert image in the text block? Or more advanced user will start checking permissions of /upload, find that they are correct and will stay puzzled why he can't upload while directory permissions are correct... Will test on latest stable tonight anyway, but doubt it is fixed. The issue is around for a while, I am sure these guys encountered the same case - uploading before saving new survey:
http://www.limesurvey.org/en/forum/installation-a-update-issues/75677-image-upload-apache-cannot-write-to-upload-folder-even-when-permissions-are-ok

http://www.limesurvey.org/en/forum/installation-a-update-issues/65062-problem-with-image-upload-ckfinder

rkabalin: Please try to understand what I wrote before you start ranting. I wrote, that normally you should get a message 'you don't have permission to do this' when trying to upload a file. (and which is currently the expected behaviour) I am completely aware that the message is not optimal.

Still, according to your bug report you get a PHP error, which is quite a difference.
I am asking you kindly to update to the latest version, clear your browser cache and see if you still have this error. Can you do this, please? Thank you.

c_schmitz: sorry, my mistake, was not that clear in the original description. The message obviously pops-up, it says "Cannot create images folder". The PHP error I mentioned is from apache error log.

Updated to latest master (a40d6b1f), the problem with upload persists.

That's really strange because I cannot reproduce it. What browser are you using?

latest Chrome on windows. Also I have debug setting set to 2, that might be why the mkdir is reporting error to log. Quick modification makes it show where it actually tries to write:

--- a/third_party/kcfinder/core/uploader.php
+++ b/third_party/kcfinder/core/uploader.php
@@ -265,7 +265,7 @@ class uploader {
// CHECK & CREATE UPLOAD FOLDER
if (!is_dir($this->typeDir)) {
if (!mkdir($this->typeDir, $this->config['dirPerms']))

• $this->backMsg("Cannot create {dir} folder.", array('dir' => $this->type));
• $this->backMsg("Cannot create {dir} folder.", array('dir' => $this->typeDir));
  } elseif (!is_readable($this->typeDir))
  $this->backMsg("Cannot read upload folder.");
  }

and the JS error pop-up becomes " Cannot create /var/www/limesurvey/third_party/kcfinder/upload/images folder"

Can you dump the contents of $_SESSION['KCFINDER'] please?

yep, sure.

At the point when there is an error:

array(2) {
["types"]=>
array(3) {
["files"]=>
string(223) "7z aiff asf avi bmp csv doc fla flv gif gz gzip ico jpeg jpg mid mov mp3 mp4 mpc mpeg mpg ods odt pdf png ppt pxd qt ram rar rm rmi rmvb rtf sdc sitd swf sxc sxw tar tgz tif tiff txt vsd wav wma wmv xls xml zip pstpl css js"
["flash"]=>
string(223) "7z aiff asf avi bmp csv doc fla flv gif gz gzip ico jpeg jpg mid mov mp3 mp4 mpc mpeg mpg ods odt pdf png ppt pxd qt ram rar rm rmi rmvb rtf sdc sitd swf sxc sxw tar tgz tif tiff txt vsd wav wma wmv xls xml zip pstpl css js"
["images"]=>
string(223) "7z aiff asf avi bmp csv doc fla flv gif gz gzip ico jpeg jpg mid mov mp3 mp4 mpc mpeg mpg ods odt pdf png ppt pxd qt ram rar rm rmi rmvb rtf sdc sitd swf sxc sxw tar tgz tif tiff txt vsd wav wma wmv xls xml zip pstpl css js"
}
["self"]=>
&array(0) {
}
}

after the survey has been saved (when there is no error):

array(5) {
["types"]=>
array(3) {
["files"]=>
string(223) "7z aiff asf avi bmp csv doc fla flv gif gz gzip ico jpeg jpg mid mov mp3 mp4 mpc mpeg mpg ods odt pdf png ppt pxd qt ram rar rm rmi rmvb rtf sdc sitd swf sxc sxw tar tgz tif tiff txt vsd wav wma wmv xls xml zip pstpl css js"
["flash"]=>
string(223) "7z aiff asf avi bmp csv doc fla flv gif gz gzip ico jpeg jpg mid mov mp3 mp4 mpc mpeg mpg ods odt pdf png ppt pxd qt ram rar rm rmi rmvb rtf sdc sitd swf sxc sxw tar tgz tif tiff txt vsd wav wma wmv xls xml zip pstpl css js"
["images"]=>
string(223) "7z aiff asf avi bmp csv doc fla flv gif gz gzip ico jpeg jpg mid mov mp3 mp4 mpc mpeg mpg ods odt pdf png ppt pxd qt ram rar rm rmi rmvb rtf sdc sitd swf sxc sxw tar tgz tif tiff txt vsd wav wma wmv xls xml zip pstpl css js"
}
["disabled"]=>
bool(false)
["uploadURL"]=>
string(34) "/limesurvey/upload/surveys/178169/"
["uploadDir"]=>
string(42) "/var/www/limesurvey/upload/surveys/178169/"
["self"]=>
&array(0) {
}
}

Open application/helper/htmleditor_helper.php

and after line

 Yii::app()->session['KCFINDER'] = array();

insert this line
$_SESSION['KCFINDER']['disabled'] = true;

save, then try again and see if anything changed.

With a fix above it works fine now. Thanks a lot!

The potential problem that may araise from here - what will happen if user uploaded the image, but changed his mind and decided not to save survey. What will happen with the file, will it be deleted from /upload directory at some point?

Wait, with that fix applied you should get now the message 'You don't have permission to do this'.
You should not be able to upload files during survey creation. Can you check again please?

please ignore the previous post, it was a red herring )) I must have created survey when posted the session dump, then deleted it, and then created a new one, which allowed me to save since the directory with the same id for some reason remained there (potentially a different bug).

No, the $_SESSION['KCFINDER']['disabled'] = true; fix does not do anything for the new survey. I still get the same "Cannot create images folder" message.

Is there a way I can get access to that system?
I would need FTP/SSH access and a login to the limesurvey administration.

If yes, please send it (email address removed)

no, sorry, it is my local dev environment. The bug is on the latest master and there is nothing specific about the system, just a minimal apache/php/mysql with no weird settings, or unusual modules. I can's see why you are not able to replicate it.

I am sorry but I can fix only something that I can reproduce.
For example you can see the correct behaviour in our online demo.

I am sorry but since I cannot reproduce I will need to close the issue. If you find a fix or have new hints regarding the issue feel free to re-open it.

The thing is that we focused on the error message issue here, rather than responding to the original problem defined in the subject. That might be a part of the problem since I reported that php error, but more like a separate bug rather than the original issue. Even with correct message the bug is still there. The message "You don\'t have permissions to upload files" is clearly misleading. If there is no way to upload image before storing survey, it should explicitly say so. Since the discussion of this one was focused on the message content, I suggest to rename it to "incorrect message on file upload attempt at the newly created survey" and fill in another report claiming that "You don't have permissions to upload files" message is misleading as there is nothing wrong with permissions really.

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

Fix committed to 2.1 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=10822

Thanks, c_schmitz. There is a typo I commented on in github.

Fix committed to 2.1 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=10823

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

2.00 Build 121211 released