Opened 17 years ago

Closed 17 years ago

#1692 closed New Feature (fixed)

Check for invalid temp directory in ColdFusion filemanager connector

Reported by: John Pansewicz Owned by: Wiktor Walc
Priority: Normal Milestone: FCKeditor 2.6
Component: File Browser Version: FCKeditor 2.5
Keywords: Confirmed Review+ Cc:

Description

This feature request fixes a problem when the ColdFusion server returns an empty string when calling the function GetTempDirectory (). This is a known problem. Restarting the server will fix it, but it's also a good idea to allow the developer to specify a temp directory. I ran into this problem when integrating FCKEditor 2.5 into my application framework, and I could not upload files. After some debugging, I found that my development CF server was not returning a valid temp directory.

In filemanager\browser\default\connectors\cfm\config.cfm, add a new temp file setting that can be set by the developer:

Set a temporary directory Config.TempDirectory = GetTempDirectory ();

In filemanager\browser\default\connectors\cfm\cf_commands.cfm, test to see if the specified temp directory is valid. If not throw an error.

<cfset fsTempDir = REQUEST.Config.TempDirectory> <cfif NOT DirectoryExists (fsTempDir)>

<cfthrow message="Invalid temporary directory: #fsTempDir#">

</cfif>

Attachments (1)

1692.patch (2.5 KB) - added by Wiktor Walc 17 years ago.

Download all attachments as: .zip

Change History (4)

Changed 17 years ago by Wiktor Walc

Attachment: 1692.patch added

comment:1 Changed 17 years ago by Wiktor Walc

Keywords: Confirmed Review? added
Owner: set to Wiktor Walc

comment:2 Changed 17 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed
Milestone: FCKeditor 2.6

Please add the changelog entry on commit.

comment:3 Changed 17 years ago by Wiktor Walc

Resolution: fixed
Status: newclosed

Fixed with [1580].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy