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)
Change History (2)
Changed 3 years ago by Master One
comment:1 Changed 3 years ago by thomasb
- Resolution set to duplicate
- Status changed from new to closed
Duplicate of #1486543

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