chg: usr: completely disable the contact form #15

src/Controller/GameController.php

@@ -21,7 +21,6 @@ use Symfony\Component\DependencyInjection\Attribute\Autowire;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\HttpKernel\Attribute\AsController;
-use Symfony\Component\Mailer\MailerInterface;
 use Symfony\Component\Routing\Attribute\Route;
 
 /**
@@ -85,8 +84,11 @@ class GameController extends AbstractController
     public function contact(
         Request                $request,
         EntityManagerInterface $em,
-        MailerInterface        $mailer,
     ): Response {
+        // Keep the contact form implementation available, but disable public
+        // access while automated abuse protection is insufficient.
+        throw $this->createNotFoundException();
+
         $contactMessage = new ContactMessage();
         $form = $this->createForm(ContactFormType::class, $contactMessage);
         $form->handleRequest($request);

templates/Game/index.html.twig

@@ -260,7 +260,6 @@
         <li><a href="{{ path('MineSeekerBundle_rules') }}">Game Rules</a></li>
         <li><a href="{{ path('MineSeekerBundle_terms') }}">Terms of Use</a></li>
         <li><a href="{{ path('MineSeekerBundle_privacy') }}">Privacy Policy</a></li>
-        <li><a href="{{ path('MineSeekerBundle_contact') }}">Contact</a></li>
       </ul>
     </nav>
   </div>

tests/Controller/GameControllerTest.php

@@ -89,14 +89,12 @@ class GameControllerTest extends WebTestCase
     }
 
     #[Test]
-    #[TestDox('Contact page loads successfully with form')]
-    public function contactPageLoadsSuccessfully(): void
+    #[TestDox('Contact page is not publicly accessible')]
+    public function contactPageIsNotPubliclyAccessible(): void
     {
         $crawler = $this->client->request('GET', '/contact');
 
-        self::assertResponseStatusCodeSame(Response::HTTP_OK);
-        self::assertSelectorExists('form');
-        self::assertSelectorExists('button[type="submit"]');
+        self::assertResponseStatusCodeSame(Response::HTTP_NOT_FOUND);
     }
 
     #[Test]