1 | <?php |
||
2 | |||
0 ignored issues
–
show
Coding Style
introduced
by
![]() |
|||
3 | namespace BristolSU\ControlDB\Cache\Pivots; |
||
4 | |||
5 | use BristolSU\ControlDB\Contracts\Models\Group; |
||
6 | use BristolSU\ControlDB\Contracts\Models\User; |
||
7 | use BristolSU\ControlDB\Contracts\Repositories\Pivots\UserGroup as UserGroupContract; |
||
8 | use Illuminate\Contracts\Cache\Repository; |
||
9 | use Illuminate\Support\Collection; |
||
10 | |||
11 | class UserGroup implements UserGroupContract |
||
0 ignored issues
–
show
|
|||
12 | { |
||
13 | |||
14 | /** |
||
0 ignored issues
–
show
|
|||
15 | * @var UserGroupContract |
||
16 | */ |
||
17 | private $userGroup; |
||
0 ignored issues
–
show
|
|||
18 | /** |
||
0 ignored issues
–
show
|
|||
19 | * @var Repository |
||
20 | */ |
||
21 | private $cache; |
||
0 ignored issues
–
show
|
|||
22 | |||
23 | 25 | public function __construct(UserGroupContract $userGroup, Repository $cache) |
|
0 ignored issues
–
show
|
|||
24 | { |
||
25 | 25 | $this->userGroup = $userGroup; |
|
26 | 25 | $this->cache = $cache; |
|
27 | 25 | } |
|
28 | |||
29 | /** |
||
30 | * Get all users who are members of a group |
||
31 | * |
||
32 | * @param Group $group |
||
0 ignored issues
–
show
|
|||
33 | * @return User[]|Collection Users who are members of the group |
||
0 ignored issues
–
show
|
|||
34 | */ |
||
35 | 4 | public function getUsersThroughGroup(Group $group): Collection |
|
36 | { |
||
37 | return $this->cache->rememberForever(static::class . '@getUsersThroughGroup:' . $group->id(), function() use ($group) { |
||
0 ignored issues
–
show
|
|||
38 | 4 | return $this->userGroup->getUsersThroughGroup($group); |
|
39 | 4 | }); |
|
0 ignored issues
–
show
For multi-line function calls, the closing parenthesis should be on a new line.
If a function call spawns multiple lines, the coding standard suggests to move the closing parenthesis to a new line: someFunctionCall(
$firstArgument,
$secondArgument,
$thirdArgument
); // Closing parenthesis on a new line.
![]() |
|||
40 | } |
||
41 | |||
42 | /** |
||
43 | * Get all groups a user belongs to |
||
44 | * |
||
45 | * @param User $user |
||
0 ignored issues
–
show
|
|||
46 | * @return Group[]|Collection Groups the user is a member of |
||
0 ignored issues
–
show
|
|||
47 | */ |
||
48 | 4 | public function getGroupsThroughUser(User $user): Collection |
|
49 | { |
||
50 | return $this->cache->rememberForever(static::class . '@getGroupsThroughUser:' . $user->id(), function() use ($user) { |
||
0 ignored issues
–
show
|
|||
51 | 4 | return $this->userGroup->getGroupsThroughUser($user); |
|
52 | 4 | }); |
|
0 ignored issues
–
show
For multi-line function calls, the closing parenthesis should be on a new line.
If a function call spawns multiple lines, the coding standard suggests to move the closing parenthesis to a new line: someFunctionCall(
$firstArgument,
$secondArgument,
$thirdArgument
); // Closing parenthesis on a new line.
![]() |
|||
53 | } |
||
54 | |||
55 | /** |
||
56 | * Add a user to a group |
||
57 | * |
||
58 | * @param User $user User to add to the group |
||
0 ignored issues
–
show
|
|||
59 | * @param Group $group Group to add the user to |
||
60 | * @return void |
||
0 ignored issues
–
show
|
|||
61 | */ |
||
62 | 12 | public function addUserToGroup(User $user, Group $group): void |
|
63 | { |
||
64 | 12 | $this->userGroup->addUserToGroup($user, $group); |
|
65 | 12 | } |
|
66 | |||
67 | /** |
||
68 | * Remove a user from a group |
||
69 | * @param User $user User to remove from the group |
||
0 ignored issues
–
show
|
|||
70 | * @param Group $group Group to remove the user from |
||
71 | * @return void |
||
0 ignored issues
–
show
|
|||
72 | */ |
||
73 | 5 | public function removeUserFromGroup(User $user, Group $group): void |
|
74 | { |
||
75 | 5 | $this->userGroup->removeUserFromGroup($user, $group); |
|
76 | } |
||
77 | } |