From 7ff01c630693060fd7f6a48d9e7050164ea7806b Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Jan 25 2016 14:32:22 +0000 Subject: [PATCH 1/2] Optimize the author2user filter to avoid a round-trip Since we already know the default_email let's give it instead of asking the DB for it. --- diff --git a/pagure/ui/filters.py b/pagure/ui/filters.py index 5cfb746..9dc28f0 100644 --- a/pagure/ui/filters.py +++ b/pagure/ui/filters.py @@ -317,7 +317,7 @@ def author_to_user(author, size=16): user = pagure.lib.search_user(SESSION, email=author.email) if user: output = "%s %s" % ( - avatar(user.user, size), + avatar(user.default_email, size), flask.url_for('view_user', username=user.username), author.name, ) From 570175a5a3cab1ff68c927739f7284d781aeedf8 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Jan 25 2016 14:32:22 +0000 Subject: [PATCH 2/2] Use the email of the author of the commit to get the avatar Since with the username, it won't quite work --- diff --git a/pagure/ui/filters.py b/pagure/ui/filters.py index 9dc28f0..2db2fce 100644 --- a/pagure/ui/filters.py +++ b/pagure/ui/filters.py @@ -329,7 +329,7 @@ def author_to_avatar(author, size=32): """ Template filter transforming a pygit2 Author object into an avatar. """ user = pagure.lib.search_user(SESSION, email=author.email) - output = user.user if user else author.name + output = user.default_email if user else author.email return avatar(output, size)