431704 - Remove useless newlines in replies sent by email_in.pl

Status: RESOLVED FIXED

(Bugzilla :: Incoming Email, defect)

Description

[Frédéric Buclin]

When you pass illegal changes to email_in.pl, it replies to the sender about what's wrong:

You tried to change the 
    Priority field 
      from P4
      to P1
    , but only
      the assignee or reporter 
      of the bug, or
    a sufficiently empowered user may change that field.


Frédéric Buclin wrote:
> > @priority = P1


As you can see, the message is pretty ugly, because every newline from user-error.html.tmpl is displayed as is in the reply. These newlines are not a problem from the web browser because HTML doesn't render them. But they are obviously a problem in a text message. We should remove newlines from error messages before sending the email back to the sender.

Comment 1

[Max Kanat-Alexander]

  This could be fixed by wrapping it in "FILTER collapse". However, if there was any <pre> formatting in the message, it would be destroyed. I think that's OK, though.

  This isn't significant enough for me to actually block a release on it, but I agree that it'd be nice to fix.

Comment 2

[Frédéric Buclin]

Attachment: patch, v1

Comment 3

[Max Kanat-Alexander]

Comment on attachment 318862 [details] [diff] [review]
patch, v1

  No, you should just wrap message.txt.tmpl with FILTER collapse, I think.

Comment 4

[Max Kanat-Alexander]

Comment on attachment 318862 [details] [diff] [review]
patch, v1

  Okay, since you pointed out on IRC that ThrowUserError doesn't use message.txt.tmpl (nor would CodeError or TemplateError, etc.) this is the best and most centralized way to do this.

However, this should be s/[\s\n]+/ /gms I think, no?

Comment 5

[Frédéric Buclin]

Comment on attachment 318862 [details] [diff] [review]
patch, v1

message.txt.tmpl is not called by ThrowUserError().

Comment 6

[Max Kanat-Alexander]

Comment on attachment 318862 [details] [diff] [review]
patch, v1

  See above comment about regex.

Comment 7

[Frédéric Buclin]

Attachment: patch, v2

The result is pretty much the same. :)

Comment 8

[Max Kanat-Alexander]

Comment on attachment 318864 [details] [diff] [review]
patch, v2

Looks good to me.

Comment 9

[Frédéric Buclin]

tip:

Checking in email_in.pl;
/cvsroot/mozilla/webtools/bugzilla/email_in.pl,v  <--  email_in.pl
new revision: 1.19; previous revision: 1.18
done

3.0.4:

Checking in email_in.pl;
/cvsroot/mozilla/webtools/bugzilla/email_in.pl,v  <--  email_in.pl
new revision: 1.5.2.12; previous revision: 1.5.2.11
done