CKEditor 4 unable to override GetUrl

[Michael]

In previous version you could override geturl by creating a function called CKEDITOR_GETURL. This seems be broken because of this block in ckeditor_base.js:

// Make it possible to override the "url" function with a custom
// implementation pointing to a global named CKEDITOR_GETURL.
var newGetUrl = window.CKEDITOR_GETURL;
if ( newGetUrl ) {
	var originalGetUrl = CKEDITOR.url;
	CKEDITOR.url = function( resource ) {
		return newGetUrl.call( CKEDITOR, resource ) || originalGetUrl.call( CKEDITOR, resource );
	};
}

It should be:

// Make it possible to override the "url" function with a custom
// implementation pointing to a global named CKEDITOR_GETURL.
var newGetUrl = window.CKEDITOR_GETURL;
if ( newGetUrl ) {
	var originalGetUrl = CKEDITOR.getUrl;
	CKEDITOR.getUrl = function( resource ) {
		return newGetUrl.call( CKEDITOR, resource ) || originalGetUrl.call( CKEDITOR, resource );
	};
}

[Jakub Ś]

Api and docs say about getUrl (E.g. ​http://docs.cksource.com/CKEditor_3.x/Developers_Guide/Editor_Core_URLs_Manipulation) while code for CKEditor v4 contains CKEDITOR.url

Either there is some different approach used and not described in docs or an error in code.

[Bill DePhillips]

I am using CKEditor 4.0 and am also running into this problem. Do you know when it might be fixed?

[oiha]

I am also interested in upgrading to v4, but this feature is required to do so.

[oiha]

It would be nice to see a milestone. :)

[Piotrek Koszuliński]

Here you are ;)

I haven't checked it yet, so I'm not sure if it's a regression or just a change that requires different approach when overwriting. I'll check this in a first place.

[Frederico Caldeira Knabben]

Fixed with ​git:6e60de2. Thanks mharris!

[Frederico Caldeira Knabben]

Followup update with ​git:4ca27cb.