Merge pull request #11316 from creative-commoners/pulls/6/remove-graphql

API Remove GraphQL

src/Forms/FormRequestHandler.php

@@ -77,7 +77,6 @@ class FormRequestHandler extends RequestHandler
     public function Link($action = null)
     {
         // Forms without parent controller have no link;
-        // E.g. Submission handled via graphql
         $controller = $this->form->getController();
         if (empty($controller)) {
             return null;

src/Forms/GridField/GridField.php

@@ -146,6 +146,17 @@ class GridField extends FormField
      */
     protected $readonlyComponents = [];
 
+    /**
+     * Intentionally not set to FormField::SCHEMA_DATA_TYPE_STRUCTURAL even though there is no corresponding
+     * react component because we want a hard exception thrown for devleopers to see rather than have
+     * them wonder why the field is not rendering.
+     *
+     * Marked as @interal to allow change in a minor release as a react GridField may be implemented in the future
+     *
+     * @internal
+     */
+    protected $schemaDataType;
+
     /**
      * Pattern used for looking up
      */

src/ORM/Connect/NullDatabase.php

@@ -8,7 +8,7 @@ use Exception;
 /**
  * Utility class required due to bad coupling in framework.
  * Not every framework execution should require a working database connection.
- * For example, when generating class and config manifests for deployment bundles,
+ * For example, when generating class and config manifests for deployment bundles
  * or when generating code in a silverstripe/graphql schema build.
  *
  * This class creates the required no-ops to fulfill the contract,

tests/php/Control/Middleware/CanonicalURLMiddlewareTest.php

@@ -182,7 +182,6 @@ class CanonicalURLMiddlewareTest extends SapphireTest
         $requestSlash = $requestHasSlash ? '/' : '';
         $noRedirectPaths = [
             "/admin{$requestSlash}",
-            "/admin/graphql{$requestSlash}",
             "/dev/tasks/my-task{$requestSlash}",
         ];
         $allowRedirectPaths = [