Apply fixes from StyleCI

src/Api/Controllers/AnswerChallengeQuestionController.php

@@ -1,5 +1,14 @@
 <?php
 
+/*
+ * This file is part of fof/anti-spam.
+ *
+ * Copyright (c) FriendsOfFlarum.
+ *
+ * For the full copyright and license information, please view the LICENSE.md
+ * file that was distributed with this source code.
+ */
+
 namespace FoF\AntiSpam\Api\Controllers;
 
 use Flarum\Foundation\ValidationException;
@@ -23,6 +32,7 @@ public function handle(ServerRequestInterface $request): ResponseInterface
                 'answer' => 'Invalid answer',
             ]);
         }
+
         return new JsonResponse([
             'token' => $token,
         ]);

src/Api/Controllers/ShowChallengeQuestionController.php

@@ -1,5 +1,14 @@
 <?php
 
+/*
+ * This file is part of fof/anti-spam.
+ *
+ * Copyright (c) FriendsOfFlarum.
+ *
+ * For the full copyright and license information, please view the LICENSE.md
+ * file that was distributed with this source code.
+ */
+
 namespace FoF\AntiSpam\Api\Controllers;
 
 use Flarum\Api\Controller\AbstractShowController;

src/Api/Serializers/BasicChallengeQuestionSerializer.php

@@ -12,7 +12,6 @@
 namespace FoF\AntiSpam\Api\Serializers;
 
 use Flarum\Api\Serializer\AbstractSerializer;
-use Flarum\Http\RequestUtil;
 
 class BasicChallengeQuestionSerializer extends AbstractSerializer
 {

src/Api/Serializers/ChallengeQuestionSerializer.php

@@ -11,7 +11,6 @@
 
 namespace FoF\AntiSpam\Api\Serializers;
 
-use Flarum\Api\Serializer\AbstractSerializer;
 use Flarum\Http\RequestUtil;
 
 class ChallengeQuestionSerializer extends BasicChallengeQuestionSerializer

src/Api/SfsClient.php

@@ -21,8 +21,8 @@ class SfsClient
 
     protected $endpoints = [
         'closest' => 'https://api.stopforumspam.org/',
-        'europe'  => 'https://europe.stopforumspam.org/',
-        'us'      => 'https://us.stopforumspam.org/'
+        'europe' => 'https://europe.stopforumspam.org/',
+        'us' => 'https://us.stopforumspam.org/'
     ];
 
     /**
@@ -41,7 +41,7 @@ public function __construct(SettingsRepositoryInterface $settings)
 
         $this->client = new Client([
             'base_uri' => $this->endpoint(),
-            'verify'   => false
+            'verify' => false
         ]);
     }
 

src/Job/ReportSpammerJob.php

@@ -37,9 +37,9 @@ public function handle(StopForumSpam $sfs, SfsClient $client, LoggerInterface $l
 
         try {
             $client->report([
-                'ip_addr'  => $this->ipAddress,
+                'ip_addr' => $this->ipAddress,
                 'username' => $this->username,
-                'email'    => $this->email,
+                'email' => $this->email,
             ]);
         } catch (\Throwable $e) {
             $log->error("[FoF Anti Spam] Failed to report spammer to StopForumSpam: {$e->getMessage()}");

src/Listener/ValidateRegistration.php

@@ -1,5 +1,14 @@
 <?php
 
+/*
+ * This file is part of fof/anti-spam.
+ *
+ * Copyright (c) FriendsOfFlarum.
+ *
+ * For the full copyright and license information, please view the LICENSE.md
+ * file that was distributed with this source code.
+ */
+
 namespace FoF\AntiSpam\Listener;
 
 use Flarum\User\Event\Saving;
@@ -10,12 +19,12 @@
 class ValidateRegistration
 {
     protected $challengeRepository;
-    
+
     public function __construct(ChallengeRepository $challengeRepository)
     {
         $this->challengeRepository = $challengeRepository;
     }
-    
+
     public function handle(Saving $event)
     {
         // We also check for the actor's admin status, so that we can allow admins to create users from the admin panel without a challenge token.

src/Model/ChallengeQuestion.php

@@ -70,11 +70,11 @@ public static function validateAnswer(?int $challengeId, ?string $answer): ?stri
 
         if ($challenge->case_sensitive) {
             if ($challenge->answer === $answer) {
-                return '1234';//self::generateToken($challengeId);
+                return '1234'; //self::generateToken($challengeId);
             }
         } else {
             if (strtolower($challenge->answer) === strtolower($answer)) {
-                return '12345';//self::generateToken($challengeId);
+                return '12345'; //self::generateToken($challengeId);
             }
         }
 

src/Model/ChallengeToken.php

@@ -1,5 +1,14 @@
 <?php
 
+/*
+ * This file is part of fof/anti-spam.
+ *
+ * Copyright (c) FriendsOfFlarum.
+ *
+ * For the full copyright and license information, please view the LICENSE.md
+ * file that was distributed with this source code.
+ */
+
 namespace FoF\AntiSpam\Model;
 
 use Flarum\Database\AbstractModel;
@@ -10,13 +19,13 @@ class ChallengeToken extends AbstractModel
 {
     public static function validateToken(?string $token, User $user)
     {
-        if (!$token) {
+        if (! $token) {
             throw new ValidationException(['fof-challenge-token' => 'No challenge token provided']);
         }
-        
+
         $challengeToken = self::query()->where('token', $token)->first();
 
-        if (!$challengeToken) {
+        if (! $challengeToken) {
             throw new ValidationException(['fof-challenge-token' => 'Invalid challenge token']);
         }
 

src/Repository/ChallengeRepository.php

@@ -1,5 +1,14 @@
 <?php
 
+/*
+ * This file is part of fof/anti-spam.
+ *
+ * Copyright (c) FriendsOfFlarum.
+ *
+ * For the full copyright and license information, please view the LICENSE.md
+ * file that was distributed with this source code.
+ */
+
 namespace FoF\AntiSpam\Repository;
 
 use Flarum\Settings\SettingsRepositoryInterface;
@@ -8,7 +17,7 @@
 class ChallengeRepository
 {
     protected $settings;
-    
+
     public function __construct(SettingsRepositoryInterface $settings)
     {
         $this->settings = $settings;

tests/integration/api/ChallengeQuestions/AnswerChallengeTest.php

@@ -1,5 +1,14 @@
 <?php
 
+/*
+ * This file is part of fof/anti-spam.
+ *
+ * Copyright (c) FriendsOfFlarum.
+ *
+ * For the full copyright and license information, please view the LICENSE.md
+ * file that was distributed with this source code.
+ */
+
 namespace FoF\AntiSpam\Tests\integration\Api;
 
 use Flarum\Testing\integration\TestCase;
@@ -40,7 +49,7 @@ public function can_answer_challenge_question_incorrectly()
         $this->assertEquals(422, $response->getStatusCode());
 
         $json = json_decode($response->getBody()->getContents(), true);
-        
+
         $this->assertArrayHasKey('errors', $json);
 
         $errors = $json['errors'];
@@ -70,14 +79,10 @@ public function can_answer_challenge_question_correctly()
         $this->assertEquals(200, $response->getStatusCode());
 
         $json = json_decode($response->getBody()->getContents(), true);
-        
+
         $this->assertArrayHasKey('token', $json);
 
         $token = $json['token'];
         $this->assertNotNull($token, 'Correct answer should have a token');
-
-
     }
-
-
 }