Author Topic: tinymce and ckeditor break Civi URLs (Read 927 times)

Shai · October 18, 2013, 06:04:30 am

Folks,

In tinymce and ckeditor both, they are converting urls like

https://example.com/civicrm/contribute/transact?reset=1&id=2&{contact.checksum}&cid={contact.contact_id}

to

https://example.com/civicrm/contribute/transact?reset=1&id=2&{contact.checksum}&cid={contact.contact_id}

converting ampersands to their html entity code.

Firefox on a Mac handles it okay, but most other browsers do not; the result error message from Civi says the link is broken or the back button was used.
I tried editing sites/all/modules/civicrm/packages/ckeditor/config.js by adding

Quote

config.basicEntities = false;

but it didn't work.

I'd appreciate any help in pointing me toward how I can fix this problem.

Thanks so much,

Shai Gluskin

Rajesh · October 18, 2013, 06:28:51 am

Hi shai

For CKEditor you can add the below line to the config file to stop replacing & with &. Not sure how to do it in tinymce editor.

Code: [Select]

config.forceSimpleAmpersand = true;
Below is my CKEditor config file for you reference

Code: [Select]

CKEDITOR.editorConfig = function( config ) {
    // Define changes to default configuration here.
    // For the complete reference:
    // http://docs.ckeditor.com/#!/api/CKEDITOR.config

    // The toolbar groups arrangement, optimized for two toolbar rows.
    config.toolbarGroups = [
        { name: 'clipboard',   groups: [ 'clipboard', 'undo' ] },
        { name: 'editing',     groups: [ 'find', 'selection', 'spellchecker' ] },
        { name: 'links' },
        { name: 'insert' },
        { name: 'forms' },
        { name: 'tools' },
        { name: 'document',       groups: [ 'mode', 'document', 'doctools' ] },
        { name: 'others' },
        '/',
        { name: 'basicstyles', groups: [ 'basicstyles', 'cleanup' ] },
        { name: 'paragraph',   groups: [ 'list', 'indent', 'blocks', 'align' ] },
        { name: 'styles' },
        { name: 'colors' },
        { name: 'about' }
    ];

    // Remove some buttons, provided by the standard plugins, which we don't
    // need to have in the Standard(s) toolbar.
    config.removeButtons = 'Underline,Subscript,Superscript';
    
    // Do NOT replace instances of ampersands ('&') with &amp;
    config.forceSimpleAmpersand = true;
};

Shai · November 12, 2013, 01:10:31 pm

@Rajesh,

Thanks so much for the suggestion.

I did exactly what you said. I added

// Do NOT replace instances of ampersands ('&') with &
config.forceSimpleAmpersand = true;

to sites/all/modules/civicrm/packages/ckeditor/config.js

I cleared caches... but no change, & always converts to &

Any ideas? Is there something in the php config that could be doing this?

Thanks for your help.

Shai Gluskin

Shai · November 12, 2013, 02:39:30 pm

@Rajesh,

Never mind. I hadn't cleared my browser caches after I made the change. My bad.

I have a follow up question.

I made the change to sites/all/modules/civicrm/packages/ckeditor/config.js so I need to create documentation to change it on every civi upgrade.

Is there a way, like in Drupal's hook system, to make the change in a custom module or somewhere else that has a chance of surviving a Civi upgrade?

Thanks,

Shai

CiviTeacher.com · November 14, 2013, 11:20:22 pm

"Break" links for "most other browsers"?

Shai, this link works: https://civicrm.org/civicrm/contribute/transact?reset=1&id=2 for the following browsers:

1. firefox
2. chrome
3. ie10

Before you intensify your situation with custom code, I suggest you expand your browser testing. Most browsers are capable of recognizing and converting HTML codes in urls.

Rajesh · November 19, 2013, 01:00:19 am

@Shai

Not sure about how to make the change in a module or extension, as ckeditor is in CiviCRM's packages directory and i dont think we can override the directory. There is a drupal hook(drupal_add_js) to add custom js files in specific pages, but still that will be CMS specific and will not working in Joomla/Wordpress.

Anyone have any better idea or do we make this change in the core?

Thanks
Rajesh

CiviCRM Community Forums (archive)

News:

Author Topic: tinymce and ckeditor break Civi URLs (Read 927 times)

Shai

tinymce and ckeditor break Civi URLs

Rajesh

Re: tinymce and ckeditor break Civi URLs

Shai

Re: tinymce and ckeditor break Civi URLs

Shai

Re: tinymce and ckeditor break Civi URLs

CiviTeacher.com

Re: tinymce and ckeditor break Civi URLs

Rajesh

Re: tinymce and ckeditor break Civi URLs