[2.x] fix(tags): defer policy if min primary & secondary tags 0 (#4279)

* test(tags): implement forum attribute tests

* fix(tags): defer policy if min primary & secondary tags 0

* style(tags): change formatting

* Update extensions/tags/composer.json

---------

Co-authored-by: IanM <[email protected]>

Author: DavideIadeluca

extensions/tags/composer.json

@@ -79,7 +79,8 @@
     },
     "require-dev": {
         "flarum/core": "*@dev",
-        "flarum/testing": "^2.0"
+        "flarum/testing": "^2.0",
+        "flarum/suspend": "^2.0"
     },
     "repositories": [
         {

extensions/tags/src/Access/GlobalPolicy.php

@@ -32,6 +32,15 @@ public function can(User $actor, string $ability): ?string
             return $this->allow();
         }
 
+        if ($ability === 'startDiscussion') {
+            $minPrimaryTags = (int) $this->settings->get('flarum-tags.min_primary_tags');
+            $minSecondaryTags = (int) $this->settings->get('flarum-tags.min_secondary_tags');
+
+            if ($minPrimaryTags === 0 && $minSecondaryTags === 0) {
+                return null;
+            }
+        }
+
         if (in_array($ability, ['viewForum', 'startDiscussion'])) {
             if (! isset($enoughPrimary[$actor->id][$ability])) {
                 $primaryTagsWhereNeedsPermission = $this->settings->get('flarum-tags.min_primary_tags');

extensions/tags/tests/integration/forum/ForumAttributeTest.php

@@ -0,0 +1,92 @@
+<?php
+
+/*
+ * This file is part of Flarum.
+ *
+ * For detailed copyright and license information, please view the
+ * LICENSE file that was distributed with this source code.
+ */
+
+namespace Flarum\Tags\Tests\integration\api\forum;
+
+use Flarum\Tags\Tag;
+use Flarum\Tags\Tests\integration\RetrievesRepresentativeTags;
+use Flarum\Testing\integration\RetrievesAuthorizedUsers;
+use Flarum\Testing\integration\TestCase;
+use Flarum\User\User;
+use Illuminate\Support\Arr;
+use PHPUnit\Framework\Attributes\DataProvider;
+use PHPUnit\Framework\Attributes\Test;
+
+class ForumAttributeTest extends TestCase
+{
+    use RetrievesAuthorizedUsers;
+    use RetrievesRepresentativeTags;
+
+    /**
+     * @inheritDoc
+     */
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        $this->extension('flarum-tags');
+        $this->extension('flarum-suspend');
+
+        $this->prepareDatabase([
+            Tag::class => $this->tags(),
+            User::class => [
+                $this->normalUser(),
+                ['id' => 3, 'username' => 'suspended-user', 'email' => '[email protected]', 'suspended_until' => '2043-11-17 22:05:23', 'is_email_confirmed' => 1],
+            ]
+        ]);
+    }
+
+    public static function canStartDiscussionProvider(): array
+    {
+        return [
+            'admin user, min 0/0' => ['authenticatedAs' => 1, 'minPrimary' => 0, 'minSecondary' => 0, 'expected' => true],
+            'normal user, min 0/0' => ['authenticatedAs' => 2, 'minPrimary' => 0, 'minSecondary' => 0, 'expected' => true],
+            'suspended user, min 0/0' => ['authenticatedAs' => 3, 'minPrimary' => 0, 'minSecondary' => 0, 'expected' => false],
+            'guest user, min 0/0' => ['authenticatedAs' => null, 'minPrimary' => 0, 'minSecondary' => 0, 'expected' => false],
+
+            'admin user, min 1/0' => ['authenticatedAs' => 1, 'minPrimary' => 1, 'minSecondary' => 0, 'expected' => true],
+            'normal user, min 1/0' => ['authenticatedAs' => 2, 'minPrimary' => 1, 'minSecondary' => 0, 'expected' => true],
+            'suspended user, min 1/0' => ['authenticatedAs' => 3, 'minPrimary' => 1, 'minSecondary' => 0, 'expected' => false],
+            'guest user, min 1/0' => ['authenticatedAs' => null, 'minPrimary' => 1, 'minSecondary' => 0, 'expected' => false],
+
+            'admin user, min 0/1' => ['authenticatedAs' => 1, 'minPrimary' => 0, 'minSecondary' => 1, 'expected' => true],
+            'normal user, min 0/1' => ['authenticatedAs' => 2, 'minPrimary' => 0, 'minSecondary' => 1, 'expected' => true],
+            'suspended user, min 0/1' => ['authenticatedAs' => 3, 'minPrimary' => 0, 'minSecondary' => 1, 'expected' => false],
+            'guest user, min 0/1' => ['authenticatedAs' => null, 'minPrimary' => 0, 'minSecondary' => 1, 'expected' => false],
+
+            'admin user, min 1/1' => ['authenticatedAs' => 1, 'minPrimary' => 1, 'minSecondary' => 1, 'expected' => true],
+            'normal user, min 1/1' => ['authenticatedAs' => 2, 'minPrimary' => 1, 'minSecondary' => 1, 'expected' => true],
+            'suspended user, min 1/1' => ['authenticatedAs' => 3, 'minPrimary' => 1, 'minSecondary' => 1, 'expected' => false],
+            'guest user, min 1/1' => ['authenticatedAs' => null, 'minPrimary' => 1, 'minSecondary' => 1, 'expected' => false],
+        ];
+    }
+
+    #[Test]
+    #[DataProvider('canStartDiscussionProvider')]
+    public function itReturnsTheExpectedCanStartDiscussionAttribute(
+        ?int $authenticatedAs,
+        int $minPrimary,
+        int $minSecondary,
+        bool $expected
+    ) {
+        $this->setting('flarum-tags.min_primary_tags', $minPrimary);
+        $this->setting('flarum-tags.min_secondary_tags', $minSecondary);
+
+        $response = $this->send(
+            $this->request('GET', '/api', [
+                'authenticatedAs' => $authenticatedAs,
+            ])
+        );
+
+        $this->assertEquals(200, $response->getStatusCode());
+
+        $json = json_decode($response->getBody()->getContents(), true);
+        $this->assertEquals($expected, Arr::get($json, 'data.attributes.canStartDiscussion'));
+    }
+}