omerkel / announcement Goto Github PK

Current: controlannouncement.py:117: Code always sends attachments if dict current is defining these with key "attachments" as in announcementconfig.py:79. This is independent of the chosen mail type. Avoiding attachments can only be done by complete removal of "attachments" plus its corresponding value.

Intended: in announcementconfig.py: current to hold multiple possible attachment keys. Each defining a lists of files to attach. These keys are referenced by each mail type in mail. current then defines a data set that is possibly reusable over different mail types. Any mail template that should not result in mails having attachments will simply not have any of the possible attachment keys.

Session log as follows:

c:\Python34\python.exe controlannouncement.py
ControlAnnouncement::updateUser
ControlAnnouncement::sendMail
Exception in Tkinter callback
Traceback (most recent call last):
File "c:\Python34\lib\tkinter__init__.py", line 1533, in call
return self.func(*args)
File "controlannouncement.py", line 117, in sendMail
self.announcement.attach()
File "D:\work\ReleaseCandidateTest\announcement.py", line 130, in attach
announcement.message.attach(part)
NameError: name 'announcement' is not defined