Plugin divarea causes baseHref not working

[Dave Jackson]

This worked fine in 4.0. I've only tried it with the minified ckeditor.js, so it's possible it might be a bug in that.

Please look into this as the use of baseHref is critical for my client's application.

Thanks.

---

Edit:
*Problem can be reporduced from CKEditor 4.0.1
*To reproduce open attached ckeditor-bug package and test 4.0.1 sample page. It turns out that when divarea is used baseHref doesn't work.
Remove the plugin and it works again:

 $(document).ready(function() {
        $('#make-editor').click(function(){
            CKEDITOR.replace('testeditor', {
		removePlugins : 'divarea'
		});
        });
});

Please see comment:3 for detailed TC

[Dave Jackson]

ckeditor file that does not pick up and use baseHref.

[Jakub Ś]

I have set in config.js

config.baseHref = 'http://192.168.1.112:9593/';

then I have used relative path in image dialog (/test_image.png) and it worked without problems. The same thing happened with initial data loaded into editor.

Could you double check and if you still get this error could you specify exact TC steps to reproduce this problem in fresh (without nay customizations) CKEditor?

[Dave Jackson]

Standalone test-case demonstrating bug

[Dave Jackson]

Hi,

I have just attached a standalone test case. Here are steps to reproduce the problem with the tarball I sent you:

1. unzip the file.
2. In a browser, navigate to ckeditortest-4.0.html
3. Click the Make button - that should convert the textarea into an editor
4. Click the image icon in the editor - an image popup will appear
5. Enter (without the quotes) "nakedeye.jpg" into the url field and hit tab.
6. You should now see the image previewed. (It uses baseHref setting in the html to actually point to image/test/path/nakedeye.jpg)
7. Now, navigate to ckeditortest-4.0.1.html and repeat steps 3-6. In this case, it shows a broken image.
8. The only difference between these two tests is the version of the ckeditor library.

Thanks, -Dave J.

[Jakub Ś]

Hi,

Please note that set of plugins in 4.0.1 is very different to what you have in 4.0.

I have tried your sample with default CKEditor 4.0.1.1 (and 4.0.1) full version and image did show up.

My guess is that either one of these custom plugins or some custom change of yours is causing this as provided TC works with default version.

I'm closing this ticket as invalid. If you are able to provide TC that breaks default version - please provide it and I will reopen this ticket.

[Dave Jackson]

The plugins included are ones your website has provided. I didn't add these from arbitrary places.

I'd be happy to remove all of the plugins and keep adding them back one at a time until I can identify which one causes the problem, but then isn't that still a bug on your part, since you are providing the plugins?

[Jakub Ś]

since you are providing the plugins?

That is not quite true. There are not only plugins provided by CKSource but also third-party plugins provided by users.

I'm not sure if you have made any customizations to editor code. If yes, this doesn't have to be plugin fault but one of your changes.

[Dave Jackson]

I made zero customizations to editor code. I just used your builder to pull some plugins that you provide on your site and things broke.

I think you guys have a responsibility to test for interoperability of third party plugins with your own code if you're going to distribute them with your code.

It's pretty unprofessional to put this on your users. At the very least, you should have a really big flashing warning letting your users know that you *don't* do any interoperability testing and it's therefore at our own risk to use any plugins that aren't shipped by you.

I rebuilt an install from just your Full version plus 4 plugins and things seem to work now. Still, I'm disappointed in your lack of responsibility for what you distribute to your users.

[Jakub Ś]

@dave@... in case you haven't noticed it, before building you can visit every plugin website (directly from builder) and check its author and other information like compatibility. You will probably say "I don't have time for this" but if programmer is creating software and just takes 'whatever' that is unprofessional.

[Jakub Ś]

@dave@ it seems that I need to apologise.
To make sure, I have checked your list of plugins for editor 4.0.1 and it seems that one of our plugins - divarea is causing this (without it works fine):

 $(document).ready(function() {
        $('#make-editor').click(function(){
            CKEDITOR.replace('testeditor', {
		removePlugins : 'divarea'
		});
        });
});