Opened 17 years ago

Closed 17 years ago

#5887 closed (fixed)

consolidate common code in SafeMIMEText and SafeMIMEMultipart

Reported by: Carl Karsten <carl@…> Owned by: nobody
Component: Core (Mail) Version: dev
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Browsing the source, noticed the same code in 2 places. moved it into def forbid_multi_line_headers(name, val):

Did not include setitem call - would have been messy, and removed any chance of that function being useful for anything else.

Change History (3)

by Carl Karsten <carl@…>, 17 years ago

Attachment: mail.diff​ added

svn diff django/core/mail.py

comment:1 by Simon G <dev@…>, 17 years ago

Summary: cosolidate common code in SafeMIMEText and SafeMIMEMultipart → consolidate common code in SafeMIMEText and SafeMIMEMultipart
Triage Stage: Unreviewed → Ready for checkin

comment:2 by Adrian Holovaty, 17 years ago

Resolution: → fixed
Status: new → closed

(In [6987]) Fixed #5887 -- Consolidated some duplicate code in SafeMIMEText and SafeMIMEMultipart. Thanks, Carl Karsten

Note: See TracTickets for help on using tickets.
Back to Top