Inline unnecessary method in OpenAPI schema generator.

Author: carltongibson

rest_framework/schemas/openapi.py

@@ -32,31 +32,30 @@ def get_info(self):
 
         return info
 
-    def get_paths(self, request=None):
-        result = {}
+    def get_schema(self, request=None, public=False):
+        """
+        Generate a OpenAPI schema.
+        """
+        self._initialise_endpoints()
 
-        _, view_endpoints = self._get_paths_and_endpoints(request)
+        # Iterate endpoints generating per method path operations.
+        # TODO: …and reference components.
+        paths = {}
+        _, view_endpoints = self._get_paths_and_endpoints(None if public else request)
         for path, method, view in view_endpoints:
             if not self.has_view_permissions(path, method, view):
                 continue
+
             operation = view.schema.get_operation(path, method)
             # Normalise path for any provided mount url.
             if path.startswith('/'):
                 path = path[1:]
             path = urljoin(self.url or '/', path)
 
-            result.setdefault(path, {})
-            result[path][method.lower()] = operation
-
-        return result
-
-    def get_schema(self, request=None, public=False):
-        """
-        Generate a OpenAPI schema.
-        """
-        self._initialise_endpoints()
+            paths.setdefault(path, {})
+            paths[path][method.lower()] = operation
 
-        paths = self.get_paths(None if public else request)
+        # Compile final schema.
         schema = {
             'openapi': '3.0.2',
             'info': self.get_info(),