Drop coreapi support for DRF 3.17

requirements/requirements-optionals.txt

@@ -1,6 +1,4 @@
 # Optional packages which may be used with REST framework.
-coreapi==2.3.1
-coreschema==0.0.4
 django-filter
 django-guardian>=2.4.0,<2.5
 inflection==0.5.1

rest_framework/compat.py

@@ -23,32 +23,20 @@ def unicode_http_header(value):
     postgres_fields = None
 
 
-# coreapi is required for CoreAPI schema generation
-try:
-    import coreapi
-except ImportError:
-    coreapi = None
-
-# uritemplate is required for OpenAPI and CoreAPI schema generation
+# uritemplate is required for OpenAPI schema generation
 try:
     import uritemplate
 except ImportError:
     uritemplate = None
 
 
-# coreschema is optional
-try:
-    import coreschema
-except ImportError:
-    coreschema = None
-
-
 # pyyaml is optional
 try:
     import yaml
 except ImportError:
     yaml = None
 
+
 # inflection is optional
 try:
     import inflection

rest_framework/filters.py

@@ -3,7 +3,6 @@
 returned by list views.
 """
 import operator
-import warnings
 from functools import reduce
 
 from django.core.exceptions import FieldDoesNotExist, ImproperlyConfigured
@@ -14,8 +13,6 @@
 from django.utils.text import smart_split, unescape_string_literal
 from django.utils.translation import gettext_lazy as _
 
-from rest_framework import RemovedInDRF317Warning
-from rest_framework.compat import coreapi, coreschema
 from rest_framework.fields import CharField
 from rest_framework.settings import api_settings
 
@@ -48,13 +45,6 @@ def filter_queryset(self, request, queryset, view):
         """
         raise NotImplementedError(".filter_queryset() must be overridden.")
 
-    def get_schema_fields(self, view):
-        assert coreapi is not None, 'coreapi must be installed to use `get_schema_fields()`'
-        if coreapi is not None:
-            warnings.warn('CoreAPI compatibility is deprecated and will be removed in DRF 3.17', RemovedInDRF317Warning)
-        assert coreschema is not None, 'coreschema must be installed to use `get_schema_fields()`'
-        return []
-
     def get_schema_operation_parameters(self, view):
         return []
 
@@ -186,23 +176,6 @@ def to_html(self, request, queryset, view):
         template = loader.get_template(self.template)
         return template.render(context)
 
