Fix for 'NoneType' object has no attribute 'email'

[ilikerobots]

Fixes #807

[gwasser]

Thank you! Hmm, any thoughts why the tests fail on Python 2?

[ilikerobots]

Not sure, actually. I tried in my own env and everything passes (see log below). It's odd though, the error is in the CC handling test which makes me think it might be legit. However, the error seems to be involving date handling.

(django-helpdesk) $ python2.7 --version
Python 2.7.17
(django-helpdesk) $ git branch 
  master
* patch-2
(django-helpdesk) $ python2.7  quicktest.py helpdesk
Creating test database for alias 'default'...
System check identified no issues (0 silenced).
.................................................................
----------------------------------------------------------------------
Ran 65 tests in 2.705s

OK
Destroying test database for alias 'default'...

[ilikerobots]

Hi, regarding the test failure, I don't think it has anything to do with the change in this PR.
I triggered a build with just a text change and it fails in exactly the same way: https://travis-ci.org/github/ilikerobots/django-helpdesk

I think it's an issue with python-tzlocal, probably related to this: https://bugzilla.redhat.com/show_bug.cgi?id=1295709/

[ilikerobots]

I created a fix for the test failure here: #813

[gwasser]

Sounds good thanks!