Cherry-Pie / Jarboe

src/Jarboe/Http/Controllers/AbstractTableController.php

Doc Comments +4 added lines, -4 removed lines

@@ -104,9 +104,6 @@ 
      * Handle relation search action.
      * Currently used for SelectField with type `select2` and `ajax = true`.
      *
-     * @param string $field
-     * @param string $page
-     * @param string $term
      * @return \Illuminate\Http\JsonResponse
      */
     public function searchRelation(Request $request)
@@ -712,6 +709,9 @@ 
         }
     }
 
+    /**
+     * @param string $title
+     */
     protected function addTab($title, array $fields)
     {
         foreach ($fields as $field) {
@@ -807,7 +807,7 @@ 
     /**
      * Check if user has permission for the action.
      *
-     * @param $action
+     * @param string $action
      * @return bool
      */
     protected function can($action): bool

Spacing +7 added lines, -7 removed lines

@@ -119,7 +119,7 @@ 
 
         $query = $request->get('term');
         $fieldName = $request->get('field');
-        $page = (int) $request->get('page');
+        $page = (int)$request->get('page');
 
         /** @var Select $field */
         $field = $this->crud()->getFieldByName($fieldName);
@@ -134,7 +134,7 @@ 
                 $options = $field->getOptions($page, $perPage, $query, $total, $index);
                 array_walk($options, function(&$item, $key) use($group) {
                     $item = [
-                        'id'   => crc32($group['group']) .'~~~'. $key,
+                        'id'   => crc32($group['group']).'~~~'.$key,
                         'text' => $item,
                     ];
                 });
@@ -282,7 +282,7 @@ 
             $formRequest = new $formRequestClass();
             if (method_exists($formRequest, 'rules')) {
                 foreach ($formRequest->rules() as $param => $paramRules) {
-                    if (preg_match('~^'. preg_quote($name) .'(\.\*)?$~', $param)) {
+                    if (preg_match('~^'.preg_quote($name).'(\.\*)?$~', $param)) {
                         return [
                             $paramRules,
                             $formRequest->messages(),
@@ -380,7 +380,7 @@ 
         $this->init();
         $this->bound();
 
-        $this->crud()->setPerPageParam((int) $perPage);
+        $this->crud()->setPerPageParam((int)$perPage);
 
         return redirect($this->crud()->listUrl());
     }
@@ -882,11 +882,11 @@ 
                     return $this->handleInline($request);
 
                 default:
-                    throw new \RuntimeException('Invalid method ' . $name);
+                    throw new \RuntimeException('Invalid method '.$name);
             }
         } catch (ValidationException $e) {
             throw $e;
-        } catch(UnauthorizedException $e) {
+        } catch (UnauthorizedException $e) {
             return $this->createUnauthorizedResponse($request, $e);
         } catch (\Exception $e) {
             return redirect()
@@ -962,7 +962,7 @@ 
      */
     protected function notify(string $title, string $content = null, int $timeout = 4000, string $color = null, string $icon = null, string $type = 'small')
     {
-        $ident = 'jarboe_notifications.'. $type;
+        $ident = 'jarboe_notifications.'.$type;
 
         $messages = session()->get($ident, []);
         $messages[] = [

src/Jarboe/Table/CRUD.php

Doc Comments +9 added lines

@@ -273,6 +273,9 @@ 
         return sprintf('%s%s%s/force-delete', $this->baseUrl(), self::BASE_URL_DELIMITER, $id);
     }
 
+    /**
+     * @param string $identifier
+     */
     public function toolbarUrl($identifier)
     {
         return sprintf('%s%stoolbar/%s', $this->baseUrl(), self::BASE_URL_DELIMITER, $identifier);
@@ -325,6 +328,9 @@ 
         return rtrim($chunks[0], '/');
     }
 
+    /**
+     * @param integer $ident
+     */
     public function tableIdentifier($ident = null)
     {
         if (is_null($ident)) {
@@ -369,6 +375,9 @@ 
         $this->preferences()->saveCurrentLocale($this->tableIdentifier(), $locale);
     }
 
+    /**
+     * @return \Illuminate\Http\Request
+     */
     public function getTool($ident)
     {
         if (array_key_exists($ident, $this->toolbar)) {

Spacing +1 added lines, -1 removed lines

@@ -493,7 +493,7 @@
 
     public function isSortableByWeight()
     {
-        return (bool) $this->getSortableWeightFieldName();
+        return (bool)$this->getSortableWeightFieldName();
     }
 
     public function isSortableByWeightActive()

src/Jarboe/Table/Fields/AbstractField.php

Doc Comments +3 added lines

@@ -40,6 +40,9 @@
         return $field;
     }
 
+    /**
+     * @param string $model
+     */
     public function setModel($model)
     {
         $this->model = $model;

Spacing +2 added lines, -2 removed lines

@@ -166,7 +166,7 @@ 
     {
         $name = $this->name();
         if ($locale) {
-            $name .= '.'. $locale;
+            $name .= '.'.$locale;
         }
 
         if ($this->hasOld($name)) {
@@ -180,7 +180,7 @@ 
     {
         $name = $this->name();
         if ($locale) {
-            $name .= '.'. $locale;
+            $name .= '.'.$locale;
         }
 
         if ($this->hasOld($name)) {

src/Jarboe/Table/Fields/Traits/Storage.php

Doc Comments +3 added lines

@@ -52,6 +52,9 @@
         return $this;
     }
 
+    /**
+     * @return string
+     */
     public function getPath()
     {
         return $this->path;

src/Jarboe/Table/Toolbar/MassDeleteTool.php

Braces +17 added lines, -17 removed lines

@@ -5,18 +5,18 @@ 
 use Illuminate\Http\Request;

 use Yaro\Jarboe\Table\CRUD;

 

-class MassDeleteTool extends AbstractTool

-{

+class MassDeleteTool extends AbstractTool
+{
     /**

      * Set CRUD object.

      *

      * @param CRUD $crud

      */

-    public function setCrud(CRUD $crud)

-    {

+    public function setCrud(CRUD $crud)
+    {
         parent::setCrud($crud);

 

-        if ($this->check()) {

+        if ($this->check()) {
             $this->crud()->enableBatchCheckboxes(true);

         }

     }

@@ -24,8 +24,8 @@ 
     /**

      * Position where should tool be placed.

      */

-    public function position()

-    {

+    public function position()
+    {
         return self::POSITION_BODY_TOP;

     }

 

@@ -40,8 +40,8 @@ 
     /**

      * Tool's view.

      */

-    public function render()

-    {

+    public function render()
+    {
         return view('jarboe::crud.toolbar.mass_delete', [

             'tool' => $this,

             'crud' => $this->crud(),

@@ -55,24 +55,24 @@ 
      *

      * @return \Illuminate\Http\JsonResponse

      */

-    public function handle(Request $request)

-    {

+    public function handle(Request $request)
+    {
         $errors = [];

         $hidden = [];

         $removed = [];

-        foreach ($request->get('ids') as $id) {

-            try {

-                if (!$this->crud()->repo()->delete($id)) {

+        foreach ($request->get('ids') as $id) {
+            try {
+                if (!$this->crud()->repo()->delete($id)) {
                     throw new \Exception(__('jarboe::toolbar.mass_delete.delete_event_failed'));

                 }

 

-                try {

+                try {
                     $this->crud()->repo()->find($id);

                     $hidden[] = $id;

-                } catch (\Exception $e) {

+                } catch (\Exception $e) {
                     $removed[] = $id;

                 }

-            } catch (\Exception $e) {

+            } catch (\Exception $e) {
                 $errors[$id] = $e->getMessage();

             }

         }

src/Jarboe/Table/Filters/TextFilter.php

Spacing +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
                 $query->where(
                     $this->field()->getRelationTitleField(),
                     $sign,
-                    $leftPart . $value . $rightPart
+                    $leftPart.$value.$rightPart
                 );
             });
             return;
@@ -51,7 +51,7 @@ 
         $query->where(
             $this->field->name(),
             $sign,
-            $leftPart . $value . $rightPart
+            $leftPart.$value.$rightPart
         );
     }
 }

src/Jarboe/Table/Filters/DateFilter.php

Spacing +3 added lines, -3 removed lines

@@ -25,7 +25,7 @@ 
             $view .= '_range';
         }
 
-        return view('jarboe::crud.filters.'. $view, [
+        return view('jarboe::crud.filters.'.$view, [
             'filter' => $this,
         ])->render();
     }
@@ -38,10 +38,10 @@ 
         }
 
         if ($this->isRange()) {
-            $model->when($value['from'] ?? null, function ($query, $value) {
+            $model->when($value['from'] ?? null, function($query, $value) {
                 return $query->whereDate($this->field()->name(), '>=', $value);
             });
-            $model->when($value['to'] ?? null, function ($query, $value) {
+            $model->when($value['to'] ?? null, function($query, $value) {
                 return $query->whereDate($this->field()->name(), '<=', $value);
             });
             return;

src/Jarboe/Table/Filters/NumberFilter.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         $model->where(
             $this->field()->name(),
             $sign,
-            $leftPart . $this->value() . $rightPart
+            $leftPart.$this->value().$rightPart
         );
     }
 }
\ No newline at end of file

src/Jarboe/Table/Filters/TextareaFilter.php

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
                 $query->where(
                     $this->field()->getRelationTitleField(),
                     $sign,
-                    $leftPart . $value . $rightPart
+                    $leftPart.$value.$rightPart
                 );
             });
             return;
@@ -52,7 +52,7 @@ 
         $query->where(
             $this->field()->name(),
             $sign,
-            $leftPart . $value . $rightPart
+            $leftPart.$value.$rightPart
         );
     }
 }
\ No newline at end of file