Don't crash speaker lookup on speakers with no userid
authorMagnus Hagander <magnus@hagander.net>
Mon, 13 Aug 2018 20:27:47 +0000 (22:27 +0200)
committerMagnus Hagander <magnus@hagander.net>
Mon, 13 Aug 2018 20:27:47 +0000 (22:27 +0200)
These would either be old migrated speakers or speakers who have been
added manually into the system (which we really shouldn't do..)

postgresqleu/confreg/backendlookups.py

index 8762b5ef442883b6774774e57e016c543dbdbff7..ee989a389033d6df0d55ba196fe4404d1a74bc1a 100644 (file)
@@ -83,7 +83,7 @@ class SpeakerLookup(LookupBase):
 
        @classmethod
        def get_values(self, query):
-               return [{'id': s.id, 'value': u"%s (%s)" % (s.fullname, s.user.username)}
+               return [{'id': s.id, 'value': u"%s (%s)" % (s.fullname, s.user.username if s.user else '')}
                                for s in Speaker.objects.filter(
                                                Q(fullname__icontains=query) | Q(twittername__icontains=query) | Q(user__username__icontains=query)
                                )[:30]]