thinktomorrow / chief

src/Sets/Set.php

Spacing +4 added lines, -4 removed lines

@@ -51,7 +51,7 @@ 
 
         // If no view has been created for this page collection, we try once again to fetch the content value if any. This will silently fail
         // if no content value is present. We don't consider the 'content' attribute to be a default as we do for module.
-        return $this->map(function ($item) {
+        return $this->map(function($item) {
             return ($this->viewParent && $item instanceof ViewableContract)
                 ? $item->setViewParent($this->viewParent)->renderView()
                 : ($item->content ?? '');
@@ -84,7 +84,7 @@ 
     {
         $currentPage = $currentPage ?? request()->get('page', 1);
         $path = request()->path();
-        $items = array_slice($this->all(), ($currentPage - 1) * $perPage);
+        $items = array_slice($this->all(), ($currentPage-1) * $perPage);
 
         return (new \Illuminate\Pagination\Paginator($items, $perPage ?? $this->paginateSetting('perPage', '12'), $currentPage))->setPath($path);
     }
@@ -101,14 +101,14 @@ 
     {
         $currentPage = $currentPage ?? request()->get('page', 1);
         $path = '/'.request()->path();
-        $items = array_slice($this->all(), ($currentPage - 1) * $perPage, $perPage);
+        $items = array_slice($this->all(), ($currentPage-1) * $perPage, $perPage);
 
         return (new \Illuminate\Pagination\LengthAwarePaginator($items, $this->paginateSetting('total', $this->count()), $perPage ?? $this->paginateSetting('perPage', '12'), $currentPage))->setPath($path);
     }
 
     private function paginateSetting($key, $default = null)
     {
-        if(!isset($this->settings['paginate']) || !isset($this->settings['paginate'][$key])) return $default;
+        if (!isset($this->settings['paginate']) || !isset($this->settings['paginate'][$key])) return $default;
 
         return $this->settings['paginate'][$key];
     }

Braces +3 added lines, -1 removed lines

@@ -108,7 +108,9 @@
 
     private function paginateSetting($key, $default = null)
     {
-        if(!isset($this->settings['paginate']) || !isset($this->settings['paginate'][$key])) return $default;
+        if(!isset($this->settings['paginate']) || !isset($this->settings['paginate'][$key])) {
+            return $default;
+        }
 
         return $this->settings['paginate'][$key];
     }

src/Sets/SetReference.php

Spacing +12 added lines, -12 removed lines

@@ -39,7 +39,7 @@ 
     {
         // Constraints
         if (!isset($values['action'])) {
-            throw new \InvalidArgumentException('Set reference array is missing required values for the "action" keys. Given: ' . print_r($values, true));
+            throw new \InvalidArgumentException('Set reference array is missing required values for the "action" keys. Given: '.print_r($values, true));
         }
 
         return new static(
@@ -54,14 +54,14 @@ 
     {
         $sets = config('thinktomorrow.chief.sets', []);
 
-        return collect($sets)->map(function ($set, $key) {
+        return collect($sets)->map(function($set, $key) {
             return SetReference::fromArray($key, $set);
         });
     }
 
     public static function find($key): ?SetReference
     {
-        return static::all()->filter(function ($ref) use ($key) {
+        return static::all()->filter(function($ref) use ($key) {
             return $ref->key() == $key;
         })->first();
     }
@@ -78,13 +78,13 @@ 
 
         $this->validateAction($class, $method);
 
-        $result = call_user_func_array([app($class),$method], $this->parameters($class, $method, $parent));
+        $result = call_user_func_array([app($class), $method], $this->parameters($class, $method, $parent));
 
-        if (! $result instanceof Set && $result instanceof Collection) {
+        if (!$result instanceof Set && $result instanceof Collection) {
             return new Set($result->all(), $this->key);
         }
 
-        if (! $result instanceof Set && $result instanceof Paginator) {
+        if (!$result instanceof Set && $result instanceof Paginator) {
             return new Set($result->all(), $this->key, [
                 'paginate' => [
                     'total' => $result->total(),
@@ -104,22 +104,22 @@ 
      */
     protected function parameters(string $class, string $method, ActsAsParent $parent): array
     {
-        try{
+        try {
             $parameters = $this->parameters;
 
             $reflection = new \ReflectionClass($class);
-            foreach($reflection->getMethod($method)->getParameters() as $parameter){
-                if($parameter->getType() && $parameter->getType()->getName() == ActsAsParent::class){
+            foreach ($reflection->getMethod($method)->getParameters() as $parameter) {
+                if ($parameter->getType() && $parameter->getType()->getName() == ActsAsParent::class) {
                     $parameters[] = $parent;
                 }
-                if($parameter->getType() && $parameter->getType()->getName() == Request::class){
+                if ($parameter->getType() && $parameter->getType()->getName() == Request::class) {
                     $parameters[] = request();
                 }
             }
 
             return $parameters;
         }
-        catch(\Exception $e) {
+        catch (\Exception $e) {
             if (config('thinktomorrow.chief.strict')) {
                 throw $e;
             }
@@ -153,7 +153,7 @@ 
 
     private static function validateAction($class, $method)
     {
-        if (! class_exists($class)) {
+        if (!class_exists($class)) {
             throw new \InvalidArgumentException('The class ['.$class.'] isn\'t a valid class reference or does not exist in the chief-settings.sets config entry.');
         }