Skip to content

Commit 595c656

Browse files
committed
Require a login instead of throwing an exception when an anonymous
user visits. Handle a lack of UserMap or an incorrect PersonOrOrgInfo by passing None to the template. - Legacy-Id: 234
1 parent 5c9e460 commit 595c656

1 file changed

Lines changed: 8 additions & 6 deletions

File tree

ietf/my/views.py

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,16 @@
1-
from django.http import HttpResponse,HttpResponseRedirect
2-
from django import newforms as forms
31
from django.template import RequestContext
42
from django.shortcuts import render_to_response
53
from ietf.idtracker.models import PersonOrOrgInfo
4+
from django.contrib.auth.decorators import login_required
5+
from django.core.exceptions import ObjectDoesNotExist
66

7+
@login_required
78
def my(request, addr=None):
8-
if request.user:
9-
person = request.user.get_profile().person
10-
else:
11-
person = PersonOrOrgInfo.objects.distinct().get(emailaddresses__email_address=addr)
9+
try:
10+
profile = request.user.get_profile()
11+
person = profile.person
12+
except ObjectDoesNotExist:
13+
person = None
1214
return render_to_response('my/my.html', {
1315
'me': person,
1416
}, context_instance=RequestContext(request))

0 commit comments

Comments
 (0)