Ticket #1692 (closed New Feature: fixed)

Opened 2 years ago

Last modified 2 years ago

Check for invalid temp directory in ColdFusion filemanager connector

Reported by: redtopia Owned by: 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 Download (2.5 KB) - added by wwalc 2 years ago.

Change History

Changed 2 years ago by wwalc

Changed 2 years ago by wwalc

  • keywords Confirmed Review? added
  • owner set to wwalc

Changed 2 years ago by fredck

  • keywords Review+ added; Review? removed
  • milestone set to FCKeditor 2.6

Please add the changelog entry on commit.

Changed 2 years ago by wwalc

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

Fixed with [1580].

Note: See TracTickets for help on using tickets.