Apply fixes from StyleCI

Author: StyleCIBot

extensions/likes/tests/integration/api/LikePostTest.php

@@ -60,7 +60,7 @@ protected function rewriteDefaultPermissionsAfterBoot()
      * @dataProvider allowedUsersToLike
      * @test
      */
-    public function can_like_a_post_if_allowed(int $postId, ?int $authenticatedAs, string $message, bool $canLikeOwnPost = null)
+    public function can_like_a_post_if_allowed(int $postId, ?int $authenticatedAs, string $message, ?bool $canLikeOwnPost = null)
     {
         if (! is_null($canLikeOwnPost)) {
             $this->setting('flarum-likes.like_own_post', $canLikeOwnPost);
@@ -80,7 +80,7 @@ public function can_like_a_post_if_allowed(int $postId, ?int $authenticatedAs, s
      * @dataProvider unallowedUsersToLike
      * @test
      */
-    public function cannot_like_a_post_if_not_allowed(int $postId, ?int $authenticatedAs, string $message, bool $canLikeOwnPost = null)
+    public function cannot_like_a_post_if_not_allowed(int $postId, ?int $authenticatedAs, string $message, ?bool $canLikeOwnPost = null)
     {
         if (! is_null($canLikeOwnPost)) {
             $this->setting('flarum-likes.like_own_post', $canLikeOwnPost);
@@ -100,7 +100,7 @@ public function cannot_like_a_post_if_not_allowed(int $postId, ?int $authenticat
      * @dataProvider allowedUsersToLike
      * @test
      */
-    public function can_dislike_a_post_if_liked_and_allowed(int $postId, ?int $authenticatedAs, string $message, bool $canLikeOwnPost = null)
+    public function can_dislike_a_post_if_liked_and_allowed(int $postId, ?int $authenticatedAs, string $message, ?bool $canLikeOwnPost = null)
     {
         if (! is_null($canLikeOwnPost)) {
             $this->setting('flarum-likes.like_own_post', $canLikeOwnPost);

extensions/lock/src/Post/DiscussionLockedPost.php

@@ -24,7 +24,7 @@ class DiscussionLockedPost extends AbstractEventPost implements MergeableInterfa
     /**
      * {@inheritdoc}
      */
-    public function saveAfter(Post $previous = null)
+    public function saveAfter(?Post $previous = null)
     {
         // If the previous post is another 'discussion locked' post, and it's
         // by the same user, then we can merge this post into it. If we find

extensions/mentions/src/Formatter/FormatPostMentions.php

@@ -44,7 +44,7 @@ public function __construct(TranslatorInterface $translator, SlugManager $slugMa
      * @param \Psr\Http\Message\ServerRequestInterface|null $request
      * @return string $xml to be rendered
      */
-    public function __invoke(Renderer $renderer, $context, $xml, Request $request = null)
+    public function __invoke(Renderer $renderer, $context, $xml, ?Request $request = null)
     {
         return Utils::replaceAttributes($xml, 'POSTMENTION', function ($attributes) use ($context) {
             $post = (($context && isset($context->getRelations()['mentionsPosts'])) || $context instanceof Post)

extensions/mentions/src/Formatter/FormatTagMentions.php

@@ -17,7 +17,7 @@
 
 class FormatTagMentions
 {
-    public function __invoke(Renderer $renderer, $context, ?string $xml, Request $request = null): string
+    public function __invoke(Renderer $renderer, $context, ?string $xml, ?Request $request = null): string
     {
         return Utils::replaceAttributes($xml, 'TAGMENTION', function ($attributes) use ($context) {
             /** @var Tag|null $tag */

extensions/package-manager/src/Task/TaskRepository.php

@@ -27,7 +27,7 @@ public function query()
      * @param User $actor
      * @return Task
      */
-    public function findOrFail($id, User $actor = null): Task
+    public function findOrFail($id, ?User $actor = null): Task
     {
         return Task::findOrFail($id);
     }

extensions/package-manager/tests/integration/SetupComposer.php

@@ -17,7 +17,7 @@ class SetupComposer
 
     private $config;
 
-    public function __construct(array $config = null)
+    public function __construct(?array $config = null)
     {
         $this->config = $config;
     }

extensions/sticky/src/Post/DiscussionStickiedPost.php

@@ -24,7 +24,7 @@ class DiscussionStickiedPost extends AbstractEventPost implements MergeableInter
     /**
      * {@inheritdoc}
      */
-    public function saveAfter(Post $previous = null)
+    public function saveAfter(?Post $previous = null)
     {
         // If the previous post is another 'discussion stickied' post, and it's
         // by the same user, then we can merge this post into it. If we find

extensions/tags/src/Post/DiscussionTaggedPost.php

@@ -24,7 +24,7 @@ class DiscussionTaggedPost extends AbstractEventPost implements MergeableInterfa
     /**
      * {@inheritdoc}
      */
-    public function saveAfter(Post $previous = null)
+    public function saveAfter(?Post $previous = null)
     {
         // If the previous post is another 'discussion tagged' post, and it's
         // by the same user, then we can merge this post into it. If we find

extensions/tags/src/Tag.php

@@ -137,7 +137,7 @@ public function refreshLastPostedDiscussion()
         return $this;
     }
 
-    public function setLastPostedDiscussion(Discussion $discussion = null)
+    public function setLastPostedDiscussion(?Discussion $discussion = null)
     {
         $this->last_posted_at = optional($discussion)->last_posted_at;
         $this->last_posted_discussion_id = optional($discussion)->id;

extensions/tags/src/TagRepository.php

@@ -78,7 +78,7 @@ public function getAuthorizedRelations($relations, User $actor): array
      * @param User|null $actor
      * @return Tag
      */
-    public function findOrFail($id, User $actor = null)
+    public function findOrFail($id, ?User $actor = null)
     {
         $query = Tag::where('id', $id);
 
@@ -92,7 +92,7 @@ public function findOrFail($id, User $actor = null)
      * @param User|null $user
      * @return \Illuminate\Database\Eloquent\Collection<Tag>
      */
-    public function all(User $user = null)
+    public function all(?User $user = null)
     {
         $query = Tag::query();
 
@@ -106,7 +106,7 @@ public function all(User $user = null)
      * @param User|null $user
      * @return int
      */
-    public function getIdForSlug($slug, User $user = null): ?int
+    public function getIdForSlug($slug, ?User $user = null): ?int
     {
         $query = Tag::where('slug', $slug);