@@ -257,7 +257,7 @@ paths:
schema:
type: array
items:
- $ref: '#/components/schemas/CoverLetterList'
+ $ref: '#/components/schemas/CoverList'
tags:
- covers
/api/covers/{id}/:
@@ -278,7 +278,7 @@ paths:
content:
application/json:
schema:
- $ref: '#/components/schemas/CoverLetterDetail'
+ $ref: '#/components/schemas/CoverDetail'
'404':
description: Not found
content:
@@ -1557,7 +1557,7 @@ components:
items:
type: string
readOnly: true
- CoverLetterList:
+ CoverList:
type: object
properties:
id:
@@ -1619,9 +1619,9 @@ components:
type: string
format: uri
readOnly: true
- CoverLetterDetail:
+ CoverDetail:
allOf:
- - $ref: '#/components/schemas/CoverLetterList'
+ - $ref: '#/components/schemas/CoverList'
- properties:
headers:
title: Headers
@@ -2125,7 +2125,7 @@ components:
format: uri
readOnly: true
cover_letter:
- $ref: '#/components/schemas/CoverLetterEmbedded'
+ $ref: '#/components/schemas/CoverEmbedded'
patches:
title: Patches
type: array
@@ -2238,7 +2238,7 @@ components:
maxLength: 255
minLength: 1
readOnly: true
- CoverLetterEmbedded:
+ CoverEmbedded:
type: object
properties:
id:
@@ -262,7 +262,7 @@ paths:
schema:
type: array
items:
- $ref: '#/components/schemas/CoverLetterList'
+ $ref: '#/components/schemas/CoverList'
tags:
- covers
/api/{{ version_url }}covers/{id}/:
@@ -283,7 +283,7 @@ paths:
content:
application/json:
schema:
- $ref: '#/components/schemas/CoverLetterDetail'
+ $ref: '#/components/schemas/CoverDetail'
'404':
description: Not found
content:
@@ -1603,7 +1603,7 @@ components:
items:
type: string
readOnly: true
- CoverLetterList:
+ CoverList:
type: object
properties:
id:
@@ -1673,9 +1673,9 @@ components:
format: uri
readOnly: true
{% endif %}
- CoverLetterDetail:
+ CoverDetail:
allOf:
- - $ref: '#/components/schemas/CoverLetterList'
+ - $ref: '#/components/schemas/CoverList'
- properties:
headers:
title: Headers
@@ -2193,7 +2193,7 @@ components:
format: uri
readOnly: true
cover_letter:
- $ref: '#/components/schemas/CoverLetterEmbedded'
+ $ref: '#/components/schemas/CoverEmbedded'
patches:
title: Patches
type: array
@@ -2308,7 +2308,7 @@ components:
maxLength: 255
minLength: 1
readOnly: true
- CoverLetterEmbedded:
+ CoverEmbedded:
type: object
properties:
id:
@@ -142,7 +142,7 @@ paths:
schema:
type: array
items:
- $ref: '#/components/schemas/CoverLetterList'
+ $ref: '#/components/schemas/CoverList'
tags:
- covers
/api/1.0/covers/{id}/:
@@ -163,7 +163,7 @@ paths:
content:
application/json:
schema:
- $ref: '#/components/schemas/CoverLetterDetail'
+ $ref: '#/components/schemas/CoverDetail'
'404':
description: Not found
content:
@@ -1389,7 +1389,7 @@ components:
items:
type: string
readOnly: true
- CoverLetterList:
+ CoverList:
type: object
properties:
id:
@@ -1431,9 +1431,9 @@ components:
items:
$ref: '#/components/schemas/SeriesEmbedded'
readOnly: true
- CoverLetterDetail:
+ CoverDetail:
allOf:
- - $ref: '#/components/schemas/CoverLetterList'
+ - $ref: '#/components/schemas/CoverList'
- properties:
headers:
title: Headers
@@ -1881,7 +1881,7 @@ components:
format: uri
readOnly: true
cover_letter:
- $ref: '#/components/schemas/CoverLetterEmbedded'
+ $ref: '#/components/schemas/CoverEmbedded'
patches:
title: Patches
type: array
@@ -1967,7 +1967,7 @@ components:
maxLength: 255
minLength: 1
readOnly: true
- CoverLetterEmbedded:
+ CoverEmbedded:
type: object
properties:
id:
@@ -142,7 +142,7 @@ paths:
schema:
type: array
items:
- $ref: '#/components/schemas/CoverLetterList'
+ $ref: '#/components/schemas/CoverList'
tags:
- covers
/api/1.1/covers/{id}/:
@@ -163,7 +163,7 @@ paths:
content:
application/json:
schema:
- $ref: '#/components/schemas/CoverLetterDetail'
+ $ref: '#/components/schemas/CoverDetail'
'404':
description: Not found
content:
@@ -1399,7 +1399,7 @@ components:
items:
type: string
readOnly: true
- CoverLetterList:
+ CoverList:
type: object
properties:
id:
@@ -1456,9 +1456,9 @@ components:
type: string
format: uri
readOnly: true
- CoverLetterDetail:
+ CoverDetail:
allOf:
- - $ref: '#/components/schemas/CoverLetterList'
+ - $ref: '#/components/schemas/CoverList'
- properties:
headers:
title: Headers
@@ -1932,7 +1932,7 @@ components:
format: uri
readOnly: true
cover_letter:
- $ref: '#/components/schemas/CoverLetterEmbedded'
+ $ref: '#/components/schemas/CoverEmbedded'
patches:
title: Patches
type: array
@@ -2018,7 +2018,7 @@ components:
maxLength: 255
minLength: 1
readOnly: true
- CoverLetterEmbedded:
+ CoverEmbedded:
type: object
properties:
id:
@@ -257,7 +257,7 @@ paths:
schema:
type: array
items:
- $ref: '#/components/schemas/CoverLetterList'
+ $ref: '#/components/schemas/CoverList'
tags:
- covers
/api/1.2/covers/{id}/:
@@ -278,7 +278,7 @@ paths:
content:
application/json:
schema:
- $ref: '#/components/schemas/CoverLetterDetail'
+ $ref: '#/components/schemas/CoverDetail'
'404':
description: Not found
content:
@@ -1557,7 +1557,7 @@ components:
items:
type: string
readOnly: true
- CoverLetterList:
+ CoverList:
type: object
properties:
id:
@@ -1619,9 +1619,9 @@ components:
type: string
format: uri
readOnly: true
- CoverLetterDetail:
+ CoverDetail:
allOf:
- - $ref: '#/components/schemas/CoverLetterList'
+ - $ref: '#/components/schemas/CoverList'
- properties:
headers:
title: Headers
@@ -2125,7 +2125,7 @@ components:
format: uri
readOnly: true
cover_letter:
- $ref: '#/components/schemas/CoverLetterEmbedded'
+ $ref: '#/components/schemas/CoverEmbedded'
patches:
title: Patches
type: array
@@ -2238,7 +2238,7 @@ components:
maxLength: 255
minLength: 1
readOnly: true
- CoverLetterEmbedded:
+ CoverEmbedded:
type: object
properties:
id:
@@ -11,14 +11,14 @@ from rest_framework.reverse import reverse
from rest_framework.serializers import SerializerMethodField
from patchwork.api.base import BaseHyperlinkedModelSerializer
-from patchwork.api.filters import CoverLetterFilterSet
+from patchwork.api.filters import CoverFilterSet
from patchwork.api.embedded import PersonSerializer
from patchwork.api.embedded import ProjectSerializer
from patchwork.api.embedded import SeriesSerializer
from patchwork.models import CoverLetter
-class CoverLetterListSerializer(BaseHyperlinkedModelSerializer):
+class CoverListSerializer(BaseHyperlinkedModelSerializer):
web_url = SerializerMethodField()
project = ProjectSerializer(read_only=True)
@@ -43,7 +43,7 @@ class CoverLetterListSerializer(BaseHyperlinkedModelSerializer):
# NOTE(stephenfin): This is here to ensure our API looks the same even
# after we changed the series-patch relationship from M:N to 1:N. It
# will be removed in API v2
- data = super(CoverLetterListSerializer, self).to_representation(
+ data = super(CoverListSerializer, self).to_representation(
instance)
data['series'] = [data['series']] if data['series'] else []
return data
@@ -63,7 +63,7 @@ class CoverLetterListSerializer(BaseHyperlinkedModelSerializer):
}
-class CoverLetterDetailSerializer(CoverLetterListSerializer):
+class CoverDetailSerializer(CoverListSerializer):
headers = SerializerMethodField()
@@ -83,18 +83,18 @@ class CoverLetterDetailSerializer(CoverLetterListSerializer):
class Meta:
model = CoverLetter
- fields = CoverLetterListSerializer.Meta.fields + (
+ fields = CoverListSerializer.Meta.fields + (
'headers', 'content')
read_only_fields = fields
- extra_kwargs = CoverLetterListSerializer.Meta.extra_kwargs
- versioned_fields = CoverLetterListSerializer.Meta.versioned_fields
+ extra_kwargs = CoverListSerializer.Meta.extra_kwargs
+ versioned_fields = CoverListSerializer.Meta.versioned_fields
-class CoverLetterList(ListAPIView):
+class CoverList(ListAPIView):
"""List cover letters."""
- serializer_class = CoverLetterListSerializer
- filter_class = filterset_class = CoverLetterFilterSet
+ serializer_class = CoverListSerializer
+ filter_class = filterset_class = CoverFilterSet
search_fields = ('name',)
ordering_fields = ('id', 'name', 'date', 'submitter')
ordering = 'id'
@@ -105,10 +105,10 @@ class CoverLetterList(ListAPIView):
.defer('content', 'headers')
-class CoverLetterDetail(RetrieveAPIView):
+class CoverDetail(RetrieveAPIView):
"""Show a cover letter."""
- serializer_class = CoverLetterDetailSerializer
+ serializer_class = CoverDetailSerializer
def get_queryset(self):
return CoverLetter.objects.all()\
@@ -102,7 +102,7 @@ class CheckSerializer(SerializedRelatedField):
}
-class CoverLetterSerializer(SerializedRelatedField):
+class CoverSerializer(SerializedRelatedField):
class _Serializer(MboxMixin, WebURLMixin, BaseHyperlinkedModelSerializer):
@@ -11,7 +11,7 @@ from rest_framework.serializers import SerializerMethodField
from rest_framework.serializers import SlugRelatedField
from patchwork.api.embedded import CheckSerializer
-from patchwork.api.embedded import CoverLetterSerializer
+from patchwork.api.embedded import CoverSerializer
from patchwork.api.embedded import PatchSerializer
from patchwork.api.embedded import ProjectSerializer
from patchwork.api.embedded import SeriesSerializer
@@ -26,7 +26,7 @@ class EventSerializer(ModelSerializer):
actor = UserSerializer()
patch = PatchSerializer(read_only=True)
series = SeriesSerializer(read_only=True)
- cover = CoverLetterSerializer(read_only=True)
+ cover = CoverSerializer(read_only=True)
previous_state = SlugRelatedField(slug_field='slug', read_only=True)
current_state = SlugRelatedField(slug_field='slug', read_only=True)
previous_delegate = UserSerializer()
@@ -172,7 +172,7 @@ class SeriesFilterSet(TimestampMixin, BaseFilterSet):
fields = ('submitter', 'project')
-class CoverLetterFilterSet(TimestampMixin, BaseFilterSet):
+class CoverFilterSet(TimestampMixin, BaseFilterSet):
project = ProjectFilter(queryset=Project.objects.all())
# NOTE(stephenfin): We disable the select-based HTML widgets for these
@@ -10,7 +10,7 @@ from rest_framework.serializers import SerializerMethodField
from patchwork.api.base import BaseHyperlinkedModelSerializer
from patchwork.api.base import PatchworkPermission
from patchwork.api.filters import SeriesFilterSet
-from patchwork.api.embedded import CoverLetterSerializer
+from patchwork.api.embedded import CoverSerializer
from patchwork.api.embedded import PatchSerializer
from patchwork.api.embedded import PersonSerializer
from patchwork.api.embedded import ProjectSerializer
@@ -23,7 +23,7 @@ class SeriesSerializer(BaseHyperlinkedModelSerializer):
project = ProjectSerializer(read_only=True)
submitter = PersonSerializer(read_only=True)
mbox = SerializerMethodField()
- cover_letter = CoverLetterSerializer(read_only=True)
+ cover_letter = CoverSerializer(read_only=True)
patches = PatchSerializer(read_only=True, many=True)
def get_web_url(self, instance):
@@ -21,7 +21,7 @@ if settings.ENABLE_REST_API:
@unittest.skipUnless(settings.ENABLE_REST_API, 'requires ENABLE_REST_API')
-class TestCoverLetterAPI(utils.APITestCase):
+class TestCoverAPI(utils.APITestCase):
fixtures = ['default_tags']
@staticmethod
@@ -11,7 +11,7 @@ from patchwork.tests.utils import create_cover
from patchwork.tests.utils import create_patch
-class CoverLetterViewTest(TestCase):
+class CoverViewTest(TestCase):
def test_redirect(self):
patch = create_patch()
@@ -207,10 +207,10 @@ if settings.ENABLE_REST_API:
api_person_views.PersonDetail.as_view(),
name='api-person-detail'),
url(r'^covers/$',
- api_cover_views.CoverLetterList.as_view(),
+ api_cover_views.CoverList.as_view(),
name='api-cover-list'),
url(r'^covers/(?P<pk>[^/]+)/$',
- api_cover_views.CoverLetterDetail.as_view(),
+ api_cover_views.CoverDetail.as_view(),
name='api-cover-detail'),
url(r'^patches/$',
api_patch_views.PatchList.as_view(),
We're going to be doing some model surgery shortly. Do the necessary renaming of variables ahead of this. Signed-off-by: Stephen Finucane <stephen@that.guru> --- docs/api/schemas/latest/patchwork.yaml | 14 +++++++------- docs/api/schemas/patchwork.j2 | 14 +++++++------- docs/api/schemas/v1.0/patchwork.yaml | 14 +++++++------- docs/api/schemas/v1.1/patchwork.yaml | 14 +++++++------- docs/api/schemas/v1.2/patchwork.yaml | 14 +++++++------- patchwork/api/cover.py | 24 ++++++++++++------------ patchwork/api/embedded.py | 2 +- patchwork/api/event.py | 4 ++-- patchwork/api/filters.py | 2 +- patchwork/api/series.py | 4 ++-- patchwork/tests/api/test_cover.py | 2 +- patchwork/tests/test_detail.py | 2 +- patchwork/urls.py | 4 ++-- 13 files changed, 57 insertions(+), 57 deletions(-)