-    def get_schema_fields(self, view):
-        assert coreapi is not None, 'coreapi must be installed to use `get_schema_fields()`'
-        if coreapi is not None:
-            warnings.warn('CoreAPI compatibility is deprecated and will be removed in DRF 3.17', RemovedInDRF317Warning)
-        assert coreschema is not None, 'coreschema must be installed to use `get_schema_fields()`'
-        return [
-            coreapi.Field(
-                name=self.search_param,
-                required=False,
-                location='query',
-                schema=coreschema.String(
-                    title=force_str(self.search_title),
-                    description=force_str(self.search_description)
-                )
-            )
-        ]
-
     def get_schema_operation_parameters(self, view):
         return [
             {
@@ -348,23 +321,6 @@ def to_html(self, request, queryset, view):
         context = self.get_template_context(request, queryset, view)
         return template.render(context)
 
-    def get_schema_fields(self, view):
-        assert coreapi is not None, 'coreapi must be installed to use `get_schema_fields()`'
-        if coreapi is not None:
-            warnings.warn('CoreAPI compatibility is deprecated and will be removed in DRF 3.17', RemovedInDRF317Warning)
-        assert coreschema is not None, 'coreschema must be installed to use `get_schema_fields()`'
-        return [
-            coreapi.Field(
-                name=self.ordering_param,
-                required=False,
-                location='query',
-                schema=coreschema.String(
-                    title=force_str(self.ordering_title),
-                    description=force_str(self.ordering_description)
-                )
-            )
-        ]
-
     def get_schema_operation_parameters(self, view):
         return [
             {

rest_framework/pagination.py

@@ -4,7 +4,6 @@
 """
 
 import contextlib
-import warnings
 from base64 import b64decode, b64encode
 from collections import namedtuple
 from urllib import parse
@@ -15,8 +14,6 @@
 from django.utils.encoding import force_str
 from django.utils.translation import gettext_lazy as _
 
-from rest_framework import RemovedInDRF317Warning
-from rest_framework.compat import coreapi, coreschema
 from rest_framework.exceptions import NotFound
 from rest_framework.response import Response
 from rest_framework.settings import api_settings
@@ -151,12 +148,6 @@ def to_html(self):  # pragma: no cover
     def get_results(self, data):
         return data['results']
 
-    def get_schema_fields(self, view):
-        assert coreapi is not None, 'coreapi must be installed to use `get_schema_fields()`'
-        if coreapi is not None:
-            warnings.warn('CoreAPI compatibility is deprecated and will be removed in DRF 3.17', RemovedInDRF317Warning)
-        return []
-
     def get_schema_operation_parameters(self, view):
         return []
 
@@ -313,36 +304,6 @@ def to_html(self):
         context = self.get_html_context()
         return template.render(context)
 
-    def get_schema_fields(self, view):
-        assert coreapi is not None, 'coreapi must be installed to use `get_schema_fields()`'
-        if coreapi is not None:
-            warnings.warn('CoreAPI compatibility is deprecated and will be removed in DRF 3.17', RemovedInDRF317Warning)
-        assert coreschema is not None, 'coreschema must be installed to use `get_schema_fields()`'
-        fields = [
-            coreapi.Field(
-                name=self.page_query_param,
-                required=False,
-                location='query',
-                schema=coreschema.Integer(
-                    title='Page',
-                    description=force_str(self.page_query_description)
-                )
-            )
-        ]
-        if self.page_size_query_param is not None:
-            fields.append(
-                coreapi.Field(
-                    name=self.page_size_query_param,
-                    required=False,
-                    location='query',
-                    schema=coreschema.Integer(
-                        title='Page size',
-                        description=force_str(self.page_size_query_description)
-                    )
-                )
-            )
-        return fields
-
     def get_schema_operation_parameters(self, view):
         parameters = [
             {
@@ -530,32 +491,6 @@ def get_count(self, queryset):
         except (AttributeError, TypeError):
             return len(queryset)
 
-    def get_schema_fields(self, view):
-        assert coreapi is not None, 'coreapi must be installed to use `get_schema_fields()`'
-        if coreapi is not None:
-            warnings.warn('CoreAPI compatibility is deprecated and will be removed in DRF 3.17', RemovedInDRF317Warning)
-        assert coreschema is not None, 'coreschema must be installed to use `get_schema_fields()`'
-        return [
-            coreapi.Field(
-                name=self.limit_query_param,
-                required=False,
-                location='query',
-                schema=coreschema.Integer(
-                    title='Limit',
-                    description=force_str(self.limit_query_description)
-                )
-            ),
-            coreapi.Field(
-                name=self.offset_query_param,
-                required=False,
-                location='query',
-                schema=coreschema.Integer(
-                    title='Offset',
-                    description=force_str(self.offset_query_description)
-                )
-            )
-        ]
-
     def get_schema_operation_parameters(self, view):
         parameters = [
             {
@@ -933,36 +868,6 @@ def to_html(self):
         context = self.get_html_context()
         return template.render(context)
 
-    def get_schema_fields(self, view):
-        assert coreapi is not None, 'coreapi must be installed to use `get_schema_fields()`'
-        if coreapi is not None:
-            warnings.warn('CoreAPI compatibility is deprecated and will be removed in DRF 3.17', RemovedInDRF317Warning)
-        assert coreschema is not None, 'coreschema must be installed to use `get_schema_fields()`'
-        fields = [
-            coreapi.Field(
-                name=self.cursor_query_param,
-                required=False,
-                location='query',
-                schema=coreschema.String(
-                    title='Cursor',
-                    description=force_str(self.cursor_query_description)
-                )
-            )
-        ]
-        if self.page_size_query_param is not None:
-            fields.append(
-                coreapi.Field(
-                    name=self.page_size_query_param,
-                    required=False,
-                    location='query',
-                    schema=coreschema.Integer(
-                        title='Page size',
-                        description=force_str(self.page_size_query_description)
-                    )
-                )
-            )
-        return fields
-
     def get_schema_operation_parameters(self, view):
         parameters = [
             {