diff options
| author |   Paul Eggleton <paul.eggleton@linux.intel.com> | 2013-05-19 16:22:10 +0100 |
|---|---|---|
| committer | Paul Eggleton <paul.eggleton@linux.intel.com> | 2013-05-22 17:45:36 +0100 |
| commit | eaa7e4a8d7c735bf5b2aeb482059d43d5b8de837 (patch) | |
| tree | 0ab9fc4bb07f0f4866bf16d9bdc06696bb9dab26 /layerindex | |
| parent | 4502a3894678addecace8aeea8db543611b5948f (diff) | |
| download | openembedded-core-contrib-eaa7e4a8d7c735bf5b2aeb482059d43d5b8de837.tar.gz | |
Use user's first name or login name in review email
It's not really appropriate to say "Hi Joe Bloggs" at the start of the
email - we only need the user's first name or the login name if the
first name hasn't been filled in.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
Diffstat (limited to 'layerindex')
| -rw-r--r-- | layerindex/views.py | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/layerindex/views.py b/layerindex/views.py index 5ef2316c4e..e64c446c02 100644 --- a/layerindex/views.py +++ b/layerindex/views.py @@ -145,8 +145,12 @@ def edit_layer_view(request, template_name, slug=None): perm = Permission.objects.get(codename='publish_layer') users = User.objects.filter(Q(groups__permissions=perm) | Q(user_permissions=perm) ).distinct() for user in users: + if user.first_name: + user_name = user.first_name + else: + user_name = user.username d = Context({ - 'user_name': user.get_full_name(), + 'user_name': user_name, 'layer_name': layeritem.name, 'layer_url': request.build_absolute_uri(reverse('layer_review', args=(layeritem.name,))) + '?branch=master', }) |