Opened 10 years ago
Closed 10 years ago
#416 closed Bug (fixed)
PHP Upload assumes to much
| Reported by: | metric152 | Owned by: | alfonsoml |
|---|---|---|---|
| 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 fredck
- Keywords upload usefiletype removed
- Priority changed from High to Normal
comment:2 Changed 10 years ago by alfonsoml
- Owner changed from fredck to alfonsoml
- Status changed from new to assigned
comment:3 Changed 10 years ago by alfonsoml
comment:4 Changed 10 years ago by alfonsoml
- Resolution set to fixed
- Status changed from assigned to closed
fixed in [285]

#429 has been marked as dup