Skip to content

Commit 483d9db

Browse files
committed
Removes classes obviated in r12520. Commit ready for merge. Should be held for release after r12520 is released?
- Legacy-Id: 12521
1 parent b24bdb5 commit 483d9db

3 files changed

Lines changed: 1 addition & 90 deletions

File tree

ietf/review/admin.py

Lines changed: 1 addition & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
from django.contrib import admin
22

3-
from ietf.review.models import (ReviewerSettings, UnavailablePeriod, ReviewWish,
4-
ResultUsedInReviewTeam, TypeUsedInReviewTeam, NextReviewerInTeam,
3+
from ietf.review.models import (ReviewerSettings, UnavailablePeriod, ReviewWish, NextReviewerInTeam,
54
ReviewRequest, ReviewTeamSettings )
65

76
class ReviewerSettingsAdmin(admin.ModelAdmin):
@@ -36,24 +35,6 @@ class ReviewWishAdmin(admin.ModelAdmin):
3635

3736
admin.site.register(ReviewWish, ReviewWishAdmin)
3837

39-
class ResultUsedInReviewTeamAdmin(admin.ModelAdmin):
40-
list_display = ["team", "result"]
41-
list_display_links = ["team"]
42-
list_filter = ["team"]
43-
ordering = ["team", "result__order"]
44-
raw_id_fields = ["team"]
45-
46-
admin.site.register(ResultUsedInReviewTeam, ResultUsedInReviewTeamAdmin)
47-
48-
class TypeUsedInReviewTeamAdmin(admin.ModelAdmin):
49-
list_display = ["team", "type"]
50-
list_display_links = ["team"]
51-
list_filter = ["team"]
52-
ordering = ["team", "type__order"]
53-
raw_id_fields = ["team"]
54-
55-
admin.site.register(TypeUsedInReviewTeam, TypeUsedInReviewTeamAdmin)
56-
5738
class NextReviewerInTeamAdmin(admin.ModelAdmin):
5839
list_display = ["team", "next_reviewer"]
5940
list_display_links = ["team"]

ietf/review/models.py

Lines changed: 0 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -86,33 +86,6 @@ def __unicode__(self):
8686
class Meta:
8787
verbose_name_plural = "review wishes"
8888

89-
class ResultUsedInReviewTeam(models.Model):
90-
"""Captures that a result name is valid for a given team for new
91-
reviews. This also implicitly defines which teams are review
92-
teams - if there are no possible review results valid for a given
93-
team, it can't be a review team."""
94-
team = models.ForeignKey(Group)
95-
result = models.ForeignKey(ReviewResultName)
96-
97-
def __unicode__(self):
98-
return u"{} in {}".format(self.result.name, self.team.acronym)
99-
100-
class Meta:
101-
verbose_name = "review result used in team setting"
102-
verbose_name_plural = "review result used in team settings"
103-
104-
class TypeUsedInReviewTeam(models.Model):
105-
"""Captures that a type name is valid for a given team for new
106-
reviews. """
107-
team = models.ForeignKey(Group, limit_choices_to=~models.Q(reviewteamsettings=None))
108-
type = models.ForeignKey(ReviewTypeName)
109-
110-
def __unicode__(self):
111-
return u"{} in {}".format(self.type.name, self.team.acronym)
112-
113-
class Meta:
114-
verbose_name = "review type used in team setting"
115-
verbose_name_plural = "review type used in team settings"
11689

11790
class NextReviewerInTeam(models.Model):
11891
team = models.ForeignKey(Group, limit_choices_to=~models.Q(reviewteamsettings=None))

ietf/review/resources.py

Lines changed: 0 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
from ietf.api import ToOneField # pyflakes:ignore
99

1010
from ietf.review.models import (ReviewerSettings, ReviewRequest,
11-
ResultUsedInReviewTeam, TypeUsedInReviewTeam,
1211
UnavailablePeriod, ReviewWish, NextReviewerInTeam,
1312
ReviewSecretarySettings, ReviewTeamSettings )
1413

@@ -65,27 +64,6 @@ class Meta:
6564
}
6665
api.review.register(ReviewRequestResource())
6766

68-
69-
70-
from ietf.group.resources import GroupResource
71-
from ietf.name.resources import ReviewResultNameResource
72-
class ResultUsedInReviewTeamResource(ModelResource):
73-
team = ToOneField(GroupResource, 'team')
74-
result = ToOneField(ReviewResultNameResource, 'result')
75-
class Meta:
76-
queryset = ResultUsedInReviewTeam.objects.all()
77-
serializer = api.Serializer()
78-
cache = SimpleCache()
79-
#resource_name = 'resultusedinreviewteam'
80-
filtering = {
81-
"id": ALL,
82-
"team": ALL_WITH_RELATIONS,
83-
"result": ALL_WITH_RELATIONS,
84-
}
85-
api.review.register(ResultUsedInReviewTeamResource())
86-
87-
88-
8967
from ietf.person.resources import PersonResource
9068
from ietf.group.resources import GroupResource
9169
class UnavailablePeriodResource(ModelResource):
@@ -146,27 +124,6 @@ class Meta:
146124
}
147125
api.review.register(NextReviewerInTeamResource())
148126

149-
150-
151-
from ietf.group.resources import GroupResource
152-
from ietf.name.resources import ReviewTypeNameResource
153-
class TypeUsedInReviewTeamResource(ModelResource):
154-
team = ToOneField(GroupResource, 'team')
155-
type = ToOneField(ReviewTypeNameResource, 'type')
156-
class Meta:
157-
queryset = TypeUsedInReviewTeam.objects.all()
158-
serializer = api.Serializer()
159-
cache = SimpleCache()
160-
#resource_name = 'typeusedinreviewteam'
161-
filtering = {
162-
"id": ALL,
163-
"team": ALL_WITH_RELATIONS,
164-
"type": ALL_WITH_RELATIONS,
165-
}
166-
api.review.register(TypeUsedInReviewTeamResource())
167-
168-
169-
170127
from ietf.person.resources import PersonResource
171128
from ietf.group.resources import GroupResource
172129
class ReviewSecretarySettingsResource(ModelResource):

0 commit comments

Comments
 (0)