Refactor _error helper function

datasette/views/base.py

@@ -546,3 +546,7 @@ class DataView(BaseView):
         if self.ds.cors:
             add_cors_headers(response.headers)
         return response
+
+
+def _error(messages, status=400):
+    return Response.json({"ok": False, "errors": messages}, status=status)

datasette/views/row.py

@@ -1,6 +1,6 @@
 from datasette.utils.asgi import NotFound, Forbidden, Response
 from datasette.database import QueryInterrupted
-from .base import DataView, BaseView
+from .base import DataView, BaseView, _error
 from datasette.utils import (
     tilde_decode,
     urlsafe_components,
@@ -11,10 +11,6 @@ import sqlite_utils
 from .table import _sql_params_pks, display_columns_and_rows
 
 
-def _error(messages, status=400):
-    return Response.json({"ok": False, "errors": messages}, status=status)
-
-
 class RowView(DataView):
     name = "row"
 

datasette/views/table.py

@@ -31,7 +31,7 @@ from datasette.utils import (
 from datasette.utils.asgi import BadRequest, Forbidden, NotFound, Response
 from datasette.filters import Filters
 import sqlite_utils
-from .base import BaseView, DataView, DatasetteError, ureg
+from .base import BaseView, DataView, DatasetteError, ureg, _error
 from .database import QueryView
 
 LINK_WITH_LABEL = (
@@ -40,10 +40,6 @@ LINK_WITH_LABEL = (
 LINK_WITH_VALUE = '<a href="{base_url}{database}/{table}/{link_id}">{id}</a>'
 
 
-def _error(messages, status=400):
-    return Response.json({"ok": False, "errors": messages}, status=status)
-
-
 class Row:
     def __init__(self, cells):
         self.cells = cells