MINOR: two email title lines, when merged need a space, not empty str

ubernostrum / django-registration

An extensible user-registration app for Django.

BSD 3-Clause "New" or "Revised" License

923 stars 241 forks source link

MINOR: two email title lines, when merged need a space, not empty str #169

Closed griswolf closed 6 years ago

griswolf commented 6 years ago

file: django_registration/backends/activation/views.py line: 104 in send_activation_email Original: subject = ''.join(subject.splitlines()) Better: subject = ' '.join(subject.splitlines())

Note that this assumes no blank lines. Blank lines will be replaced by two spaces

ubernostrum commented 6 years ago

This is actually deliberate -- I want the subject line to look "broken" so the person who wrote the template will go investigate why, and figure out they tried to do a multi-line subject.