Opened 3 years ago

Closed 3 years ago

#1486622 closed Bugs (duplicate)

Wrong Line-Wrapping of Quoted Text

Reported by: Master One Owned by:
Priority: 5 Milestone: later
Component: Core functionality Version: 0.3.1
Severity: normal Keywords: wrap,line wrap,word wrap
Cc:

Description

The problem is, that the function rcmail_wrap_quoted in program/steps/mail/func.inc first removes the "> ", does the wordwrap, and then rebuilds the lines with the "> ", resulting in a max. line-length of 77 (= line wrapped to 75 characters + "> "), which is then line-wrapped again at 75 characters in program/steps/mail/sendmail.inc, causing problem with messed up quoted text, once it's sent.

The problem could be fixed in Roundcube 0.3.1 with the enclosed patch.

So calling the function rcmail_wrap_quoted with a desired line-length value of just 73 should lead to the intended correct behaviour (I am not quite sure about the behavior with quoted text inside quoted text, so with lines starting with more than one instance of "> " or just ">", like "> > > " or ">>> ").

I just checked trunk, and although a configurable $LINE_LENGTH has been introduced, it seems the problem for text-emails has not been fixed (there is only one part in compose.inc, which reduces $LINE_LENGTH by 2 for html part).

Attachments (1)

roundcube_wrap_quoted.patch (338 bytes) - added by Master One 3 years ago.
Patch for correcting line-length of quoted text in program/steps/mail/compose.inc

Download all attachments as: .zip

Change History (2)

Changed 3 years ago by Master One

Patch for correcting line-length of quoted text in program/steps/mail/compose.inc

comment:1 Changed 3 years ago by thomasb

  • Resolution set to duplicate
  • Status changed from new to closed

Duplicate of #1486543

Note: See TracTickets for help on using tickets.