Mailing List Archive

HTML vs. plain text editor
Matt Zagrabelny <mzagrabe@d.umn.edu> wrote:

> There may be some errors in JS console in your browser.

Hi Matt, thanks for the kick in right direction!

Now I see this with XHR logging and Javascript errors enabled:
---
GET http://rt.domain/Ticket/Update.html [HTTP/1.1 200 OK 501ms]
GET http://rt.domain/NoAuth/css/rudder/squished-e94f4fd9bce9a4e038d25a7a9d07571f.css [HTTP/1.1 200 OK 0ms]
GET http://rt.domain/NoAuth/js/squished-bf4b1701ea2422a43504908c42d926d9.js [HTTP/1.1 200 OK 0ms]
TypeError: CKEDITOR.instances.UpdateContent is undefined Update.html:557:7
GET http://rt.domain/static/images/favicon.png [HTTP/1.1 200 OK 0ms]
GET http://rt.domain/NoAuth/css/rudder/squished-e94f4fd9bce9a4e038d25a7a9d07571f.css [HTTP/1.1 200 OK 0ms]
GET http://rt.domain/NoAuth/css/rudder/squished-e94f4fd9bce9a4e038d25a7a9d07571f.css [HTTP/1.1 200 OK 0ms]
---

Browser is Firefox 40.0.3
(Mozilla/5.0 (X11; FreeBSD amd64; rv:40.0) Gecko/20100101 Firefox/40.0)

Update.html line 557 is:
554: <script type="text/javascript">
555: jQuery( function() {
556: var updateScrips = function() {
557: CKEDITOR.instances['UpdateContent'].updateElement();
558: var syncCheckboxes = function(ev) {
559: var target = ev.target;
560: jQuery("input[name=TxnSendMailTo]").filter( function() {
561: return this.value == target.value; }
562: ).prop("checked",jQuery(target).prop('checked'));
563: };

Unfortunately I lack Javascript skills to track properly where CKEDITOR
is loaded or initialised.

Anyway, still seems to be related somehow to scrips, so I guess it's
learning time for me. Some pointers ofcourse still appreciated.

--
Mikko Lehto
Re: HTML vs. plain text editor [ In reply to ]
Hi Mikko,

Thank you for testing the RC and for reporting this issue :)

On 2015年11月23日 at 11:29:09, Mikko Lehto (mslehto@iki.fi) wrote:

TypeError: CKEDITOR.instances.UpdateContent is undefined Update.html:557:7 

I see this error, and the symptoms it causes, too. I’ve created a ticket for tracking this: https://issues.bestpractical.com/Ticket/Display.html?id=31507

-- 
Mikko Lehto 

Cheers!
Shawn
Re: HTML vs. plain text editor (resolved) [ In reply to ]
Hi Shawn,

I can confirm that following commit fixes issue I encountered:
---
commit f9682294d772799aa23ecb44948a3bbea0282b30
Author: Shawn M Moore <shawn@bestpractical.com>
Date: Tue Nov 24 19:06:51 2015 +0000

Handle non-ckeditor for syncing scrips

Fixes: I#31507
---

I tested both values (0/1) in RT_SiteConfig.pm / $MessageBoxRichText.


Thanks a lot!

--
Mikko Lehto