Ticket #1484546 (closed Bugs: fixed)

Opened 15 months ago

Last modified 10 months ago

Problem on writing a message

Reported by: skibi82 Owned by: till
Priority: 5 Milestone: 0.2-beta
Component: Client Scripts Version: svn-trunk
Severity: normal Keywords: compose
Cc:

Description (last modified by till) (diff)

If i write a message on html edit format. On sending is showed a message

$messages['nobodywarning']

On text format all is ok

Change History

Changed 15 months ago by till

  • keywords compose added; $messages['nobodywarning'] removed
  • owner set to till
  • status changed from new to assigned
  • description modified (diff)

Which revision are you on?

Changed 15 months ago by skibi82

trunk r764

Changed 15 months ago by thomasb

  • milestone changed from 0.1-rc2 to 0.2-beta

HTML-Editor issue. Not important for 0.1-rc2

Changed 15 months ago by schmidp

if in html mode rcube_find_object('_message').value is empty even if text has been entered. the value of _message seems only relevant if in plain text mode:

Index: app.js
===================================================================
--- app.js	(revision 776)
+++ app.js	(working copy)
@@ -1658,6 +1658,7 @@
     var input_to = rcube_find_object('_to');
     var input_subject = rcube_find_object('_subject');
     var input_message = rcube_find_object('_message');
+	var is_html = (rcube_find_object('_is_html').value == '1');
 
     // check for empty recipient
     if (input_to && !rcube_check_email(input_to.value, true))
@@ -1685,7 +1686,7 @@
       }
 
     // check for empty body
-    if ((input_message.value == '' || (window.tinyMCE && tinyMCE.getContent() == '')) && !confirm(this.get_label('nobodywarning')))
+    if ((!is_html && input_message.value == '' || (window.tinyMCE && tinyMCE.getContent() == '')) && !confirm(this.get_label('nobodywarning')))
       {
       input_message.focus();
       return false;

Changed 10 months ago by estadtherr

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

Thomas's changes in r795 seemed to fix this.

Note: See TracTickets for help on using tickets.