crocodic-studio / crudbooster

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/controllers/AdminProfileController.php

Spacing +7 added lines, -7 removed lines

@@ -11,11 +11,11 @@ 
     public function getIndex() {
         $data = [];
         $data['page_title'] = 'Profile';
-        return view('crudbooster::profile',$data);
+        return view('crudbooster::profile', $data);
     }
 
     public function postUpdate() {
-        validator(request()->all(),[
+        validator(request()->all(), [
             'name'=>'required|max:255|min:3',
             'email'=>'required|email',
             'photo'=>'image',
@@ -26,19 +26,19 @@ 
             $data = [];
             $data['name'] = request('name');
             $data['email'] = request('email');
-            if(request('password')) {
+            if (request('password')) {
                 $data['password'] = Hash::make(request('password'));
             }
-            if(request()->hasFile('photo')) {
+            if (request()->hasFile('photo')) {
                 $data['photo'] = cb()->uploadFile('photo', true, 200, 200);
             }
             DB::table("users")->where("id", auth()->id())->update($data);
-        }catch (\Exception $e) {
+        } catch (\Exception $e) {
             Log::error($e);
-            return cb()->redirectBack("Something went wrong!","warning");
+            return cb()->redirectBack("Something went wrong!", "warning");
         }
 
-        return cb()->redirectBack("The profile data has been updated!","success");
+        return cb()->redirectBack("The profile data has been updated!", "success");
     }
 
 }

Braces +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
             }
 
             DB::table("users")->where("id", auth()->id())->update($data);
-        }catch (\Exception $e) {
+        } catch (\Exception $e) {
             Log::error($e);
             return cb()->redirectBack("Something went wrong!","warning");
         }

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/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
         ]);

src/types/file/FileController.php

Braces +3 added lines, -1 removed lines

@@ -14,7 +14,9 @@
 {
     public function postUploadFile()
     {
-        if(auth()->guest()) return redirect(cb()->getLoginUrl());
+        if(auth()->guest()) {
+            return redirect(cb()->getLoginUrl());
+        }
 
         $file = null;
         try {

Spacing +4 added lines, -4 removed lines

@@ -14,21 +14,21 @@
 {
     public function postUploadFile()
     {
-        if(auth()->guest()) return redirect(cb()->getLoginUrl());
+        if (auth()->guest()) return redirect(cb()->getLoginUrl());
 
         $file = null;
         try {
 
             cb()->validation([
-                'userfile' => 'required|mimes:' . implode(",",config('crudbooster.UPLOAD_FILE_EXTENSION_ALLOWED'))
+                'userfile' => 'required|mimes:'.implode(",", config('crudbooster.UPLOAD_FILE_EXTENSION_ALLOWED'))
             ]);
 
             $file = cb()->uploadFile('userfile', request("encrypt"));
 
         } catch (CBValidationException $e) {
-            return response()->json(['status'=>false,'message'=>$e->getMessage()]);
+            return response()->json(['status'=>false, 'message'=>$e->getMessage()]);
         } catch (\Exception $e) {
-            return response()->json(['status'=>false,'message'=>$e->getMessage()]);
+            return response()->json(['status'=>false, 'message'=>$e->getMessage()]);
         }
 
         return response()->json([

src/types/file/Route.php

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@
 <?php
 
-cb()->routeGroupBackend(function () {
-    cb()->routePost("upload-file",'\crocodicstudio\crudbooster\types\file\FileController@postUploadFile');
+cb()->routeGroupBackend(function() {
+    cb()->routePost("upload-file", '\crocodicstudio\crudbooster\types\file\FileController@postUploadFile');
 });
\ No newline at end of file

src/types/radio/component.blade.php

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
         <?php /** @var \crocodicstudio\crudbooster\types\radio\RadioModel $column */ ?>
         @foreach($column->getOptions() as $key=>$value)
             <?php
-                $columnValue = old($column->getName())?:($column->getDefaultValue())?:$column->getValue();
+                $columnValue = old($column->getName()) ?: ($column->getDefaultValue()) ?: $column->getValue();
             ?>
             <div class="{{ $column->getDisabled()?"disabled":"" }}">
                 <label class='radio-inline'>

src/types/wysiwyg/Hook.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     public function indexRender($row, $column)
     {
         $value = trim(strip_tags($row->{ $column->getField() }));
-        if($column->getLimit()) {
+        if ($column->getLimit()) {
             $value = Str::limit($value, $column->getLimit());
         }
         return $value;