From 035fc5209d373ef4ba9d23dc1deac94c47ccf816 Mon Sep 17 00:00:00 2001
From: Hugh Rundle <hugh@hughrundle.net>
Date: Sat, 25 Sep 2021 20:23:59 +1000
Subject: [PATCH] better logic for identifying group manager

---
 bookwyrm/templates/groups/members.html |  6 +++++-
 bookwyrm/templatetags/bookwyrm_tags.py | 13 -------------
 2 files changed, 5 insertions(+), 14 deletions(-)

diff --git a/bookwyrm/templates/groups/members.html b/bookwyrm/templates/groups/members.html
index 7dce0891..80dab21c 100644
--- a/bookwyrm/templates/groups/members.html
+++ b/bookwyrm/templates/groups/members.html
@@ -38,6 +38,11 @@
           <span title="{{ member.display_name }}" class="is-block is-6 has-text-weight-bold">{{ member.display_name|truncatechars:10 }}</span>
           <span title="@{{ member|username }}" class="is-block pb-3">@{{ member|username|truncatechars:8 }}</span>
       </a>
+      {% if group.manager == member %}
+      <span class="icon icon-star-full" title="Manager">
+        <span class="is-sr-only">Manager</span>
+      </span>
+      {% endif %}
       {% include 'snippets/add_to_group_button.html' with user=member minimal=True %}
       {% if member.mutuals %}
       <p class="help">
@@ -59,7 +64,6 @@
           {% trans "Follows you" %}
       </p>
       {% endif %}
-      {% identify_manager %}
     </div>
     {% endfor %}
   </div>
\ No newline at end of file
diff --git a/bookwyrm/templatetags/bookwyrm_tags.py b/bookwyrm/templatetags/bookwyrm_tags.py
index a219beb4..e00a8331 100644
--- a/bookwyrm/templatetags/bookwyrm_tags.py
+++ b/bookwyrm/templatetags/bookwyrm_tags.py
@@ -1,7 +1,6 @@
 """ template filters """
 from django import template
 from django.db.models import Avg
-from django.utils.safestring import mark_safe
 
 from bookwyrm import models, views
 
@@ -99,15 +98,3 @@ def mutuals_count(context, user):
     if not viewer.is_authenticated:
         return None
     return user.followers.filter(followers=viewer).count()
-
-@register.simple_tag(takes_context=True)
-def identify_manager(context):
-    """boolean for whether user is group manager"""
-    group = context['group']
-    member = context['member']
-    snippet =  mark_safe('')
-
-    if group.manager == member:
-        snippet = mark_safe('<span class="icon icon-star-full" title="Manager"><span class="is-sr-only">Manager</span></span>')
-
-    return snippet
\ No newline at end of file