Ticket #1692 (closed New Feature: fixed)

Opened 5 months ago

Last modified 3 months ago

Check for invalid temp directory in ColdFusion filemanager connector

Reported by: redtopia Assigned to: wwalc
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

1692.patch (2.5 kB) - added by wwalc on 02/19/08 17:03:26.

Change History

02/19/08 17:03:26 changed by wwalc

  • attachment 1692.patch added.

02/19/08 17:04:29 changed by wwalc

  • keywords set to Confirmed Review?.
  • owner set to wwalc.

02/20/08 10:40:31 changed by fredck

  • keywords changed from Confirmed Review? to Confirmed Review+.
  • milestone set to FCKeditor 2.6.

Please add the changelog entry on commit.

02/21/08 11:58:42 changed by wwalc

  • status changed from new to closed.
  • resolution set to fixed.

Fixed with [1580].