feat: select own post permission

Author: imorland

js/src/admin/components/BestAnswerSettingsPage.tsx

@@ -31,12 +31,6 @@ export default class BestAnswerSettingsPage extends ExtensionPage {
             </div>
             <h3>{app.translator.trans('fof-best-answer.admin.settings.label.general')}</h3>
             <div className="Section">
-              {this.buildSettingComponent({
-                type: 'boolean',
-                setting: 'fof-best-answer.allow_select_own_post',
-                label: app.translator.trans('fof-best-answer.admin.settings.allow_select_own_post'),
-                help: app.translator.trans('fof-best-answer.admin.settings.allow_select_own_post_help'),
-              })}
               {this.buildSettingComponent({
                 type: 'boolean',
                 setting: 'fof-best-answer.use_alternative_ui',

js/src/admin/extend.ts

@@ -1,3 +1,4 @@
+import app from 'flarum/admin/app';
 import Extend from 'flarum/common/extenders';
 import BestAnswerSettingsPage from './components/BestAnswerSettingsPage';
 import commonExtend from '../common/extend';
@@ -22,5 +23,13 @@ export default [
         permission: 'discussion.selectBestAnswerNotOwnDiscussion',
       }),
       'reply'
+    )
+    .permission(
+      () => ({
+        icon : 'fas fa-check',
+        label: app.translator.trans('fof-best-answer.admin.permissions.allow_select_own_post'),
+        permission: 'discussion.fof-best-answer.allow_select_own_post'
+      }),
+      'reply'
     ),
 ];

js/src/forum/addBestAnswerAction.tsx

@@ -13,11 +13,6 @@ export default function addBestAnswerAction() {
     return post.isHidden() || post.number() === 1 || !discussion.canSelectBestAnswer() || !app.session.user;
   };
 
-  const blockSelectOwnPost = (post: Post): boolean => {
-    const user = post.user();
-    return !app.forum.attribute<boolean>('canSelectBestAnswerOwnPost') && user !== false && user.id() === app.session.user?.id();
-  };
-
   const isThisBestAnswer = (discussion: Discussion, post: Post): boolean => {
     const bAPost = discussion.bestAnswerPost();
     const hasBestAnswer = discussion.hasBestAnswer();
@@ -72,7 +67,7 @@ export default function addBestAnswerAction() {
 
     if (post.contentType() !== 'comment') return;
 
-    if (ineligible(discussion, post) || blockSelectOwnPost(post) || !app.current.matches(DiscussionPage)) return;
+    if (ineligible(discussion, post) || !app.current.matches(DiscussionPage)) return;
 
     items.add(
       'bestAnswer',
@@ -101,7 +96,7 @@ export default function addBestAnswerAction() {
 
     post.pushAttributes({ isBestAnswer });
 
-    if (ineligible(discussion, post) || blockSelectOwnPost(post) || !app.current.matches(DiscussionPage)) return;
+    if (ineligible(discussion, post) || !app.current.matches(DiscussionPage)) return;
 
     items.add(
       'bestAnswer',

resources/locale/en.yml

@@ -3,15 +3,14 @@ fof-best-answer:
     permissions:
       best_answer: Select Best Answer (own Discussion)
       best_answer_not_own_discussion: Select Best Answer (not own Discussion)
+      allow_select_own_post: Select own post as Best Answer
     settings:
       label:
         tags: Best Answer Tags
         general: General
         reminders: Reminders
         advanced: Advanced
         reminders_notice: For reminders to function, you must have set up the Flarum scheduler correctly.
-      allow_select_own_post: Select own post
-      allow_select_own_post_help: Allow a user to select their own post as a best answer to a discussion
       show_max_lines_label: Max lines to show in post preview
       show_max_lines_help: Set to 0 to disable. If a post is longer than the configured amount of lines, it will be truncated in the post preview with a fade out effect.
       select_best_answer_reminder_days: Reminder frequency

src/Api/ForumAttributes.php

@@ -11,6 +11,7 @@
 
 namespace FoF\BestAnswer\Api;
 
+use Flarum\Api\Context;
 use Flarum\Api\Schema;
 use Flarum\Settings\SettingsRepositoryInterface;
 
@@ -24,9 +25,6 @@ public function __construct(
     public function __invoke(): array
     {
         return [
-            Schema\Boolean::make('canSelectBestAnswerOwnPost')
-                ->get(fn () => (bool) $this->settings->get('fof-best-answer.allow_select_own_post')),
-
             Schema\Boolean::make('bestAnswerDiscussionSidebarJumpButton')
                 ->get(fn () => (bool) $this->settings->get('fof-best-answer.discussion_sidebar_jump_button')),
 

src/Repository/BestAnswerRepository.php

@@ -50,7 +50,7 @@ public function canSelectPostAsBestAnswer(User $user, Post $post): bool
         }
 
         if ($user->id === $post->user_id) {
-            return (bool) $this->settings->get('fof-best-answer.allow_select_own_post');
+            return $user->can('discussion.fof-best-answer.allow_select_own_post', $post->discussion);
         }
 
         return true;

tests/integration/api/SetBestAnswerTest.php

@@ -59,6 +59,7 @@ public function setUp(): void
             ],
             'group_permission' => [
                 ['group_id' => 4, 'permission' => 'discussion.selectBestAnswerNotOwnDiscussion', 'created_at' => Carbon::now()],
+                ['group_id' => 4, 'permission' => 'discussion.fof-best-answer.allow_select_own_post', 'created_at' => Carbon::now()]
             ],
             'group_user' => [
                 ['user_id' => 4, 'group_id' => 4],
@@ -166,20 +167,36 @@ public function user_without_permission_cannot_set_best_answer(int $userId)
         $this->assertEquals(403, $response->getStatusCode());
     }
 
+    public static function unauthorizedUsersOwnPostProvider(): array
+    {
+        return [
+            [2],
+            [3],
+        ];
+    }
+
     #[Test]
-    public function user_cannot_set_own_post_as_best_answer_if_not_permitted()
+    #[DataProvider('unauthorizedUsersOwnPostProvider')]
+    public function user_cannot_set_own_post_as_best_answer_if_not_permitted(int $userId)
     {
-        $response = $this->setBestAnswer(3, 5, 2);
+        $response = $this->setBestAnswer($userId, 5, 2);
 
         $this->assertEquals(403, $response->getStatusCode());
     }
 
-    #[Test]
-    public function user_can_set_own_post_as_best_answer_if_permitted()
+    public static function permittedUsersOwnPostProvider(): array
     {
-        $this->setting('fof-best-answer.allow_select_own_post', true);
+        return [
+            [1],
+            [4],
+        ];
+    }
 
-        $response = $this->setBestAnswer(3, 5, 2);
+    #[Test]
+    #[DataProvider('permittedUsersOwnPostProvider')]
+    public function user_can_set_own_post_as_best_answer_if_permitted(int $userId)
+    {
+        $response = $this->setBestAnswer($userId, 5, 2);
 
         $this->assertEquals(200, $response->getStatusCode());