diff --git a/website/members/models.py b/website/members/models.py index b7bff7f45b0c5095b5543b06a8c479a5f37ccbcc..22b2a637bc9273f1af39915bb3be8a4d903385a5 100644 --- a/website/members/models.py +++ b/website/members/models.py @@ -414,7 +414,7 @@ class Profile(models.Model): def display_name(self): pref = self.display_name_preference if pref == 'nickname' and self.nickname is not None: - return self.nickname + return f"'{self.nickname}'" elif pref == 'firstname': return self.user.first_name elif pref == 'initials': @@ -436,7 +436,7 @@ class Profile(models.Model): pref = self.display_name_preference if (self.nickname is not None and (pref == 'nickname' or pref == 'nicklast')): - return self.nickname + return f"'{self.nickname}'" elif pref == 'initials': if self.initials: return '{} {}'.format(self.initials, self.user.last_name) diff --git a/website/members/tests/test_models.py b/website/members/tests/test_models.py index 7965415b466c371ab36b2153cfedf78e45eea293..afc041849c6c48df2b746a42fbf9f5562065def5 100644 --- a/website/members/tests/test_models.py +++ b/website/members/tests/test_models.py @@ -113,8 +113,8 @@ class MemberDisplayNameTest(TestCase): self.assertEqual('Johnny Test', self.profile.display_name()) self.assertEqual('Johnny', self.profile.short_display_name()) self.profile.nickname = 'John' - self.assertEqual('John', self.profile.display_name()) - self.assertEqual('John', self.profile.short_display_name()) + self.assertEqual("'John'", self.profile.display_name()) + self.assertEqual("'John'", self.profile.short_display_name()) def test_check_display_name_firstname(self): self.profile.display_name_preference = 'firstname' @@ -143,4 +143,4 @@ class MemberDisplayNameTest(TestCase): self.assertEqual('Johnny', self.profile.short_display_name()) self.profile.nickname = 'John' self.assertEqual('\'John\' Test', self.profile.display_name()) - self.assertEqual('John', self.profile.short_display_name()) + self.assertEqual('\'John\'', self.profile.short_display_name())