Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#4987 closed Bug (fixed)

Browser Server in Link tab of Image Properties dialog links to the wrong page

Reported by: Ben Owned by: Wiktor Walc
Priority: Normal Milestone: CKEditor 3.2
Component: UI : Dialogs Version: 3.0
Keywords: Confirmed Review+ Doc Cc:


When you click the Browser Server button on the Link tab of the Image Properties dialog, it takes you to the filebrowserImageBrowseUrl...

Shouldn't it take you to the filebrowserBrowseUrl?

Attachments (1)

4987.patch (668 bytes) - added by Wiktor Walc 14 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 14 years ago by Wiktor Walc

Keywords: Confirmed added
Milestone: CKEditor 3.2
Owner: set to Wiktor Walc
Version: 3.13.0

Confirmed, in FCKeditor the "Browse Server" button opened file browser with all resource types (filebrowserBrowseUrl) in the "link" tab.

Right now filebrowser plugin is using only dialog name in attachFileBrowser() function. To solve this issue, the filebrowser definition attribute schema used in dialogs should be extended.

Current definition in image dialog (Link tab):

filebrowser : 'info:txtUrl'

we can extend it into something like

filebrowser : 'info:txtUrl:filebrowserBrowseUrl'

where the last part of the definition is optional. If the last part is defined, it has higher priority than editor.config[ 'filebrowser' + ucFirst( dialogName ) + 'BrowseUrl' ].

Changed 14 years ago by Wiktor Walc

Attachment: 4987.patch added

comment:2 Changed 14 years ago by Wiktor Walc

Keywords: Review? added

Ok... forget my last comment.

In attached patch the solution is much simpler. In the image dialog definition we're simply setting the filebrowser.url configuration option to correct url:

  • config.filebrowserImageBrowseLinkUrl - new configuration option, this way developer can precisely set what url should be shown in the "Link" tab in the Image dialog
  • config.filebrowserBrowseUrl - if filebrowserImageBrowseLinkUrl is not set, this will be the default value. It means that CKEditor will now by default work like FCKeditor (it will show all resource types instead of showing only "Images").

comment:3 Changed 14 years ago by Frederico Caldeira Knabben

Keywords: Review+ Doc? added; Review? removed

We need documentation for the configuration option.

comment:4 Changed 14 years ago by Wiktor Walc

Resolution: fixed
Status: newclosed

Fixed with [5096].

comment:5 Changed 14 years ago by Tony

Nice job wwalc, confirming that your patch is working here.

Many thanks, Tony

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