klicking linkit-button in ckeditor does nothing [#1026834]

i think i have installed everything correctly. in ckeditor i might mark a word, that is supposed to hold the link, then clicking the linkit-button in CKeditor but nothing happens ...

I had things set up correctly and functioning in D6, but now in D7 it won't work. in D7 I am using Pathologic with Linkit, in D6 it was Pathfilter from what I remember , could this be the reason for the D7 not working now?

Comment	File	Size	Author
#6	linkit.module.1026834.patch	1.25 KB	anon
#3	linkit.jpg	61.24 KB	bavarian

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

anon

Swedish

Gothenburg

CreditAttribution: anon commented 14 January 2011 at 19:08

Linkit 7.x only supports Pathologic so thats not the problem here.

Do you have firebug installed? If so, open the console and see if you get any errors.
Anyway, I will take a look at this to see if I can reproduce it.

Comment #2

anon

Swedish

Gothenburg

CreditAttribution: anon commented 14 January 2011 at 12:08

Is it Ckeditor with or without WYSIWYG?

Comment #3

bavarian CreditAttribution: bavarian commented 14 January 2011 at 12:40

File	Size
linkit.jpg	61.24 KB

-> #2 it is CKeditor WITHOUT wysiwyg

-> #1 ahhhhhhhh .... Linkit7 will only support Pathologic ...thanks for making this clear. i do have Firebug installed but don't know for sure if i am using it correctly. When trying to click the linkit-button in CKeditor Firebug gives an error-message as captured in attached picture

Comment #4

mtyaka CreditAttribution: mtyaka commented 14 January 2011 at 20:23

I've got the same problem, trying to use Linkit together with standalone CKeditor 7.x-1.0.

It seems that CKeditor module's internals have changed and the method of determining whether CKeditor is loaded for the current form in linkit_form_alter() isn't correct anymore.

Comment #5

anon

Swedish

Gothenburg

CreditAttribution: anon commented 14 January 2011 at 20:34

@mtyaka: Yes thats correct. Im morking on that now.

Comment #6

anon

Swedish

Gothenburg

CreditAttribution: anon commented 15 January 2011 at 01:32

Status:

Active

» Needs review

File	Size
linkit.module.1026834.patch	1.25 KB

Ok, I have figured it now.
My patch will make the button work again.

!!IF YOU USE CKEDITOR WITHOUT WYSIWYG!!
I found a small bug in Ckeditor that will strip the "internal:" prefix away.
#1027666: Add internal to filter_allowed_protocols
Until this is fixed, be sure to uncheck, "Limit allowed HTML tags" under Security filters for each Ckeditor profile you will use with Linkit. This can still be checked in the normal Input format configuration, but not in ckeditor.