Laravel-Backpack / CRUD

src/app/Http/Controllers/CrudController.php

Unused Use Statements -3 removed lines

@@ -4,12 +4,9 @@
 use Illuminate\Routing\Controller as BaseController;
 use Illuminate\Foundation\Validation\ValidatesRequests;
 use Illuminate\Http\Request;
-use Crypt;
-use Illuminate\Support\Facades\Form as Form;
 use Alert;
 
 // VALIDATION: change the requests to match your own file names if you need form validation
-use Backpack\CRUD\app\Http\Requests\CrudRequest as StoreRequest;
 use Backpack\CRUD\app\Http\Requests\CrudRequest as UpdateRequest;
 
 class CrudController extends BaseController {

Spacing +9 added lines, -9 removed lines

@@ -128,7 +128,7 @@ 
 		// if it's a relationship with a pivot table, also sync that
 		$this->prepareFields();
 		foreach ($this->crud['fields'] as $k => $field) {
-			if (isset($field['pivot']) && $field['pivot']==true && \Request::has($field['name']))
+			if (isset($field['pivot']) && $field['pivot'] == true && \Request::has($field['name']))
 			{
 				$model::find($item->id)->$field['name']()->attach(\Request::input($field['name']));
 			}
@@ -205,7 +205,7 @@ 
 
 		// if it's a relationship with a pivot table, also sync that
 		foreach ($this->crud['fields'] as $k => $field) {
-			if (isset($field['pivot']) && $field['pivot']==true && \Request::has($field['name']))
+			if (isset($field['pivot']) && $field['pivot'] == true && \Request::has($field['name']))
 			{
 				$model::find(\Request::input('id'))->$field['name']()->sync(\Request::input($field['name']));
 			}
@@ -365,7 +365,7 @@ 
 			$this->data['languages'] = \Backpack\LangFileManager\app\Models\Language::all();
 			$this->data['languages_already_translated_in'] = $this->data['entry']->translationLanguages();
 			$this->data['languages_to_translate_in'] = $this->data['languages']->diff($this->data['languages_already_translated_in']);
-			$this->data['languages_to_translate_in'] = $this->data['languages_to_translate_in']->reject(function ($item) {
+			$this->data['languages_to_translate_in'] = $this->data['languages_to_translate_in']->reject(function($item) {
 			    return $item->abbr == \Lang::locale();
 			});
 		}
@@ -428,13 +428,13 @@ 
 		// go through each defined field
 		foreach ($this->crud['fields'] as $k => $field) {
 			// if it's a fake field
-			if (isset($this->crud['fields'][$k]['fake']) && $this->crud['fields'][$k]['fake']==true) {
+			if (isset($this->crud['fields'][$k]['fake']) && $this->crud['fields'][$k]['fake'] == true) {
 				// add it to the request in its appropriate variable - the one defined, if defined
 				if (isset($this->crud['fields'][$k]['store_in'])) {
 					$request[$this->crud['fields'][$k]['store_in']][$this->crud['fields'][$k]['name']] = $request[$this->crud['fields'][$k]['name']];
 
 					$remove_fake_field = array_pull($request, $this->crud['fields'][$k]['name']);
-					if(!in_array($this->crud['fields'][$k]['store_in'], $fake_field_columns_to_encode, true)){
+					if (!in_array($this->crud['fields'][$k]['store_in'], $fake_field_columns_to_encode, true)) {
 				        array_push($fake_field_columns_to_encode, $this->crud['fields'][$k]['store_in']);
 				    }
 				}
@@ -443,7 +443,7 @@ 
 					$request['extras'][$this->crud['fields'][$k]['name']] = $request[$this->crud['fields'][$k]['name']];
 
 					$remove_fake_field = array_pull($request, $this->crud['fields'][$k]['name']);
-					if(!in_array('extras', $fake_field_columns_to_encode, true)){
+					if (!in_array('extras', $fake_field_columns_to_encode, true)) {
 				        array_push($fake_field_columns_to_encode, 'extras');
 				    }
 				}
@@ -476,16 +476,16 @@ 
 
 		foreach ($this->crud['fields'] as $k => $field) {
 			// if it's a fake field
-			if (isset($this->crud['fields'][$k]['fake']) && $this->crud['fields'][$k]['fake']==true) {
+			if (isset($this->crud['fields'][$k]['fake']) && $this->crud['fields'][$k]['fake'] == true) {
 				// add it to the request in its appropriate variable - the one defined, if defined
 				if (isset($this->crud['fields'][$k]['store_in'])) {
-					if(!in_array($this->crud['fields'][$k]['store_in'], $fake_field_columns_to_encode, true)){
+					if (!in_array($this->crud['fields'][$k]['store_in'], $fake_field_columns_to_encode, true)) {
 				        array_push($fake_field_columns_to_encode, $this->crud['fields'][$k]['store_in']);
 				    }
 				}
 				else //otherwise in the one defined in the $crud variable
 				{
-					if(!in_array('extras', $fake_field_columns_to_encode, true)){
+					if (!in_array('extras', $fake_field_columns_to_encode, true)) {
 				        array_push($fake_field_columns_to_encode, 'extras');
 				    }
 				}

src/CrudServiceProvider.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace Backpack\CRUD;
 
 use Illuminate\Support\ServiceProvider;
-use Illuminate\Routing\Router;
 use Route;
 
 class CrudServiceProvider extends ServiceProvider

Spacing +4 added lines, -4 removed lines

@@ -32,11 +32,11 @@ 
 
         // PUBLISH FILES
         // publish lang files
-        $this->publishes([ __DIR__.'/resources/lang' => resource_path('lang/vendor/backpack'), ], 'lang');
+        $this->publishes([__DIR__.'/resources/lang' => resource_path('lang/vendor/backpack'), ], 'lang');
         // publish views
-        $this->publishes([ __DIR__.'/resources/views' => resource_path('views/vendor/backpack/crud'), ], 'views');
+        $this->publishes([__DIR__.'/resources/views' => resource_path('views/vendor/backpack/crud'), ], 'views');
         // publish public Backpack CRUD assets
-        $this->publishes([ __DIR__.'/public' => public_path('vendor/backpack'), ], 'public');
+        $this->publishes([__DIR__.'/public' => public_path('vendor/backpack'), ], 'public');
         // publish custom files for elFinder
         $this->publishes([
                             __DIR__.'/config/elfinder.php' => config_path('elfinder.php'),
@@ -73,7 +73,7 @@ 
 
     private function registerCRUD()
     {
-        $this->app->bind('CRUD',function($app){
+        $this->app->bind('CRUD', function($app) {
             return new CRUD($app);
         });
     }

src/CrudTrait.php

Unused Use Statements -1 removed lines

@@ -1,6 +1,5 @@
 <?php namespace Backpack\CRUD;
 
-use Illuminate\Database\Eloquent\Model;
 use DB;
 use Lang;
 

Spacing +7 added lines, -7 removed lines

@@ -12,14 +12,14 @@ 
     |--------------------------------------------------------------------------
     */
 
-	public static function getPossibleEnumValues($field_name){
+	public static function getPossibleEnumValues($field_name) {
         $instance = new static; // create an instance of the model to be able to get the table name
-        $type = DB::select( DB::raw('SHOW COLUMNS FROM '.$instance->getTable().' WHERE Field = "'.$field_name.'"') )[0]->Type;
+        $type = DB::select(DB::raw('SHOW COLUMNS FROM '.$instance->getTable().' WHERE Field = "'.$field_name.'"'))[0]->Type;
         preg_match('/^enum\((.*)\)$/', $type, $matches);
         $enum = array();
         $exploded = explode(',', $matches[1]);
-        foreach($exploded as $value){
-            $v = trim( $value, "'" );
+        foreach ($exploded as $value) {
+            $v = trim($value, "'");
             $enum[] = $v;
         }
         return $enum;
@@ -27,9 +27,9 @@ 
 
     public static function isColumnNullable($column_name) {
         $instance = new static; // create an instance of the model to be able to get the table name
-        $answer = DB::select( DB::raw("SELECT IS_NULLABLE FROM INFORMATION_SCHEMA.COLUMNS WHERE TABLE_NAME='".$instance->getTable()."' AND COLUMN_NAME='".$column_name."' AND table_schema='".env('DB_DATABASE')."'") )[0];
+        $answer = DB::select(DB::raw("SELECT IS_NULLABLE FROM INFORMATION_SCHEMA.COLUMNS WHERE TABLE_NAME='".$instance->getTable()."' AND COLUMN_NAME='".$column_name."' AND table_schema='".env('DB_DATABASE')."'"))[0];
 
-        return ($answer->IS_NULLABLE == 'YES'?true:false);
+        return ($answer->IS_NULLABLE == 'YES' ? true : false);
     }
 
 
@@ -129,7 +129,7 @@ 
 
     public function translation($translation_lang = false)
     {
-        if ($translation_lang==false) {
+        if ($translation_lang == false) {
             $translation_lang = Lang::locale();
         }
 

src/resources/views-elfinder/filepicker.php

Spacing +5 added lines, -5 removed lines

@@ -10,28 +10,28 @@
     <script src="//ajax.googleapis.com/ajax/libs/jqueryui/1.10.4/jquery-ui.min.js"></script>
 
     <!-- elFinder CSS (REQUIRED) -->
-    <link rel="stylesheet" type="text/css" href="<?= asset($dir . '/css/elfinder.min.css') ?>">
+    <link rel="stylesheet" type="text/css" href="<?= asset($dir.'/css/elfinder.min.css') ?>">
     <!-- <link rel="stylesheet" type="text/css" href="<?= asset($dir.'/css/theme.css') ?>"> -->
     <link rel="stylesheet" type="text/css" href="<?= asset('vendor/backpack/elfinder/elfinder.backpack.theme.css') ?>">
 
     <!-- elFinder JS (REQUIRED) -->
-    <script src="<?= asset($dir . '/js/elfinder.min.js') ?>"></script>
+    <script src="<?= asset($dir.'/js/elfinder.min.js') ?>"></script>
 
     <?php if ($locale) { ?>
         <!-- elFinder translation (OPTIONAL) -->
-        <script src="<?= asset($dir . "/js/i18n/elfinder.$locale.js") ?>"></script>
+        <script src="<?= asset($dir."/js/i18n/elfinder.$locale.js") ?>"></script>
     <?php } ?>
     <!-- Include jQuery, jQuery UI, elFinder (REQUIRED) -->
 
     <?php
-    $mimeTypes = implode(',',array_map(function($t){return "'".$t."'";}, explode(',',$type)));
+    $mimeTypes = implode(',', array_map(function($t) {return "'".$t."'"; }, explode(',', $type)));
     ?>
 
     <script type="text/javascript">
         $().ready(function () {
             var elf = $('#elfinder').elfinder({
                 // set your elFinder options here
-                <?php if($locale){ ?>
+                <?php if ($locale) { ?>
                     lang: '<?= $locale ?>', // locale
                 <?php } ?>
                 customData: {

src/resources/views-elfinder/tinymce4.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
     <!-- elFinder JS (REQUIRED) -->
     <script src="<?= asset($dir.'/js/elfinder.min.js') ?>"></script>
 
-    <?php if($locale){ ?>
+    <?php if ($locale) { ?>
         <!-- elFinder translation (OPTIONAL) -->
         <script src="<?= asset($dir."/js/i18n/elfinder.$locale.js") ?>"></script>
     <?php } ?>
@@ -40,7 +40,7 @@ 
         $().ready(function() {
             var elf = $('#elfinder').elfinder({
                 // set your elFinder options here
-                <?php if($locale){ ?>
+                <?php if ($locale) { ?>
                     lang: '<?= $locale ?>', // locale
                 <?php } ?>
                 customData: {

src/resources/views/fields/page_or_link.blade.php

Spacing +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
     </div>
     <div class="col-sm-9">
         <!-- external link input -->
-          <div class="page_or_link_value <?php if (!isset($entry) || $entry->type!='external_link') { echo 'hidden'; } ?>" id="page_or_link_external_link">
+          <div class="page_or_link_value <?php if (!isset($entry) || $entry->type != 'external_link') { echo 'hidden'; } ?>" id="page_or_link_external_link">
             <input
                 type="url"
                 class="form-control"
@@ -58,7 +58,7 @@ 
                 >
           </div>
           <!-- internal link input -->
-          <div class="page_or_link_value <?php if (!isset($entry) || $entry->type!='internal_link') { echo 'hidden'; } ?>" id="page_or_link_internal_link">
+          <div class="page_or_link_value <?php if (!isset($entry) || $entry->type != 'internal_link') { echo 'hidden'; } ?>" id="page_or_link_internal_link">
             <input
                 type="text"
                 class="form-control"
@@ -75,7 +75,7 @@ 
                 >
           </div>
           <!-- page slug input -->
-          <div class="page_or_link_value <?php if (isset($entry) && $entry->type!='page_link') { echo 'hidden'; } ?>" id="page_or_link_page">
+          <div class="page_or_link_value <?php if (isset($entry) && $entry->type != 'page_link') { echo 'hidden'; } ?>" id="page_or_link_page">
             <select
                 class="form-control"
                 name="page_id"

src/resources/views-elfinder/elfinder.blade.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
     <!-- elFinder JS (REQUIRED) -->
     <script src="<?= asset($dir.'/js/elfinder.min.js') ?>"></script>
 
-    <?php if($locale){ ?>
+    <?php if ($locale) { ?>
         <!-- elFinder translation (OPTIONAL) -->
         <script src="<?= asset($dir."/js/i18n/elfinder.$locale.js") ?>"></script>
     <?php } ?>
@@ -40,7 +40,7 @@ 
         $().ready(function() {
             var elf = $('#elfinder').elfinder({
                 // set your elFinder options here
-                <?php if($locale){ ?>
+                <?php if ($locale) { ?>
                     lang: '<?= $locale ?>', // locale
                 <?php } ?>
                 customData: {

src/resources/views-elfinder/tinymce.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
     <!-- elFinder JS (REQUIRED) -->
     <script src="<?= asset($dir.'/js/elfinder.min.js') ?>"></script>
 
-    <?php if($locale){ ?>
+    <?php if ($locale) { ?>
         <!-- elFinder translation (OPTIONAL) -->
         <script src="<?= asset($dir."/js/i18n/elfinder.$locale.js") ?>"></script>
     <?php } ?>
@@ -40,7 +40,7 @@ 
         $().ready(function() {
             var elf = $('#elfinder').elfinder({
                 // set your elFinder options here
-                <?php if($locale){ ?>
+                <?php if ($locale) { ?>
                     lang: '<?= $locale ?>', // locale
                 <?php } ?>
                 customData: {

src/resources/views-elfinder/ckeditor4.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
     <!-- elFinder JS (REQUIRED) -->
     <script src="<?= asset($dir.'/js/elfinder.min.js') ?>"></script>
 
-    <?php if($locale){ ?>
+    <?php if ($locale) { ?>
         <!-- elFinder translation (OPTIONAL) -->
         <script src="<?= asset($dir."/js/i18n/elfinder.$locale.js") ?>"></script>
     <?php } ?>
@@ -40,7 +40,7 @@ 
         $().ready(function() {
             var elf = $('#elfinder').elfinder({
                 // set your elFinder options here
-                <?php if($locale){ ?>
+                <?php if ($locale) { ?>
                     lang: '<?= $locale ?>', // locale
                 <?php } ?>
                 customData: {