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/Support/PositionSettingRetrieval.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
 
     protected function getCacheKey(Group $group)
     {
-        return PositionSettingRetrieval::class . '.' . $group->id();
+        return PositionSettingRetrieval::class.'.'.$group->id();
     }
     
     public function parse($setting)

app/CompletionConditions/RequiredPositionsFilled.php

Indentation +1 added lines, -1 removed lines

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

Spacing +6 added lines, -6 removed lines

@@ -59,15 +59,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;
@@ -152,7 +152,7 @@ 
      */
     private function getGroup(ActivityInstance $activityInstance)
     {
-        if($activityInstance->resource_type === 'group') {
+        if ($activityInstance->resource_type === 'group') {
             return $activityInstance->participant();
         }
         return $activityInstance->participant()->group();
@@ -166,7 +166,7 @@ 
     private function rolesThroughGroup(Group $group, ModuleInstance $moduleInstance)
     {
         $roles = $group->roles();
-        if($this->logicGroupId($moduleInstance) !== null) {
+        if ($this->logicGroupId($moduleInstance) !== null) {
             $logicGroup = app(LogicRepository::class)->getById($this->logicGroupId($moduleInstance));
             return $roles->filter(function(\BristolSU\ControlDB\Contracts\Models\Role $role) use ($moduleInstance, $logicGroup) {
                 return LogicTester::evaluate($logicGroup, null, $role->group(), $role);
@@ -178,7 +178,7 @@ 
     private function logicGroupId(ModuleInstance $moduleInstance)
     {
         $id = $moduleInstance->setting('logic_group', null);
-        if($id === null) {
+        if ($id === null) {
             return null;
         }
         return (int) $id;

app/Support/RequiredSettingRetrieval.php

Spacing +3 added lines, -3 removed lines

@@ -34,10 +34,10 @@ 
 
     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 ((array_key_exists('required', $settings)?$settings['required']:[]) as $setting) {
+        foreach ((array_key_exists('required', $settings) ? $settings['required'] : []) as $setting) {
             if ($this->groupIsForSetting($group, $setting)) {
                 Cache::put($this->getCacheKey($group), $setting['required'], 7200);
                 return $setting['required'];
@@ -48,7 +48,7 @@ 
 
     protected function getCacheKey(Group $group)
     {
-        return RequiredSettingRetrieval::class . '.' . $group->id();
+        return RequiredSettingRetrieval::class.'.'.$group->id();
     }
     
     protected function groupIsForSetting(Group $group, array $setting): bool