API Remove GraphQL

silverstripe · Aug 22, 2024 · c4d0751 · c4d0751
1 parent 379bd67
commit c4d0751
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 3 deletions.
diff --git a/src/Forms/FormRequestHandler.php b/src/Forms/FormRequestHandler.php
@@ -77,7 +77,6 @@ public function __construct(Form $form)
     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;

diff --git a/src/ORM/Connect/NullDatabase.php b/src/ORM/Connect/NullDatabase.php
@@ -8,7 +8,7 @@
 /**
  * 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,

diff --git a/tests/php/Control/Middleware/CanonicalURLMiddlewareTest.php b/tests/php/Control/Middleware/CanonicalURLMiddlewareTest.php
@@ -182,7 +182,6 @@ public function testRedirectTrailingSlashIgnorePaths(bool $addTrailingSlash, boo
         $requestSlash = $requestHasSlash ? '/' : '';
         $noRedirectPaths = [
             "/admin{$requestSlash}",
-            "/admin/graphql{$requestSlash}",
             "/dev/tasks/my-task{$requestSlash}",
         ];
         $allowRedirectPaths = [