Opened 16 years ago

Last modified 16 years ago

#1944 closed Bug

Connector interpretes parameter wrong — at Version 5

Reported by: Michael Osipov Owned by:
Priority: Normal Milestone: FCKeditor 2.6
Component: File Browser Version: FCKeditor 2.6 Beta
Keywords: Confirmed Cc:

Description (last modified by Wiktor Walc)

Fire PHP Connector with:

http://www.fckeditor.net/fckeditor/editor/filemanager/connectors/php/connector.php?Command=GetFolders&Type=Image&CurrentFolder=\

Response should be an error because the CurrentFolder does not start with a /

The actual response is:

<Connector command="GetFolders" resourceType="Image">
<CurrentFolder path="/\/" url="/_userfiles/image/\/"/>
<Folders/>
</Connector>

Change History (5)

comment:1 Changed 16 years ago by Michael Osipov

Description: modified (diff)

comment:2 Changed 16 years ago by Frederico Caldeira Knabben

Milestone: FCKeditor 2.6
Priority: HighNormal

comment:3 Changed 16 years ago by Wiktor Walc

Component: Server : PHPFile Browser
Keywords: Discussion added

Current behaviour of connectors is different to that described in Server Side Integration. If trailing or leading slash is missing, it is appended / prepended to the folder name to make a valid path from it.

Shouldn't this be corrected in documentation?

comment:4 Changed 16 years ago by Michael Osipov

Description: modified (diff)

I'd correct the PHP implementation. requiring a pre and appended slash makes sense for correctness' sake. Other implementations may suffer too.

I'd not recommend a folder containing a backslash in its name. It is sanitized when creating a folder.

comment:5 Changed 16 years ago by Wiktor Walc

Description: modified (diff)

All connectors use the same algorithm (different than described in documentation), so let's stick to it in 2.6.

Backslash in folder name is sanitized because it is not allowed in Windows, however it is a valid folder name in linux. So if someone already have a folder with such name, we should allow him to use it.

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