diff --git a/ietf/group/templatetags/group_filters.py b/ietf/group/templatetags/group_filters.py index e7fb4a181..c9481b767 100644 --- a/ietf/group/templatetags/group_filters.py +++ b/ietf/group/templatetags/group_filters.py @@ -2,7 +2,7 @@ from django import template import debug # pyflakes:ignore -from ietf.group.models import Group +from ietf.nomcom.models import NomCom register = template.Library() @@ -19,14 +19,15 @@ def active_nomcoms(user): if not (user and hasattr(user, "is_authenticated") and user.is_authenticated): return [] - groups = [] - - groups.extend(Group.objects.filter( - role__person__user=user, - type_id='nomcom', - state__slug='active').distinct().select_related("type")) - - return groups + return list( + NomCom.objects.filter( + group__role__person__user=user, + group__type_id='nomcom', # just in case... + group__state__slug='active', + ) + .distinct() + .order_by("group__acronym") + ) @register.inclusion_tag('person/person_link.html') def role_person_link(role, **kwargs): diff --git a/ietf/templates/base/menu.html b/ietf/templates/base/menu.html index 691f1972d..d97980805 100644 --- a/ietf/templates/base/menu.html +++ b/ietf/templates/base/menu.html @@ -171,7 +171,7 @@ {% endfor %} {% endif %} - {% if user|active_nomcoms %} + {% with user|active_nomcoms as nomcoms %}{% if nomcoms %} {% if flavor == 'top' %}