Ticket #1485860 (closed Bugs: fixed)

Opened 10 months ago

Last modified 8 months ago

Empty e-mail when sending mail

Reported by: Ron Owned by:
Priority: 8 Milestone: 0.3-stable
Component: User Interface Version: 0.2.2
Severity: critical Keywords:
Cc: dave@…

Description

Had this problem in 0.2, and now also in the new 0.2.2. E-mails I sent using Firefox 3.0.10 in HTML format (using tinyMCE) become empty. Coincidence is that I had the same problem yesterday when working with TinyMCE. In that situation I had to change the TinyMCE settings in order to get it fixed. The problem is that the onSubmit event isn't bound properly to the form in which the editor resides. Changing /program/js/editor.js to this:

else // mail compose

tinyMCE.init({

mode : 'exact', elements : 'compose-body',

Solves the problem. The problem arises when using mode: 'textareas'. Could be an TinyMCE bug though... I've set the priority that high because users don't see that the e-mail they sent became empty.

Change History

  Changed 10 months ago by alec

  • milestone changed from later to 0.3-stable

follow-up: ↓ 3   Changed 10 months ago by drowe

  • cc dave@… added

Works for me as of 2517 - I see the text in both plain-text and HTML on the receiving end.

in reply to: ↑ 2   Changed 10 months ago by Ron

Replying to drowe:

Works for me as of 2517 - I see the text in both plain-text and HTML on the receiving end.

Hm, TinyMCE is working OK at this moment again. Yesterday I had the same problem even with my fix applied above, but now it's working again :-(.

It could have to do with the sequence in which the onsubmit is bound to the form. I think that sometimes (1 of 10 times) the custom RC onsubmit is called first, and the TinyMCE after that. If that happens then the textarea is empty, and will be submitted empty to the server. To prevent this on some points in the javascript code the TinyMCE function TriggerSave? should be called: http://wiki.moxiecode.com/index.php/TinyMCE:Functions#tinyMCE.triggerSave

Especially before the submit() method of the form is called.

  Changed 8 months ago by alec

  • status changed from new to closed
  • resolution set to fixed

Added triggerSave() call in r2694.

Note: See TracTickets for help on using tickets.