Opened 8 years ago

Closed 8 years ago

Last modified 8 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:

Description

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 8 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 8 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 8 years ago by Wiktor Walc

Attachment: 4987.patch added

comment:2 Changed 8 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 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ Doc? added; Review? removed

We need documentation for the configuration option.

comment:4 Changed 8 years ago by Wiktor Walc

Resolution: fixed
Status: newclosed

Fixed with [5096].

comment:5 Changed 8 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 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy