crocodic-studio / crudbooster

src/types/select/Hook.php

Spacing +5 added lines, -5 removed lines

@@ -19,9 +19,9 @@ 
      */
     public function query($query, $column)
     {
-        if($column->getOptionsFromTable()) {
+        if ($column->getOptionsFromTable()) {
             $option = $column->getOptionsFromTable();
-            $query->join($option["table"],$option["table"].'.'.$option["key_field"],"=", $column->getName());
+            $query->join($option["table"], $option["table"].'.'.$option["key_field"], "=", $column->getName());
             $query->addSelect($option['table'].'.'.$option['display_field'].' as '.$option['table'].'_'.$option['display_field']);
         }
         return $query;
@@ -33,13 +33,13 @@ 
      */
     public function indexRender($row, $column)
     {
-        if($column->getOptionsFromTable()) {
+        if ($column->getOptionsFromTable()) {
             $option = $column->getOptionsFromTable();
             return $row->{ $option['table'].'_'.$option['display_field'] };
-        }else{
+        } else {
             $option = $column->getOptions();
             $key = $row->{ $column->getField() };
-            return @$option[ $key ];
+            return @$option[$key];
         }
     }
 

Braces +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         if($column->getOptionsFromTable()) {
             $option = $column->getOptionsFromTable();
             return $row->{ $option['table'].'_'.$option['display_field'] };
-        }else{
+        } else{
             $option = $column->getOptions();
             $key = $row->{ $column->getField() };
             return @$option[ $key ];

src/database/migrations/2016_08_07_152421_table_menus.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
      */
     public function up()
     {
-        Schema::create('cb_menus', function (Blueprint $table) {
+        Schema::create('cb_menus', function(Blueprint $table) {
             $table->increments("id");
             $table->string('name');
             $table->string("icon")->nullable();

src/types/checkbox/Checkbox.php

Spacing +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
      */
     public function optionsFromTable($table, $key_field, $display_field, $SQLCondition = null) {
         $data = DB::table($table);
-        if($SQLCondition && is_callable($SQLCondition)) {
+        if ($SQLCondition && is_callable($SQLCondition)) {
             $data = call_user_func($SQLCondition, $data);
         }elseif ($SQLCondition && is_string($SQLCondition)) {
             $data->whereRaw($SQLCondition);
@@ -42,7 +42,7 @@ 
         $data = $data->get();
         $options = [];
         foreach ($data as $d) {
-            $options[ $d->$key_field ] = $d->$display_field;
+            $options[$d->$key_field] = $d->$display_field;
         }
         $this->options($options);
     }

Braces +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         $data = DB::table($table);
         if($SQLCondition && is_callable($SQLCondition)) {
             $data = call_user_func($SQLCondition, $data);
-        }elseif ($SQLCondition && is_string($SQLCondition)) {
+        } elseif ($SQLCondition && is_string($SQLCondition)) {
             $data->whereRaw($SQLCondition);
         }
         $data = $data->get();

src/types/date/Hook.php

Spacing +2 added lines, -2 removed lines

@@ -20,9 +20,9 @@
      */
     public function indexRender($row, $column)
     {
-        if($column->getFormat()) {
+        if ($column->getFormat()) {
             return date($column->getFormat(), strtotime($row->{$column->getField()}));
-        }else{
+        } else {
             return $row->{$column->getField()};
         }
     }

Braces +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     {
         if($column->getFormat()) {
             return date($column->getFormat(), strtotime($row->{$column->getField()}));
-        }else{
+        } else{
             return $row->{$column->getField()};
         }
     }

src/helpers/UserSession.php

Spacing +3 added lines, -3 removed lines

@@ -1,7 +1,7 @@ 
 <?php 
 namespace crocodicstudio\crudbooster\helpers;
 
-class UserSession  {
+class UserSession {
 
     public function user()
     {
@@ -21,14 +21,14 @@ 
     public function photo()
     {
         $user = $this->user();
-        return ($user->photo)?asset($user->photo):asset(dummyPhoto());
+        return ($user->photo) ?asset($user->photo) : asset(dummyPhoto());
     }
 
     public function roleName()
     {
         $user = $this->user();
         $role = cb()->find("cb_roles", $user->cb_roles_id);
-        if($role) return $role->name;
+        if ($role) return $role->name;
         else return null;
     }
 

Braces +5 added lines, -2 removed lines

@@ -28,8 +28,11 @@
     {
         $user = $this->user();
         $role = cb()->find("cb_roles", $user->cb_roles_id);
-        if($role) return $role->name;
-        else return null;
+        if($role) {
+            return $role->name;
+        } else {
+            return null;
+        }
     }
 
     public function roleId()

src/helpers/SidebarMenus.php

Spacing +21 added lines, -21 removed lines

@@ -22,7 +22,7 @@ 
     private function assignToModel($menu) {
         $model = new SidebarModel();
         $model->setId($menu->id);
-        if($menu->type == "url") {
+        if ($menu->type == "url") {
             $model->setUrl($menu->path);
             $model->setIcon($menu->icon);
             $model->setName($menu->name);
@@ -43,9 +43,9 @@ 
             $model->setBasepath(config('crudbooster.ADMIN_PATH').'/'.basename($model->getUrl()));
         }
 
-        if(request()->is($model->getBasepath()."*")) {
+        if (request()->is($model->getBasepath()."*")) {
             $model->setIsActive(true);
-        }else{
+        } else {
             $model->setIsActive(false);
         }
 
@@ -54,22 +54,22 @@ 
 
     private function loadData($parent_id = null) {
         $menus = DB::table("cb_menus");
-        if($parent_id) {
-            $menus->where("parent_cb_menus_id",$parent_id);
-        }else{
+        if ($parent_id) {
+            $menus->where("parent_cb_menus_id", $parent_id);
+        } else {
             $menus->whereNull("parent_cb_menus_id");
         }
-        return $menus->orderBy("sort_number","asc")->get();
+        return $menus->orderBy("sort_number", "asc")->get();
     }
 
     private function rolePrivilege($cb_roles_id, $cb_menus_id) {
-        return cb()->find("cb_role_privileges",['cb_roles_id'=>$cb_roles_id,'cb_menus_id'=>$cb_menus_id]);
+        return cb()->find("cb_role_privileges", ['cb_roles_id'=>$cb_roles_id, 'cb_menus_id'=>$cb_menus_id]);
     }
 
-    private function checkPrivilege($roles_id,$menu) {
-        if($roles_id) {
+    private function checkPrivilege($roles_id, $menu) {
+        if ($roles_id) {
             $privilege = $this->rolePrivilege($roles_id, $menu->id);
-            if($privilege && !$privilege->can_browse) {
+            if ($privilege && !$privilege->can_browse) {
                 return false;
             }
         }
@@ -78,36 +78,36 @@ 
     }
 
     public function all($withPrivilege = true) {
-        $roles_id = ($withPrivilege)?cb()->session()->roleId():null;
+        $roles_id = ($withPrivilege) ?cb()->session()->roleId() : null;
         $menus = $this->loadData();
         $result = [];
         $menus_active = false;
-        foreach($menus as $menu) {
+        foreach ($menus as $menu) {
 
-            if($withPrivilege && !$this->checkPrivilege($roles_id, $menu)) continue;
+            if ($withPrivilege && !$this->checkPrivilege($roles_id, $menu)) continue;
 
             $sidebarModel = $this->assignToModel($menu);
-            if($sidebarModel->getisActive()) $menus_active = true;
-            if($menus2 = $this->loadData($menu->id)) {
+            if ($sidebarModel->getisActive()) $menus_active = true;
+            if ($menus2 = $this->loadData($menu->id)) {
                 $sub1 = [];
                 $menus2_active = false;
                 foreach ($menus2 as $menu2) {
 
-                    if($withPrivilege && !$this->checkPrivilege($roles_id, $menu2)) continue;
+                    if ($withPrivilege && !$this->checkPrivilege($roles_id, $menu2)) continue;
 
                     $sidebarModel2 = $this->assignToModel($menu2);
-                    if($sidebarModel2->getisActive()) $menus2_active = true;
+                    if ($sidebarModel2->getisActive()) $menus2_active = true;
 
-                    if($menus3 = $this->loadData($menu2->id)) {
+                    if ($menus3 = $this->loadData($menu2->id)) {
                         $sub2 = [];
                         $menus3_active = false;
                         foreach ($menus3 as $menu3) {
 
-                            if($withPrivilege && !$this->checkPrivilege($roles_id, $menu3)) continue;
+                            if ($withPrivilege && !$this->checkPrivilege($roles_id, $menu3)) continue;
 
                             $sidebarModel3 = $this->assignToModel($menu3);
 
-                            if($sidebarModel3->getisActive()) {
+                            if ($sidebarModel3->getisActive()) {
                                 $menus3_active = true;
                             }
 

Braces +19 added lines, -9 removed lines

@@ -27,7 +27,7 @@ 
             $model->setIcon($menu->icon);
             $model->setName($menu->name);
             $model->setBasepath(basename($model->getUrl()));
-        }elseif ($menu->type == "module") {
+        } elseif ($menu->type == "module") {
             $module = cb()->find("cb_modules", $menu->cb_modules_id);
             $className = '\App\Http\Controllers\\'.$module->controller;
             $controllerClass = new $className();
@@ -36,7 +36,7 @@ 
             $model->setIcon($module->icon);
             $model->setName($module->name);
             $model->setBasepath(config('crudbooster.ADMIN_PATH').'/'.basename($model->getUrl()));
-        }elseif ($menu->type == "path") {
+        } elseif ($menu->type == "path") {
             $model->setUrl(cb()->getAdminUrl($menu->path));
             $model->setIcon($menu->icon);
             $model->setName($menu->name);
@@ -45,7 +45,7 @@ 
 
         if(request()->is($model->getBasepath()."*")) {
             $model->setIsActive(true);
-        }else{
+        } else{
             $model->setIsActive(false);
         }
 
@@ -56,7 +56,7 @@ 
         $menus = DB::table("cb_menus");
         if($parent_id) {
             $menus->where("parent_cb_menus_id",$parent_id);
-        }else{
+        } else{
             $menus->whereNull("parent_cb_menus_id");
         }
         return $menus->orderBy("sort_number","asc")->get();
@@ -84,26 +84,36 @@ 
         $menus_active = false;
         foreach($menus as $menu) {
 
-            if($withPrivilege && !$this->checkPrivilege($roles_id, $menu)) continue;
+            if($withPrivilege && !$this->checkPrivilege($roles_id, $menu)) {
+                continue;
+            }
 
             $sidebarModel = $this->assignToModel($menu);
-            if($sidebarModel->getisActive()) $menus_active = true;
+            if($sidebarModel->getisActive()) {
+                $menus_active = true;
+            }
             if($menus2 = $this->loadData($menu->id)) {
                 $sub1 = [];
                 $menus2_active = false;
                 foreach ($menus2 as $menu2) {
 
-                    if($withPrivilege && !$this->checkPrivilege($roles_id, $menu2)) continue;
+                    if($withPrivilege && !$this->checkPrivilege($roles_id, $menu2)) {
+                        continue;
+                    }
 
                     $sidebarModel2 = $this->assignToModel($menu2);
-                    if($sidebarModel2->getisActive()) $menus2_active = true;
+                    if($sidebarModel2->getisActive()) {
+                        $menus2_active = true;
+                    }
 
                     if($menus3 = $this->loadData($menu2->id)) {
                         $sub2 = [];
                         $menus3_active = false;
                         foreach ($menus3 as $menu3) {
 
-                            if($withPrivilege && !$this->checkPrivilege($roles_id, $menu3)) continue;
+                            if($withPrivilege && !$this->checkPrivilege($roles_id, $menu3)) {
+                                continue;
+                            }
 
                             $sidebarModel3 = $this->assignToModel($menu3);
 

src/controllers/traits/ControllerSetting.php

Spacing +8 added lines, -8 removed lines

@@ -29,9 +29,9 @@ 
         $this->setButtonDetail(true);
         $this->setButtonSave(true);
         $this->setButtonLimitPage(true);
-        $this->hideButtonDeleteWhen(function ($row) { return false; });
-        $this->hideButtonDetailWhen(function ($row) { return false; });
-        $this->hideButtonEditWhen(function ($row) { return false; });
+        $this->hideButtonDeleteWhen(function($row) { return false; });
+        $this->hideButtonDetailWhen(function($row) { return false; });
+        $this->hideButtonEditWhen(function($row) { return false; });
     }
 
 
@@ -128,9 +128,9 @@ 
      * @param ButtonColor $color
      * @param string $attributes
      */
-    public function addIndexActionButton($label, $actionURL, $fontAwesome_icon=null, $color=null, $attributes = null)
+    public function addIndexActionButton($label, $actionURL, $fontAwesome_icon = null, $color = null, $attributes = null)
     {
-        $color = ($color)?:ButtonColor::DARK_BLUE;
+        $color = ($color) ?: ButtonColor::DARK_BLUE;
 
         $model = new IndexActionButtonModel();
         $model->setLabel($label);
@@ -185,12 +185,12 @@ 
      * @param $fontAwesome_icon
      * @param ButtonColor|string $color
      */
-    public function addActionButton($label, $url_target, $condition_callback=null, $fontAwesome_icon=null, $color=null, $confirmation = false)
+    public function addActionButton($label, $url_target, $condition_callback = null, $fontAwesome_icon = null, $color = null, $confirmation = false)
     {
         $new = new AddActionButtonModel();
         $new->setLabel($label);
-        $new->setIcon($fontAwesome_icon?:"fa fa-bars");
-        $new->setColor($color?:"primary");
+        $new->setIcon($fontAwesome_icon ?: "fa fa-bars");
+        $new->setColor($color ?: "primary");
         $new->setUrl($url_target);
         $new->setCondition($condition_callback);
         $new->setConfirmation($confirmation);

src/types/image/Hook.php

Spacing +2 added lines, -2 removed lines

@@ -28,7 +28,7 @@ 
     public function detailRender($row, $column)
     {
         $column->setValue($row->{ $column->getField() });
-        return view("types::image.detail",[
+        return view("types::image.detail", [
             'row'=>$row,
             'column'=>$column
         ]);
@@ -37,7 +37,7 @@ 
     public function indexRender($row, $column)
     {
         $column->setValue($row->{ $column->getField() });
-        return view("types::image.index",[
+        return view("types::image.index", [
             'row'=>$row,
             'column'=>$column
         ]);

src/types/file/Hook.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     public function detailRender($row, $column)
     {
         $column->setValue($row->{ $column->getField() });
-        return view("types::file.detail",[
+        return view("types::file.detail", [
             'row'=>$row,
             'column'=>$column
         ]);