bristol-su / upload-file

app/CompletionConditions/NumberOfDocumentsSubmitted.php

Spacing +3 added lines, -3 removed lines

@@ -19,11 +19,11 @@
     public function percentage($settings, ActivityInstance $activityInstance, ModuleInstance $moduleInstance): int
     {
         $count = File::forResource($activityInstance->id, $moduleInstance->id)->count();
-        $needed = ( $settings['number_of_files'] ?? 1);
+        $needed = ($settings['number_of_files'] ?? 1);
 
-        $percentage = (int) round(($count/$needed) * 100, 0);
+        $percentage = (int) round(($count / $needed) * 100, 0);
 
-        if($percentage > 100) {
+        if ($percentage > 100) {
             return 100;
         }
         return $percentage;

app/CompletionConditions/NumberOfDocumentsWithStatus.php

Spacing +3 added lines, -3 removed lines

@@ -23,11 +23,11 @@
         $count = File::forResource($activityInstance->id, $moduleInstance->id())->get()->filter(function(File $file) use ($settings) {
             return $file->status === $settings['status'];
         })->count();
-        $needed = ( $settings['number_of_files'] ?? 1);
+        $needed = ($settings['number_of_files'] ?? 1);
 
-        $percentage = (int) round(($count/$needed) * 100, 0);
+        $percentage = (int) round(($count / $needed) * 100, 0);
 
-        if($percentage > 100) {
+        if ($percentage > 100) {
             return 100; 
         }
         return $percentage;

app/Http/Controllers/ParticipantApi/FileController.php

Spacing +3 added lines, -3 removed lines

@@ -52,7 +52,7 @@ 
     {
         $this->authorize('file.destroy');
 
-        if((int) $file->activity_instance_id !== (int) app(ActivityInstanceResolver::class)->getActivityInstance()->id) {
+        if ((int) $file->activity_instance_id !== (int) app(ActivityInstanceResolver::class)->getActivityInstance()->id) {
             throw new AuthorizationException();
         }
                 
@@ -68,7 +68,7 @@ 
     {
         $this->authorize('file.index');
 
-        if((int) $file->activity_instance_id !== (int) app(ActivityInstanceResolver::class)->getActivityInstance()->id) {
+        if ((int) $file->activity_instance_id !== (int) app(ActivityInstanceResolver::class)->getActivityInstance()->id) {
             throw new AuthorizationException();
         }
         
@@ -79,7 +79,7 @@ 
     {
         $this->authorize('file.update');
 
-        if((int) $file->activity_instance_id !== (int) app(ActivityInstanceResolver::class)->getActivityInstance()->id) {
+        if ((int) $file->activity_instance_id !== (int) app(ActivityInstanceResolver::class)->getActivityInstance()->id) {
             throw new AuthorizationException();
         }
         

app/Http/Controllers/ParticipantApi/CommentController.php

Spacing +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
     public function index(Request $request, Activity $activity, ModuleInstance $moduleInstance, File $file)
     {
         $this->authorize('comment.index');
-        if ((int)$file->activity_instance_id !== (int)app(ActivityInstanceResolver::class)->getActivityInstance()->id) {
+        if ((int) $file->activity_instance_id !== (int) app(ActivityInstanceResolver::class)->getActivityInstance()->id) {
             throw new AuthorizationException();
         }
         return $file->comments;
@@ -45,7 +45,7 @@ 
     {
         $this->authorize('comment.destroy');
 
-        if ((int)$comment->file->activity_instance_id !== (int)app(ActivityInstanceResolver::class)->getActivityInstance()->id) {
+        if ((int) $comment->file->activity_instance_id !== (int) app(ActivityInstanceResolver::class)->getActivityInstance()->id) {
             throw new AuthorizationException();
         }
 
@@ -60,7 +60,7 @@ 
     {
         $this->authorize('comment.update');
 
-        if ((int)$comment->file->activity_instance_id !== (int)app(ActivityInstanceResolver::class)->getActivityInstance()->id) {
+        if ((int) $comment->file->activity_instance_id !== (int) app(ActivityInstanceResolver::class)->getActivityInstance()->id) {
             throw new AuthorizationException();
         }
 

app/Http/Controllers/Participant/ParticipantPageController.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
     {
         $this->authorize('view-page');
         
-        return view(alias() . '::participant');
+        return view(alias().'::participant');
     }
     
 }
\ No newline at end of file

app/Http/Controllers/Controller.php

Spacing +1 added lines, -1 removed lines

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

app/Http/Controllers/Admin/DownloadFileController.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
     {
         $this->authorize('admin.file.download');
         
-        if(Storage::exists($file->path)) {
+        if (Storage::exists($file->path)) {
             return Storage::download($file->path, $file->filename, [
                 'X-Vapor-Base64-Encode' => 'True'
             ]);

database/factories/FileFactory.php

Spacing +3 added lines, -3 removed lines

@@ -3,7 +3,7 @@ 
 use BristolSU\Module\UploadFile\Models\File;
 use Faker\Generator as Faker;
 
-$factory->define(File::class, function (Faker $faker) {
+$factory->define(File::class, function(Faker $faker) {
     return [
         'title' => $faker->sentence,
         'description' => $faker->sentence,
@@ -14,10 +14,10 @@ 
         'uploaded_by' => function() {
             return factory(\BristolSU\ControlDB\Models\User::class)->create()->id();
         },
-        'module_instance_id' => function () {
+        'module_instance_id' => function() {
             return factory(\BristolSU\Support\ModuleInstance\ModuleInstance::class)->create()->id;
         },
-        'activity_instance_id' => function () {
+        'activity_instance_id' => function() {
             return factory(\BristolSU\Support\ActivityInstance\ActivityInstance::class)->create()->id;
         },
         'tags' => $faker->randomElements(['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j'], 4)

app/ModuleServiceProvider.php

Indentation +1 added lines, -1 removed lines

@@ -256,7 +256,7 @@
             )->withField(
                 Field::checkList('allowed_extensions')->label('Allowed file types')->hint('Which file types can be uploaded?')
                     ->listBox(true)->values($this->app['config']->get($this->alias() . '.file_types'))
-		                ->default(['doc', 'docx', 'odt', 'rtf', 'txt', 'csv', 'ppt', 'pptx', 'pdf', 'xls'])
+                        ->default(['doc', 'docx', 'odt', 'rtf', 'txt', 'csv', 'ppt', 'pptx', 'pdf', 'xls'])
             )
         )->withGroup(
             Group::make('Status Changes')->withField(

Spacing +9 added lines, -9 removed lines

@@ -193,7 +193,7 @@ 
     
     public function baseDirectory()
     {
-        return __DIR__ . '/..';
+        return __DIR__.'/..';
     }
 
     public function register()
@@ -216,7 +216,7 @@ 
         
         Route::bind('uploadfile_file', function($id) {
             $file = File::findOrFail($id);
-            if(request()->route('module_instance_slug') && (int) $file->module_instance_id === request()->route('module_instance_slug')->id()) {
+            if (request()->route('module_instance_slug') && (int) $file->module_instance_id === request()->route('module_instance_slug')->id()) {
                 return $file;
             }
             throw (new ModelNotFoundException)->setModel(File::class);
@@ -227,7 +227,7 @@ 
             
             $currentActivityInstance = app(ActivityInstanceResolver::class)->getActivityInstance();
             $fileActivityInstance = $file->activityInstance();
-            if($currentActivityInstance->resource_type === $fileActivityInstance->resource_type
+            if ($currentActivityInstance->resource_type === $fileActivityInstance->resource_type
                 && (int) $currentActivityInstance->resource_id === (int) $fileActivityInstance->resource_id) {
                 return $file;
             }
@@ -236,7 +236,7 @@ 
         
         Route::bind('uploadfile_file_status', function($id) {
             $fileStatus = FileStatus::findOrFail($id);
-            if(request()->route('module_instance_slug') && (int) $fileStatus->file->module_instance_id === request()->route('module_instance_slug')->id()) {
+            if (request()->route('module_instance_slug') && (int) $fileStatus->file->module_instance_id === request()->route('module_instance_slug')->id()) {
                 return $fileStatus;
             }
             throw (new ModelNotFoundException)->setModel(FileStatus::class);
@@ -244,7 +244,7 @@ 
         
         Route::bind('uploadfile_comment', function($id) {
             $comment = Comment::findOrFail($id);
-            if(request()->route('module_instance_slug') && (int) $comment->file->module_instance_id === request()->route('module_instance_slug')->id()) {
+            if (request()->route('module_instance_slug') && (int) $comment->file->module_instance_id === request()->route('module_instance_slug')->id()) {
                 return $comment;
             }
             throw (new ModelNotFoundException)->setModel(Comment::class);
@@ -264,20 +264,20 @@ 
             Group::make('New Documents')->withField(
                 Field::input('document_title')->inputType('text')->label('Default document title')->hint('This will be the default title of a new file')->default('Document')
             )->withField(
-                Field::switch('multiple_files')->label('Multiple Files')->hint('Should multiple files be able to be uploaded at the same time?')
+                Field::switch ('multiple_files')->label('Multiple Files')->hint('Should multiple files be able to be uploaded at the same time?')
                     ->textOn('Allow')->textOff('Do not allow')->default(true)
             )->withField(
                 Field::checkList('allowed_extensions')->label('Allowed file types')->hint('Which file types can be uploaded?')
-                    ->listBox(true)->values($this->app['config']->get($this->alias() . '.file_types'))
+                    ->listBox(true)->values($this->app['config']->get($this->alias().'.file_types'))
 		                ->default(['doc', 'docx', 'odt', 'rtf', 'txt', 'csv', 'ppt', 'pptx', 'pdf', 'xls'])
             )
         )->withGroup(
             Group::make('Status Changes')->withField(
                 Field::input('initial_status')->inputType('select')->label('Initial Status')->hint('What status should all new files have?')
-                        ->default('Awaiting Approval')->values($this->app['config']->get($this->alias() . '.statuses'))
+                        ->default('Awaiting Approval')->values($this->app['config']->get($this->alias().'.statuses'))
             )->withField(
                 Field::checkList('statuses')->label('Available Statuses')->hint('A list of available statuses')
-                    ->listBox(true)->values($this->app['config']->get($this->alias() . '.statuses'))
+                    ->listBox(true)->values($this->app['config']->get($this->alias().'.statuses'))
             )
         )->withGroup(
             Group::make('Tags')->withField(