bristol-su / assign-roles

app/ModuleServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
 
     public function baseDirectory()
     {
-        return __DIR__ . '/..';
+        return __DIR__.'/..';
     }
 
     public function boot()

app/Rules/RoleHasSpaceToAssign.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
         } catch (\Exception $e) {
             return false;
         }
-        if(in_array($value, $settings['only_one_user'])) {
+        if (in_array($value, $settings['only_one_user'])) {
             return $this->roleRepository->getById($value)->users()->count() === 0;
         }
         return true;

app/Http/Controllers/Controller.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     public function authorize($ability, $arguments = [])
     {
         return $this->baseAuthorize(
-            'assign-roles.' . $ability,
+            'assign-roles.'.$ability,
             $arguments
         );
     }

app/Http/Requests/ParticipantApi/RoleController/StoreRequest.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
     public function rules()
     {
         return [
-            'position_id' => ['required', app(PositionIsAllowed::class),  app(PositionCanBeUsed::class)],
+            'position_id' => ['required', app(PositionIsAllowed::class), app(PositionCanBeUsed::class)],
             'role_name' => ['required', 'string', 'max:255', 'min:3'],
             'email' => ['nullable', 'sometimes', 'email']
         ];

app/Support/LogicRoleRepository.php

Spacing +2 added lines, -2 removed lines

@@ -91,7 +91,7 @@ 
     private function logicGroupId()
     {
         $id = settings('logic_group', null);
-        if($id === null) {
+        if ($id === null) {
             return null;
         }
         return (int) $id;
@@ -111,7 +111,7 @@ 
 
     private function filter(Collection $roles)
     {
-        if($this->hasLogicGroup()) {
+        if ($this->hasLogicGroup()) {
             return $roles->filter(function(\BristolSU\ControlDB\Contracts\Models\Role $role) {
                 return $this->isInLogicGroup($role);
             })->values();

app/Rules/PositionCanBeUsed.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
         } catch (\Exception $e) {
             return false;
         }
-        if(in_array($value, $settings['only_one_role'])) {
+        if (in_array($value, $settings['only_one_role'])) {
             $roles = $this->roleRepository->allThroughGroup($this->group());
             
             return $roles->filter(function(\BristolSU\ControlDB\Contracts\Models\Role $role) use ($value) {

app/CompletionConditions/RequiredPositionsFilled.php

Indentation +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
             return false;
         }
        
-       return count($remainingRoles) === 0;
+        return count($remainingRoles) === 0;
     }
     
     public function percentage($settings, ActivityInstance $activityInstance, ModuleInstance $moduleInstance): int

Spacing +8 added lines, -8 removed lines

@@ -36,9 +36,9 @@ 
     
     public function percentage($settings, ActivityInstance $activityInstance, ModuleInstance $moduleInstance): int
     {
-        if($activityInstance->resource_type === 'group') {
+        if ($activityInstance->resource_type === 'group') {
             $group = $activityInstance->participant();
-        } else if($activityInstance->resource_type === 'role') {
+        } else if ($activityInstance->resource_type === 'role') {
             $group = $activityInstance->participant()->group();
         } else {
             return collect();
@@ -51,15 +51,15 @@ 
             return false;
         }
         
-        if(count($requiredPositions) === 0) {
+        if (count($requiredPositions) === 0) {
             return 100;
         }
         
         $filled = count($requiredPositions) - count($remainingPositions);
         
-        $percentage = (int) round(($filled/count($requiredPositions)) * 100, 0);
+        $percentage = (int) round(($filled / count($requiredPositions)) * 100, 0);
 
-        if($percentage > 100) {
+        if ($percentage > 100) {
             return 100;
         }
         return $percentage;
@@ -67,9 +67,9 @@ 
 
     protected function rolesNeededtoFill($settings, ActivityInstance $activityInstance, ModuleInstance $moduleInstance)
     {
-        if($activityInstance->resource_type === 'group') {
+        if ($activityInstance->resource_type === 'group') {
             $group = $activityInstance->participant();
-        } else if($activityInstance->resource_type === 'role') {
+        } else if ($activityInstance->resource_type === 'role') {
             $group = $activityInstance->participant()->group();
         } else {
             return collect();
@@ -78,7 +78,7 @@ 
         $requiredPositions = $this->getRequiredPositions($settings, $group);
         $roles = $group->roles();
         $logicGroupId = $moduleInstance->setting('logic_group', null);
-        if($logicGroupId !== null) {
+        if ($logicGroupId !== null) {
             $logic = app(LogicRepository::class)->getById($logicGroupId);
             
             $roles = $roles->filter(function(\BristolSU\ControlDB\Contracts\Models\Role $role) use ($logic) {

app/Support/LogicUserGroupRepository.php

Spacing +2 added lines, -2 removed lines

@@ -91,7 +91,7 @@ 
     private function logicGroupId()
     {
         $id = settings('logic_group', null);
-        if($id === null) {
+        if ($id === null) {
             return null;
         }
         return (int) $id;
@@ -111,7 +111,7 @@ 
 
     private function filter(Collection $roles)
     {
-        if($this->hasLogicGroup()) {
+        if ($this->hasLogicGroup()) {
             return $roles->filter(function(\BristolSU\ControlDB\Contracts\Models\Role $role) {
                 return $this->isInLogicGroup($role);
             })->values();

app/Support/RequiredSettingRetrieval.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
 
     public function getSettings(Group $group, array $settings)
     {
-        if(Cache::has($this->getCacheKey($group))) {
+        if (Cache::has($this->getCacheKey($group))) {
             return Cache::get($this->getCacheKey($group));
         }
         foreach ($settings as $setting) {
@@ -50,7 +50,7 @@ 
 
     protected function getCacheKey(Group $group)
     {
-        return RequiredSettingRetrieval::class . '.' . $group->id();
+        return RequiredSettingRetrieval::class.'.'.$group->id();
     }
     
     protected function groupIsForSetting(Group $group, array $setting): bool
@@ -64,7 +64,7 @@ 
         $logic = $this->logicRepository->getById($setting['logic_id']);
         return app(LogicTester::class)->evaluate($logic, $user, $group, $role);
         
-        $groups = collect($this->logicAudience->groupAudience($logic))->map(function (Group $group) {
+        $groups = collect($this->logicAudience->groupAudience($logic))->map(function(Group $group) {
             return $group->id();
         });
         return $groups->contains($group->id());