diff --git a/extensions/tags/src/Access/DiscussionPolicy.php b/extensions/tags/src/Access/DiscussionPolicy.php index 1a18154b7..199cf77a6 100755 --- a/extensions/tags/src/Access/DiscussionPolicy.php +++ b/extensions/tags/src/Access/DiscussionPolicy.php @@ -23,7 +23,6 @@ use Illuminate\Database\Query\Expression; class DiscussionPolicy extends AbstractPolicy { - /** * {@inheritdoc} */ @@ -73,7 +72,7 @@ class DiscussionPolicy extends AbstractPolicy foreach ($tags as $tag) { if ($tag->is_restricted) { - if ($actor->hasPermission('tag' . $tag->id . '.discussion.' . $ability)) { + if ($actor->hasPermission('tag'.$tag->id.'.discussion.'.$ability)) { return true; } } else { diff --git a/extensions/tags/src/Access/TagPolicy.php b/extensions/tags/src/Access/TagPolicy.php index a5fc40844..c64ba2a93 100755 --- a/extensions/tags/src/Access/TagPolicy.php +++ b/extensions/tags/src/Access/TagPolicy.php @@ -39,7 +39,7 @@ class TagPolicy extends AbstractPolicy public function startDiscussion(User $actor, Tag $tag) { if ((! $tag->is_restricted && $actor->hasPermission('startDiscussion')) - || $actor->hasPermission('tag' . $tag->id . '.startDiscussion')) { + || $actor->hasPermission('tag'.$tag->id.'.startDiscussion')) { return true; } } diff --git a/extensions/tags/src/Api/Controller/CreateTagController.php b/extensions/tags/src/Api/Controller/CreateTagController.php index 98f1b0d67..c9e1e61c8 100644 --- a/extensions/tags/src/Api/Controller/CreateTagController.php +++ b/extensions/tags/src/Api/Controller/CreateTagController.php @@ -1,4 +1,5 @@ -hasOne($tag, TagSerializer::class); + return $this->hasOne($tag, self::class); } /** diff --git a/extensions/tags/src/Command/CreateTag.php b/extensions/tags/src/Command/CreateTag.php index 2109a63ef..3154edf2f 100644 --- a/extensions/tags/src/Command/CreateTag.php +++ b/extensions/tags/src/Command/CreateTag.php @@ -1,4 +1,5 @@ -content = static::buildContent($oldTagIds, $newTagIds); - $post->time = time(); + $post->content = static::buildContent($oldTagIds, $newTagIds); + $post->time = time(); $post->discussion_id = $discussionId; - $post->user_id = $userId; + $post->user_id = $userId; return $post; } diff --git a/extensions/tags/src/Tag.php b/extensions/tags/src/Tag.php index d3ece4dd7..de099afe0 100644 --- a/extensions/tags/src/Tag.php +++ b/extensions/tags/src/Tag.php @@ -57,11 +57,11 @@ class Tag extends AbstractModel { $tag = new static; - $tag->name = $name; - $tag->slug = $slug; + $tag->name = $name; + $tag->slug = $slug; $tag->description = $description; - $tag->color = $color; - $tag->is_hidden = (bool) $isHidden; + $tag->color = $color; + $tag->is_hidden = (bool) $isHidden; return $tag; } @@ -112,7 +112,7 @@ class Tag extends AbstractModel */ public function setLastDiscussion(Discussion $discussion) { - $this->last_time = $discussion->last_time; + $this->last_time = $discussion->last_time; $this->last_discussion_id = $discussion->id; return $this; @@ -135,7 +135,7 @@ class Tag extends AbstractModel $hasGlobalPermission = $user->hasPermission($permission); foreach ($tags as $tag) { - if (($hasGlobalPermission && ! $tag->is_restricted) || $user->hasPermission('tag' . $tag->id . '.' . $permission)) { + if (($hasGlobalPermission && ! $tag->is_restricted) || $user->hasPermission('tag'.$tag->id.'.'.$permission)) { $ids[] = $tag->id; } } @@ -160,7 +160,7 @@ class Tag extends AbstractModel $hasGlobalPermission = $user->hasPermission($permission); foreach ($tags as $tag) { - if (($tag->is_restricted || ! $hasGlobalPermission) && ! $user->hasPermission('tag' . $tag->id . '.' . $permission)) { + if (($tag->is_restricted || ! $hasGlobalPermission) && ! $user->hasPermission('tag'.$tag->id.'.'.$permission)) { $ids[] = $tag->id; } } diff --git a/extensions/tags/src/TagRepository.php b/extensions/tags/src/TagRepository.php index 755139035..76687ec5e 100644 --- a/extensions/tags/src/TagRepository.php +++ b/extensions/tags/src/TagRepository.php @@ -1,4 +1,5 @@ -