I ran into an issue getting the CK editor link module to work. Steps to replicate and fix issue (images below):
1. Download and configure ckeditor_link module
2. Try to link and get a javascript error
3. in sites/all/module/bootstrap/misc/autocomplete.js there is an unknown variable name $context.find
4. Remove and test
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | autocomplete_js-2594243-9.patch | 689 bytes | daniel_j |
| #8 | autocomplete_js-2594243-8.patch | 829 bytes | junghandy |
| #6 | autocomplete_conflicts_with_ckeditor_link-2594243-6.patch | 487 bytes | anrikun |
| #2 | Screen Shot 2015-12-07 at 12.08.23 AM.png | 111.54 KB | djake |
| Screen Shot 2015-10-15 at 4.49.25 PM.png | 38.25 KB | luwoldy |
Comments
Comment #2
djake commentedI am having the same problem. The link dialog is distorted and the JS error described in the issue summary occurs as soon as the dialog is open. Everything works as normal using Seven on the same site.
Comment #3
djake commentedComment #4
kbrinnerI encountered the same thing when I upgraded from 7.x-3.0 to 7.x-3.1. Reverting back to 7.x-3.0 solved the problem.
Comment #5
markhalliwellPlease see this change record: https://www.drupal.org/node/2632698
Comment #6
anrikun commented@markcarver: I don't think this have to do with https://www.drupal.org/node/2632698.
Patch attached should fix the problem with CKEditor Link.
Comment #7
junghandy commentedThe patch in #6 did not fix the issue when using ckeditor_link 7.x-2.4 and bootstrap 7.x-3.5. I receive get the same error as before.
Comment #8
junghandy commentedPiggybacking on the patch from #6 here's a patch that falls back to the global context if the autocomplete constructor was not supplied with one. This resolved issues between ckeditor_link 7.x-2.4 and bootstrap 7.x-3.5 for me.
Comment #9
daniel_j commentedRe-rolling the patch from #8 so it can be referenced from
drush makemakefiles.Comment #11
markhalliwellAhh. I see now. The module is invoking Drupal.jsAC manually and doesn't know about context. Ok. I've committed #9 with a few adjustments, mostly explaining why it's also needed there.