thinktomorrow / chief

src/Admin/Widgets/Widgets.php

Spacing +2 added lines, -2 removed lines

@@ -11,13 +11,13 @@
 
     private function __construct(array $widgets)
     {
-        array_map(function(Widget $widget){}, $widgets);
+        array_map(function(Widget $widget) {}, $widgets);
         $this->widgets = $widgets;
     }
 
     public static function fromArray(array $widgetClasses): self
     {
-        return new static(array_map(function(string $widgetClass){
+        return new static(array_map(function(string $widgetClass) {
             return app($widgetClass);
         }, $widgetClasses));
     }

src/Admin/Authorization/Permission.php

Spacing +3 added lines, -3 removed lines

@@ -26,15 +26,15 @@
     {
         $abilities = ['view', 'create', 'update', 'delete'];
 
-        return array_map(function ($val) use ($scope) {
-            return $val . '-' . $scope;
+        return array_map(function($val) use ($scope) {
+            return $val.'-'.$scope;
         }, $abilities);
     }
 
     public static function getPermissionsForIndex()
     {
         $permissions = $temp = [];
-        self::all()->each(function ($permission) use (&$permissions, &$temp) {
+        self::all()->each(function($permission) use (&$permissions, &$temp) {
             $model = explode("_", $permission->name, 2)[1];
             $temp[$model][$permission->id] = explode("_", $permission->name, 2)[0];
             $permissions = $temp;

src/Admin/Authorization/Console/GenerateRoleCommand.php

Spacing +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
             // Generate all permissions if only scope is passed
             if (false === strpos($permissionName, '-')) {
                 $cleanPermissionNames = array_merge($cleanPermissionNames, Permission::generate($permissionName));
-            } else {
+            }else {
                 // Trim the value
                 $cleanPermissionNames[] = $permissionName;
             }
@@ -55,6 +55,6 @@ 
             $role->givePermissionTo($cleanPermissionName);
         }
 
-        $this->info('Role ' . $role->name . ' was assigned the permissions: ' . implode(',', $cleanPermissionNames));
+        $this->info('Role '.$role->name.' was assigned the permissions: '.implode(',', $cleanPermissionNames));
     }
 }

src/Admin/Authorization/Console/GeneratePermissionCommand.php

Spacing +4 added lines, -4 removed lines

@@ -39,7 +39,7 @@ 
             Permission::findOrCreate($permission, 'chief');
         }
 
-        $this->info('Permissions ' . implode(', ', $permissions) . ' created.');
+        $this->info('Permissions '.implode(', ', $permissions).' created.');
     }
 
     /**
@@ -56,9 +56,9 @@ 
         foreach ($roleNames as $roleName) {
             if ($role = Role::where('name', trim($roleName))->first()) {
                 $role->syncPermissions($permissions);
-                $this->info('Role ' . $roleName . ' assigned the given permissions.');
-            } else {
-                $this->warn('Role not found by name ' . $roleName . '!');
+                $this->info('Role '.$roleName.' assigned the given permissions.');
+            }else {
+                $this->warn('Role not found by name '.$roleName.'!');
             }
         }
     }

src/Admin/Authorization/Role.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
 
     public static function rolesForSelect($includeDeveloperRole = false)
     {
-        $roles = $includeDeveloperRole ? static::all() : static::all()->reject(function ($role) {
+        $roles = $includeDeveloperRole ? static::all() : static::all()->reject(function($role) {
             return $role->name == 'developer';
         });
 
@@ -39,7 +39,7 @@ 
 
     public function getPermissionsForIndex()
     {
-        $this->permissions->each(function ($permission) {
+        $this->permissions->each(function($permission) {
             $model = explode("_", $permission->name, 2)[1];
             $temp = $this->permission;
             $temp[$model][] = explode("_", $permission->name, 2)[0];

src/Admin/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/Admin/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/Admin/Mediagallery/Http/MediagalleryController.php

Spacing +4 added lines, -4 removed lines

@@ -21,13 +21,13 @@ 
         $assets = Asset::with('media')->orderBy('created_at', 'DESC')->select('assets.*');
 
         if ($search) {
-            $assets->whereHas('media', function (Builder $query) use ($search) {
-                $query->where('file_name', 'LIKE', '%' . $search . '%');
+            $assets->whereHas('media', function(Builder $query) use ($search) {
+                $query->where('file_name', 'LIKE', '%'.$search.'%');
             });
         }
 
         if ($unused) {
-            $assets->whereNotExists(function ($query) {
+            $assets->whereNotExists(function($query) {
                 $query->select(DB::raw(1))
                     ->from('asset_pivots')
                     ->whereRaw('asset_pivots.asset_id = assets.id')
@@ -35,7 +35,7 @@ 
             });
         }
 
-        if($owner){
+        if ($owner) {
             $owner = ModelReference::fromString($owner)->instance();
             $modelAssets = collect();
             $modelAssets = $modelAssets->merge($owner->assets());

src/Admin/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');
         }