thinktomorrow / chief

src/Mediagallery/Application/RemovalAction.php

Spacing +6 added lines, -6 removed lines

@@ -22,24 +22,24 @@
     public function handle(Request $request)
     {
         // Strict protection enabled: we won't remove assets who are still being used...
-        $assetIds = collect($request->input('asset_ids', []))->reject(function($assetId){
+        $assetIds = collect($request->input('asset_ids', []))->reject(function($assetId) {
             return DB::table('asset_pivots')
                 ->where('asset_id', $assetId)
                 ->where('unused', 0)
                 ->exists();
         })->toArray();
 
-        if($assetIds)
+        if ($assetIds)
         {
-                foreach($assetIds as $k => $assetId) {
-                    try{
+                foreach ($assetIds as $k => $assetId) {
+                    try {
                         $this->deleteAsset->remove($assetId);
-                    } catch(FileNotAccessibleException $e) {
+                    } catch (FileNotAccessibleException $e) {
                         unset($assetIds[$k]); // So our count of removed assets is correct in the log.
                     }
                 }
 
-                Audit::activity()->log('removed ' . count($assetIds) . ' assets from the mediagallery.');
+                Audit::activity()->log('removed '.count($assetIds).' assets from the mediagallery.');
 
                 return true;
         }

src/Mediagallery/Application/ZipAction.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
 
     public function handle(string $filename, Request $request)
     {
-        $assets = Asset::whereIn('id', (array) $request->input('asset_ids', []))->get();
+        $assets = Asset::whereIn('id', (array)$request->input('asset_ids', []))->get();
 
         // enable output of HTTP headers TODO: this should be moved to the controller instead...
         $options = new Archive();
@@ -26,7 +26,7 @@ 
         $zip = new ZipStream($filename, $options);
 
         $assets->each(function(Asset $asset) use($zip){
-            if(file_exists($asset->getFirstMediaPath())) {
+            if (file_exists($asset->getFirstMediaPath())) {
                 $zip->addFileFromPath($asset->filename(), $asset->getFirstMediaPath());
             }
         });

src/Mediagallery/Http/BulkActionsController.php

Spacing +4 added lines, -4 removed lines

@@ -24,18 +24,18 @@ 
 
     public function bulk(Request $request)
     {
-        if($request->input('type') == 'download') {
+        if ($request->input('type') == 'download') {
             return $this->download($request);
         }
 
-        if($request->input('type') == 'remove') {
+        if ($request->input('type') == 'remove') {
             return $this->remove($request);
         }
     }
 
     private function download(Request $request)
     {
-        $filename = Str::slug(config('app.name')) . '_assets_' . date('YmdHis') . '.zip';
+        $filename = Str::slug(config('app.name')).'_assets_'.date('YmdHis').'.zip';
 
         // TODO: do the response here instead of the zip action itself
         return $this->zipAction->handle($filename, $request);
@@ -45,7 +45,7 @@ 
     {
         $result = $this->removalAction->handle($request);
 
-        if($result) {
+        if ($result) {
             return redirect()->back()->with('messages.success', 'De mediabestanden zijn verwijderd');
         }
 

src/Urls/ChiefResponse.php

Spacing +4 added lines, -4 removed lines

@@ -39,7 +39,7 @@ 
                     $targetModel = Morphables::instance($targetUrlRecord->model_type)->find($targetUrlRecord->model_id);
 
                     if (!$targetModel) {
-                        throw new ArchivedUrlException('Corrupt target model for this url request. Model by reference [' . $targetUrlRecord->model_type . '@' . $targetUrlRecord->model_id . '] has probably been archived or deleted.');
+                        throw new ArchivedUrlException('Corrupt target model for this url request. Model by reference ['.$targetUrlRecord->model_type.'@'.$targetUrlRecord->model_id.'] has probably been archived or deleted.');
                     }
 
                     return static::createRedirect($targetModel->url($locale));
@@ -49,14 +49,14 @@ 
             }
 
             if (!$model) {
-                throw new ArchivedUrlException('Corrupt target model for this url request. Model by reference [' . $urlRecord->model_type . '@' . $urlRecord->model_id . '] has probably been archived or deleted.');
+                throw new ArchivedUrlException('Corrupt target model for this url request. Model by reference ['.$urlRecord->model_type.'@'.$urlRecord->model_id.'] has probably been archived or deleted.');
             }
 
             if (method_exists($model, 'isPublished') && !$model->isPublished()) {
 
                 /** When admin is logged in and this request is in preview mode, we allow the view */
                 if (!PreviewMode::fromRequest()->check()) {
-                    throw new NotFoundHttpException('Model found for request [' . $slug . '] but it is not published.');
+                    throw new NotFoundHttpException('Model found for request ['.$slug.'] but it is not published.');
                 }
             }
 
@@ -67,7 +67,7 @@ 
             }
         }
 
-        throw new NotFoundHttpException('No url or model found for request [' . $slug . '] for locale [' . $locale . '].');
+        throw new NotFoundHttpException('No url or model found for request ['.$slug.'] for locale ['.$locale.'].');
     }
 
     private static function createRedirect(string $url)

src/OLD/Management/Assistants/ManagesAssistants.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
     public function assistant(string $assistantKey): Assistant
     {
         if (!$this->isAssistedBy($assistantKey)) {
-            throw new MissingAssistant('No assistant [' . $assistantKey . '] registered on manager ' . static::class);
+            throw new MissingAssistant('No assistant ['.$assistantKey.'] registered on manager '.static::class);
         }
 
         $instance = app($this->getAssistantClass($assistantKey));

src/OLD/Management/Register.php

Spacing +1 added lines, -1 removed lines

@@ -136,7 +136,7 @@
     private function registrationMustExistConstraint(string $key, $value, $registrations): void
     {
         if (empty($registrations) && count($registrations) != $this->registrations) {
-            throw new NonRegisteredManager('No manager found for ' . $key . ' [' . print_r($value, true) . ']. Did you perhaps forgot to register the manager?');
+            throw new NonRegisteredManager('No manager found for '.$key.' ['.print_r($value, true).']. Did you perhaps forgot to register the manager?');
         }
     }
 }

src/OLD/Management/Registration.php

Spacing +4 added lines, -4 removed lines

@@ -75,21 +75,21 @@
     private function validate($managerClass, $modelClass)
     {
         if (!class_exists($managerClass)) {
-            throw new \InvalidArgumentException('Manager class [' . $managerClass . '] is an invalid class reference. Please make sure the class exists.');
+            throw new \InvalidArgumentException('Manager class ['.$managerClass.'] is an invalid class reference. Please make sure the class exists.');
         }
 
         if (!class_exists($modelClass)) {
-            throw new \InvalidArgumentException('Model class [' . $modelClass . '] is an invalid model reference. Please make sure the class exists.');
+            throw new \InvalidArgumentException('Model class ['.$modelClass.'] is an invalid model reference. Please make sure the class exists.');
         }
 
         $manager = new \ReflectionClass($managerClass);
         if (!$manager->implementsInterface(Manager::class)) {
-            throw new \InvalidArgumentException('Class [' . $managerClass . '] is expected to implement the [' . Manager::class . '] contract.');
+            throw new \InvalidArgumentException('Class ['.$managerClass.'] is expected to implement the ['.Manager::class.'] contract.');
         }
 
         $model = new \ReflectionClass($modelClass);
         if (!$model->implementsInterface(ManagedModel::class)) {
-            throw new \InvalidArgumentException('Class [' . $modelClass . '] is expected to implement the [' . ManagedModel::class . '] contract.');
+            throw new \InvalidArgumentException('Class ['.$modelClass.'] is expected to implement the ['.ManagedModel::class.'] contract.');
         }
     }
 }

src/OLD/Management/Managers.php

Spacing +4 added lines, -4 removed lines

@@ -50,7 +50,7 @@ 
     {
         $registrations = collect($this->register->filterByTag($tag)->all());
 
-        return $registrations->map(function ($registration) {
+        return $registrations->map(function($registration) {
             return $this->instance($registration);
         });
     }
@@ -65,7 +65,7 @@ 
     {
         $registrations = collect($this->register->filterByTag($tag)->all());
 
-        return $registrations->map(function ($registration) {
+        return $registrations->map(function($registration) {
             return $this->instance($registration)->details();
         });
     }
@@ -76,7 +76,7 @@ 
         //return array with group name and values
         $grouped = [];
 
-        $managers = $managers->map(function (Manager $item) {
+        $managers = $managers->map(function(Manager $item) {
             return [
                 'id'    => $item->details()->id,
                 'group' => $item->details()->plural,
@@ -90,7 +90,7 @@ 
     {
         $registrations = collect($this->register->all());
 
-        return $registrations->map(function ($registration) {
+        return $registrations->map(function($registration) {
             return $this->instance($registration);
         });
     }

src/OLD/Management/Exceptions/NotAllowedManagerAction.php

Spacing +2 added lines, -2 removed lines

@@ -40,11 +40,11 @@
 
     public static function notAllowedAction($action, string $managerKey)
     {
-        return new self('Not allowed to ' . $action . ' a model. ' . ucfirst($action) . ' route is not allowed by the ' . $managerKey . ' manager.');
+        return new self('Not allowed to '.$action.' a model. '.ucfirst($action).' route is not allowed by the '.$managerKey.' manager.');
     }
 
     public static function notAllowedPermission($permission, string $managerKey)
     {
-        return new self('Not allowed permission for ' . $permission . ' on a model as managed by the ' . $managerKey . ' manager.');
+        return new self('Not allowed permission for '.$permission.' on a model as managed by the '.$managerKey.' manager.');
     }
 }