Opened 9 years ago

Closed 9 years ago

#1692 closed New Feature (fixed)

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:


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#">


Attachments (1)

1692.patch (2.5 KB) - added by wwalc 9 years ago.

Download all attachments as: .zip

Change History (4)

Changed 9 years ago by wwalc

comment:1 Changed 9 years ago by wwalc

  • Keywords Confirmed Review? added
  • Owner set to wwalc

comment:2 Changed 9 years ago by fredck

  • Keywords Review+ added; Review? removed
  • Milestone set to FCKeditor 2.6

Please add the changelog entry on commit.

comment:3 Changed 9 years ago by wwalc

  • Resolution set to fixed
  • Status changed from new to closed

Fixed with [1580].

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