Opened 10 years ago

Closed 10 years ago

#416 closed Bug (fixed)

PHP Upload assumes to much

Reported by: ernest leitch Owned by: Alfonso Martínez de Lizarrondo
Priority: Normal Milestone:
Component: Server : PHP Version: FCKeditor 2.4.2
Keywords: Cc:

Description

I downloaded and configured the new version of fck editor v 2.4.2 build 14978.

I was testing the upload image feature from the main navigation bar. I enabled $ConfigUseFileType? in :editor/filemanager/upload/php/config.php:34 so it would place uploaded files in the correct folder according to fck editor.

There is a problem with the upload.php file. It assumes that the folder already exists. If it doesn't the uploader will say the file uploaded but return a false path because the directory doesn't exist.

I corrected this by updating editor/filemanager/upload/php/upload.php: 103 and placing this code just above the line where it moves the uploaded file into the correct directory.

check for the directory before uploading the file

if(!is_dir($sServerDir))

{

mkdir($sServerDir);

}

Now the file uploads to the correct directory.

Change History (4)

comment:1 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: upload usefiletype removed
Priority: HighNormal

comment:2 Changed 10 years ago by Alfonso Martínez de Lizarrondo

Owner: changed from Frederico Caldeira Knabben to Alfonso Martínez de Lizarrondo
Status: newassigned

comment:3 Changed 10 years ago by Alfonso Martínez de Lizarrondo

#429 has been marked as dup

comment:4 Changed 10 years ago by Alfonso Martínez de Lizarrondo

Resolution: fixed
Status: assignedclosed

fixed in [285]

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy