Opened 17 years ago
Last modified 17 years ago
#1944 closed Bug
Connector interpretes parameter wrong — at Version 4
| 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 )
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 (4)
comment:1 Changed 17 years ago by
| Description: | modified (diff) |
|---|
comment:2 Changed 17 years ago by
| Milestone: | → FCKeditor 2.6 |
|---|---|
| Priority: | High → Normal |
comment:3 Changed 17 years ago by
| Component: | Server : PHP → File Browser |
|---|---|
| Keywords: | Discussion added |
comment:4 Changed 17 years ago by
| 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.
Note: See
TracTickets for help on using
tickets.
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?