Opened 6 years ago

Closed 5 years ago

#1484546 closed Bugs (fixed)

Problem on writing a message

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

Description (last modified by till)

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 (5)

comment:1 Changed 6 years ago by till

  • Description modified (diff)
  • Keywords compose added; $messages['nobodywarning'] removed
  • Owner set to till
  • Status changed from new to assigned

Which revision are you on?

comment:2 Changed 6 years ago by skibi82

trunk [63f9d3c0]

comment:3 Changed 6 years ago by thomasb

  • Milestone changed from 0.1-rc2 to 0.2-beta

HTML-Editor issue. Not important for 0.1-rc2

comment:4 Changed 6 years 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;

comment:5 Changed 5 years ago by estadtherr

  • Resolution set to fixed
  • Status changed from assigned to closed

Thomas's changes in [67eda085] seemed to fix this.

Note: See TracTickets for help on using tickets.