Include bug links in e-mail

Review Request #521 — Created Aug. 31, 2008 and submitted

Information

Review Board SVN (deprecated)
207

Reviewers

Instead of showing a list of bug nubmers, such as:
 'This addresses bugs 123, 456, and 789',
this shows:
 'This addresses bugs 123, 456, and 789.

    http://bugs/123
    http://bugs/456
    http://bugs/789'

Even if a mail client does not automatically turn URLs to links in plain-text emails, the user can copy and paste the URL.
This avoids the headache of rewriting the email template to use HTML (which not all users may want, anyway).

The leading blank line was difficult to get rid of without making the template harder to read than it already is (and yes, the endfor is intentionally not on its own line). In order to keep the template semi-readable, I left the leading blank line there.
My internal site (Ubuntu Server 8.04 32-bit, python 2.5, django 1.0 beta 2) - emails went out with valid bug links and proper grammar.
Tested with issues referencing no bugs, 1 bug, 2 bugs, and 3 bugs.
david
  1. I would much prefer to see this logic in the template instead of as a filter.  You can just iterate over the list of bug IDs with template tags.
    
    I'd also like the output to look like this:
    
    This addresses bugs 123, 456 and 789
        http://bugs/123
        http://bugs/456
        http://bugs/789
  2. 
      
david
  1. Looks good now.  Committed as r1494.
  2.