Comment 1 for bug 394133

Revision history for this message
Jonathan Lange (jml) wrote :

The branch I've linked doesn't fix the bug, but it demonstrates the problem. It turns out that textwrap (the Python module) will wrap lines at hyphens, even if those hyphens occur within URLs.

To run the unit test:
    ./bin/test -cvv lp.services.mail.tests.test_mailwrapper

If you like waiting for doctests:
    ./bin/test -1cvvt textformatting

It might be that the cheapest fix for this is to change the URL from +mailinglist-moderate to something without hyphens.

For reference:
  * template in lib/canonical/launchpad/emailtemplates/new-held-message.txt
  * mail handler in canonical.launchpad.mailnotification.notify_message_held
  * If you want to change the template text, you will have to edit:
    * lib/lp/registry/doc/message-holds-xmlrpc.txt
    * lib/lp/registry/doc/message-holds.txt
    * lib/lp/services/mailman/doc/postings.txt
  * If you want to change the URL, you'll have to edit:
    * lib/lp/registry/browser/configure.zcml
    * lib/lp/registry/browser/person.py
    * lib/lp/registry/stories/mailinglists/moderation.txt
    * and all the above.

Hope this helps,
jml