Opened 16 years ago
Closed 16 years ago
#2936 closed Bug (fixed)
Error creating folder 'dirname' {Can't create 'dirname' directory}
Reported by: | SLX | Owned by: | Alfonso Martínez de Lizarrondo |
---|---|---|---|
Priority: | Normal | Milestone: | FCKeditor 2.6.5 |
Component: | Server : PHP | Version: | FCKeditor 2.6.4 |
Keywords: | Review+ | Cc: |
Description ¶
If in "fckeditor\editor\filemanager\connectors\php\config.php" I set only one directory name above the root, ex.
$Config['UserFilesPath'] = '/userfiles/' ;
I can't do a quick upload a file. Message box say that: "Error creating folder 'userfiles' {Can't create '/userfiles/' directory}"
By using more directories in the path, like:
$Config['UserFilesPath'] = '/userfiles/abc/' ;
the error doesn't show.
I think this is a bug in the file "fckeditor\editor\filemanager\connectors\php\io.php":
101: // Check if the parent exists, or create it. 102: if ( !file_exists( $sParent ) ) 103: { 104: //prevents agains infinite loop when we can't create root folder
This will try to create a directory "" (empty string) if the config has one directory in the path, because the parent is root (/) and cannot be created. So I think it should be modified:
101: // Check if the parent exists, or create it. 102: if ( '''!empty($sParent) &&''' !file_exists( $sParent ) ) 103: { 104: //prevents agains infinite loop when we can't create root folder
After modification the files are uploaded correctly.

Change History (6)
comment:1 Changed 16 years ago by
comment:3 Changed 16 years ago by
Keywords: | Review? added; HasPatch removed |
---|---|
Milestone: | → FCKeditor 2.6.5 |
Owner: | set to Alfonso Martínez de Lizarrondo |
Status: | new → assigned |
The patch just adds the suggested check.
comment:4 Changed 16 years ago by
Keywords: | Review+ added; Review? removed |
---|
comment:5 Changed 16 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Fixed with [4198]
Thank you for the report and patch.

Sorry, the new code should look like this: