Treat Area Groups similar to Working Groups and Research Groups where it makes sense to do so. Commit ready for merge.

 - Legacy-Id: 13832

Author: rjsparks

ietf/community/forms.py

@@ -39,7 +39,7 @@ def restrict_state(state_type, slug=None):
                 self.fields["group"].label = "Area"
                 self.fields["group"].queryset = self.fields["group"].queryset.filter(Q(type="area") | Q(acronym="irtf")).order_by("acronym")
             else:
-                self.fields["group"].queryset = self.fields["group"].queryset.filter(type__in=("wg", "rg")).order_by("acronym")
+                self.fields["group"].queryset = self.fields["group"].queryset.filter(type__in=("wg", "rg", "ag", )).order_by("acronym")
 
             del self.fields["person"]
             del self.fields["text"]

ietf/community/utils.py

@@ -47,7 +47,7 @@ def can_manage_community_list(user, clist):
 
         if clist.group.type_id == 'area':
             return Role.objects.filter(name__slug='ad', person__user=user, group=clist.group).exists()
-        elif clist.group.type_id in ('wg', 'rg'):
+        elif clist.group.type_id in ('wg', 'rg', 'ag'):
             return Role.objects.filter(name__slug='chair', person__user=user, group=clist.group).exists()
         elif clist.group.type_id in ('program'):
             return Role.objects.filter(name__slug='lead', person__user=user, group=clist.group).exists()

ietf/doc/utils.py

@@ -107,7 +107,7 @@ def can_adopt_draft(user, doc):
                 and doc.group.type_id == "individ")
 
     roles = Role.objects.filter(name__in=("chair", "delegate", "secr"),
-                                group__type__in=("wg", "rg"),
+                                group__type__in=("wg", "rg", "ag", ),
                                 group__state="active",
                                 person__user=user)
     role_groups = [ r.group for r in roles ]

ietf/group/features.py

@@ -16,6 +16,7 @@ class GroupFeatures(object):
     admin_roles = ["chair"]
 
     def __init__(self, group):
+        # TODO: should 'ag' be in this list
         if group.type_id in ("wg", "rg"):
             self.has_milestones = True
             self.has_chartering_process = True

ietf/group/forms.py

@@ -155,6 +155,7 @@ def insert_confirm_field(label, initial):
         if existing:
             raise forms.ValidationError("Acronym used for an existing group (%s)." % existing.name)
 
+        # TODO: Why is this limited to types wg and rg? We would want to be warned about _any_ old collision I think?
         old = GroupHistory.objects.filter(acronym__iexact=acronym, type__in=("wg", "rg"))
         if old:
             insert_confirm_field(label="Confirm reusing acronym %s" % old[0].acronym, initial=False)

ietf/group/models.py

@@ -37,7 +37,7 @@ def __unicode__(self):
 
     def name_with_acronym(self):
         res = self.name
-        if self.type_id in ("wg", "rg", "area"):
+        if self.type_id in ("wg", "rg", "ag", "area"):
             res += " %s (%s)" % (self.type, self.acronym)
         return res
 
@@ -55,7 +55,7 @@ def about_url(self):
         # bridge gap between group-type prefixed URLs and /group/ ones
         from django.urls import reverse as urlreverse
         kwargs = { 'acronym': self.acronym }
-        if self.type_id in ("wg", "rg"):
+        if self.type_id in ("wg", "rg", "ag"):
             kwargs["group_type"] = self.type_id
         return urlreverse(self.features.about_page, kwargs=kwargs)
 

ietf/group/tests_info.py

@@ -1218,7 +1218,7 @@ def ensure_cant_edit(group,user):
             self.assertEqual(response.status_code, 404)
             self.client.logout()
 
-        for type_id in GroupTypeName.objects.exclude(slug__in=('wg','rg','team')).values_list('slug',flat=True):
+        for type_id in GroupTypeName.objects.exclude(slug__in=('wg','rg','ag','team')).values_list('slug',flat=True):
             group = GroupFactory.create(type_id=type_id)
             for user in (None,User.objects.get(username='secretary')):
                 ensure_updates_dont_show(group,user)

ietf/group/utils.py

@@ -124,7 +124,7 @@ def can_manage_materials(user, group):
     return has_role(user, 'Secretariat') or group.has_role(user, ("chair", "delegate", "secr", "matman"))
 
 def can_provide_status_update(user, group):
-    if not group.type_id in ['wg','rg','team']:
+    if not group.type_id in ['wg','rg','ag','team']:
         return False
     return has_role(user, 'Secretariat') or group.has_role(user, ("chair", "delegate", "secr", "ad",))
 
@@ -181,7 +181,7 @@ def construct_group_menu_context(request, group, selected, group_type, others):
         import ietf.group.views
         entries.append(("Review requests", urlreverse(ietf.group.views.review_requests, kwargs=kwargs)))
         entries.append(("Reviewers", urlreverse(ietf.group.views.reviewer_overview, kwargs=kwargs)))
-    if group.type_id in ('rg','wg','team'):
+    if group.type_id in ('rg','wg','ag','team'):
         entries.append(("Meetings", urlreverse("ietf.group.views.meetings", kwargs=kwargs)))
     entries.append(("History", urlreverse("ietf.group.views.history", kwargs=kwargs)))
     entries.append(("Photos", urlreverse("ietf.group.views.group_photos", kwargs=kwargs)))

ietf/group/views.py

@@ -767,7 +767,7 @@ def group_photos(request, group_type=None, acronym=None):
     group = get_object_or_404(Group, acronym=acronym)
     roles = sorted(Role.objects.filter(group__acronym=acronym),key=lambda x: x.name.name+x.person.last_name())
 
-    if   group.type_id in ['wg', 'rg', ]:
+    if   group.type_id in ['wg', 'rg', 'ag', ]:
         roles = reorder_roles(roles, ['chair', 'secr'])
     elif group.type_id in ['nomcom', ]:
         roles = reorder_roles(roles, ['chair', 'member', 'advisor', ])

ietf/mailinglists/views.py

@@ -4,7 +4,7 @@
 from django.shortcuts import render
 
 def groups(request):
-    groups = Group.objects.filter(type__in=("wg", "rg"), list_archive__startswith='http').order_by("acronym")
+    groups = Group.objects.filter(type__in=("wg", "rg", "ag"), list_archive__startswith='http').order_by("acronym")
 
     return render(request, "mailinglists/group_archives.html", { "groups": groups } )