When adding a rule for I-Ds, only apply the selection to active I-Ds, not those which are expired or published.

 - Legacy-Id: 4528

Author: levkowetz

ietf/community/rules.py

@@ -31,7 +31,7 @@ class WgAsociatedRule(RuleManager):
     description = 'All I-Ds associated with a particular WG'
 
     def get_documents(self):
-        return Document.objects.filter(type='draft').filter(group__acronym=self.value).distinct()
+        return Document.objects.filter(type='draft', states__slug='active').filter(group__acronym=self.value).distinct()
 
     def options(self):
         return [(i.acronym, "%s — %s"%(i.acronym, i.name)) for i in Group.objects.filter(type='wg', state='active').distinct().order_by('name')]
@@ -48,7 +48,7 @@ class AreaAsociatedRule(RuleManager):
     description = 'All I-Ds associated with all WGs in a particular Area'
 
     def get_documents(self):
-        return Document.objects.filter(type='draft').filter(group__parent__acronym=self.value, group__parent__type='area').distinct()
+        return Document.objects.filter(type='draft', states__slug='active').filter(group__parent__acronym=self.value, group__parent__type='area').distinct()
 
     def options(self):
         return [(i.acronym, "%s — %s"%(i.acronym, i.name)) for i in Group.objects.filter(type='area', state='active').distinct().order_by('name')]
@@ -65,7 +65,7 @@ class AdResponsibleRule(RuleManager):
     description = 'All I-Ds with a particular responsible AD'
 
     def get_documents(self):
-        return Document.objects.filter(type='draft').filter(ad=self.value).distinct()
+        return Document.objects.filter(type='draft', states__slug='active').filter(ad=self.value).distinct()
 
     def options(self):
         return [(i.pk, i.name) for i in Person.objects.filter(role__name='ad',group__state='active').distinct().order_by('name')]
@@ -82,39 +82,39 @@ class AuthorRule(RuleManager):
     description = 'All I-Ds with a particular author'
 
     def get_documents(self):
-        return Document.objects.filter(type='draft').filter(authors__person__name__icontains=self.value).distinct()
+        return Document.objects.filter(type='draft', states__slug='active').filter(authors__person__name__icontains=self.value).distinct()
 
 
 class ShepherdRule(RuleManager):
     codename = 'shepherd'
     description = 'All I-Ds with a particular document shepherd'
 
     def get_documents(self):
-        return Document.objects.filter(type='draft').filter(shepherd__name__icontains=self.value).distinct()
+        return Document.objects.filter(type='draft', states__slug='active').filter(shepherd__name__icontains=self.value).distinct()
 
 
 class ReferenceToRFCRule(RuleManager):
     codename = 'reference_to_rfc'
     description = 'All I-Ds that have a reference to a particular RFC'
 
     def get_documents(self):
-        return Document.objects.filter(type='draft').filter(relateddocument__target__document__states='rfc', relateddocument__target__document__name__icontains=self.value).distinct()
+        return Document.objects.filter(type='draft', states__slug='active').filter(relateddocument__target__document__states='rfc', relateddocument__target__document__name__icontains=self.value).distinct()
 
 
 class ReferenceToIDRule(RuleManager):
     codename = 'reference_to_id'
     description = 'All I-Ds that have a reference to a particular I-D'
 
     def get_documents(self):
-        return Document.objects.filter(type='draft').filter(relateddocument__target__document__type='draft', relateddocument__target__document__name__icontains=self.value).distinct()
+        return Document.objects.filter(type='draft', states__slug='active').filter(relateddocument__target__document__type='draft', relateddocument__target__document__name__icontains=self.value).distinct()
 
 
 class ReferenceFromRFCRule(RuleManager):
     codename = 'reference_from_rfc'
     description = 'All I-Ds that are referenced by a particular RFC'
 
     def get_documents(self):
-        return Document.objects.filter(type='draft').filter(relateddocument__source__states='rfc', relateddocument__source__name__icontains=self.value).distinct()
+        return Document.objects.filter(type='draft', states__slug='active').filter(relateddocument__source__states='rfc', relateddocument__source__name__icontains=self.value).distinct()
 
 
 
@@ -123,15 +123,15 @@ class ReferenceFromIDRule(RuleManager):
     description = 'All I-Ds that are referenced by a particular I-D'
 
     def get_documents(self):
-        return Document.objects.filter(type='draft').filter(relateddocument__source__type='draft', relateddocument__source__name__icontains=self.value).distinct()
+        return Document.objects.filter(type='draft', states__slug='active').filter(relateddocument__source__type='draft', relateddocument__source__name__icontains=self.value).distinct()
 
 
 class WithTextRule(RuleManager):
     codename = 'with_text'
     description = 'All I-Ds that contain a particular text string in the name'
 
     def get_documents(self):
-        return Document.objects.filter(type='draft').filter(name__icontains=self.value).distinct()
+        return Document.objects.filter(type='draft', states__slug='active').filter(name__icontains=self.value).distinct()
 
 
 TYPES_OF_RULES = [(i.codename, i.description) for i in RuleManager.__subclasses__()]