Project

General

Profile

Bug #2008

3 bugs in one, all related to excess comma when email_also_cc is set

Added by Lauri Carpenter over 8 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Normal
Start date:
10/13/2011
Due date:
% Done:

0%

Estimated time:
Duration:

Description

We get this mail:

11-10-13 13:48:56 20428    ERROR       RobotImpl> UNEXPECTED ROBOT FAILURE on job 18531 (event_id 1902, work_code EM):
EXCEPTION: NameError global name 'django' is not defined
Traceback:
  File "/fnal/ups/prd/tissue_core/v2_2c/TissueCoreImpl/RobotImpl.py", line 161, in _doProcessOne
    result = the_method(authenticated_agent, this_job.pk, **kwargs)
  File "/fnal/ups/prd/tissue_core/v2_2c/TissueCoreImpl/EventImpl.py", line 261, in processScheduledEmail
    self._processScheduledJob(agent, event_schedule_id, EventImpl.RCODE_EMBOT, self.eventFactory.handleEmailJob)
  File "/fnal/ups/prd/tissue_core/v2_2c/TissueCoreImpl/EventImpl.py", line 258, in _processScheduledJob
    handler(agent, this_job, *args, **kwargs)
  File "/fnal/ups/prd/tissue_core/v2_2c/TissueEvent/EventFactory.py", line 944, in handleEmailJob
    self.handleEmail(authenticated_agent, this_job.event, this_job.event_work)
  File "/fnal/ups/prd/tissue_core/v2_2c/TissueEvent/EventFactory.py", line 913, in handleEmail
    mail_log.save()
  File "/fnal/ups/prd/tissue_db_models/v2_2a/dj/tissue_db_models/EventEmailLogs.py", line 18, in save
    self._persist_event_contacts()
  File "/fnal/ups/prd/tissue_db_models/v2_2a/dj/tissue_db_models/EventEmailLogs.py", line 31, in _persist_event_contacts
    this_ema.save()
  File "/fnal/ups/prd/tissue_db_models/v2_2a/dj/tissue_db_models/EmailAddresses.py", line 35, in save
    raise django.db.utils.IntegrityError('null valud in column "email_address" violates not-null constraint')

It ends up being 3 separate issues:
1) "null valud" spelling error when raising the Integrity error
2) didn't import django.db or something when trying to raise the integrity error
3) And -- the REAL problem: the integrity error (which we have tracked down to a leading comma that causes a split to have an empty address when trying to split the email addresses when there is an email_also_cc set in the issue).

History

#1 Updated by Lauri Carpenter over 8 years ago

  • Status changed from New to Resolved

#2 Updated by Lauri Carpenter over 8 years ago

  • Status changed from Resolved to Closed


Also available in: Atom PDF