Laravel-Backpack / CRUD

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

src/app/Http/Controllers/CrudController.php

Doc Comments +1 added lines, -1 removed lines

@@ -161,7 +161,7 @@
 	 * Remove the specified resource from storage.
 	 *
 	 * @param  int  $id
-	 * @return string
+	 * @return \Illuminate\Http\Response
 	 */
 	public function destroy($id)
 	{

Unused Use Statements -4 removed lines

@@ -3,16 +3,12 @@
 use Illuminate\Foundation\Bus\DispatchesJobs;
 use Illuminate\Routing\Controller as BaseController;
 use Illuminate\Foundation\Validation\ValidatesRequests;
-use Illuminate\Foundation\Auth\Access\AuthorizesRequests;
-use Illuminate\Support\Facades\Form as Form;
 use Illuminate\Http\Request;
 use Backpack\CRUD\Crud;
-use Crypt;
 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 {

src/app/Http/Controllers/ExampleCrudController.php

Unused Use Statements -4 removed lines

@@ -1,12 +1,8 @@
 <?php namespace App\Http\Controllers\Admin;
 
-use App\Http\Requests;
-use App\Http\Controllers\Controller;
 use Backpack\CRUD\app\Http\Controllers\CrudController;
-use Illuminate\Http\Request;
 
 // 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 ExampleCrudController extends CrudController {

src/app/Http/Controllers/ToneCrudController.php

Unused Use Statements -3 removed lines

@@ -1,9 +1,6 @@
 <?php namespace Backpack\Crud\app\Http\Controllers;
 
-use Illuminate\Foundation\Bus\DispatchesJobs;
 use Illuminate\Routing\Controller as BaseController;
-use Illuminate\Foundation\Validation\ValidatesRequests;
-use Illuminate\Foundation\Auth\Access\AuthorizesRequests;
 use Backpack\CRUD\Crud;
 
 /**

src/app/Http/Controllers/ToneCrudNestedController.php

Doc Comments +2 added lines, -2 removed lines

@@ -54,7 +54,7 @@ 
      * Store a newly created resource in storage.
      *
      * @param  \Illuminate\Http\Request  $request
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function crudStore(Request $request = null)
     {
@@ -101,7 +101,7 @@ 
      *
      * @param  \Illuminate\Http\Request  $request
      * @param  int  $id
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function crudUpdate($parentId, $id, Request $request = null)
     {

Unused Use Statements -3 removed lines

@@ -4,10 +4,7 @@
  */
 namespace Backpack\Crud\Http\Controllers;
 
-use Illuminate\Foundation\Bus\DispatchesJobs;
 use Illuminate\Routing\Controller as BaseController;
-use Illuminate\Foundation\Validation\ValidatesRequests;
-use Illuminate\Foundation\Auth\Access\AuthorizesRequests;
 use Backpack\Crud\Crud;
 
 /**

src/Crud.php

Doc Comments +24 added lines, -2 removed lines

@@ -286,6 +286,7 @@ 
      * Delete a row from the database.
      *
      * @param  [int] The id of the item to be deleted.
+     * @param integer $id
      * @return [bool] Deletion confirmation.
      *
      * TODO: should this delete items with relations to it too?
@@ -404,7 +405,8 @@ 
      * Check if a permission is enabled for a Crud Panel. Fail if not.
      *
      * @param  [string] Permission.
-     * @return boolean
+     * @param string $permission
+     * @return boolean|null
      */
     public function hasAccessOrFail($permission)
     {
@@ -433,6 +435,7 @@ 
      * All Create-Read-Update-Delete operations are done using that Eloquent Collection.
      *
      * @param [string] Full model namespace. Ex: App\Models\Article
+     * @param string $model_namespace
      */
     public function setModel($model_namespace)
     {
@@ -460,6 +463,7 @@ 
      *
      * @param [string] Route name.
      * @param [array] Parameters.
+     * @param string $route
      */
     public function setRoute($route)
     {
@@ -492,7 +496,7 @@ 
      * - $this->crud->setRouteName('admin.article')
      * - $this->crud->route = "admin/article"
      *
-     * @return [string]
+     * @return string
      */
     public function getRoute()
     {
@@ -505,6 +509,8 @@ 
      *
      * @param [string] Entity name, in singular. Ex: article
      * @param [string] Entity name, in plural. Ex: articles
+     * @param string $singular
+     * @param string $plural
      */
     public function setEntityNameStrings($singular, $plural) {
         $this->entity_name = $singular;
@@ -1237,6 +1243,9 @@ 
         return array_filter($this->{$entity}[] = $this->syncField($field));
     }
 
+    /**
+     * @param string $entity
+     */
     public function addMultiple($entity, $field)
     {
         $this->{$entity} = array_filter(array_map([$this, 'syncField'], $fields));
@@ -1256,16 +1265,25 @@ 
 
 
 
+    /**
+     * @param string $entity
+     */
     public function remove($entity, $fields)
     {
         return array_values(array_filter($this->{$entity}, function($field) use ($fields) { return !in_array($field['name'], (array)$fields);}));
     }
 
+    /**
+     * @param string $items
+     */
     public function setSort($items, $order)
     {
         $this->sort[$items] = $order;
     }
 
+    /**
+     * @param string $items
+     */
     public function sort($items)
     {
         if (array_key_exists($items, $this->sort))
@@ -1301,6 +1319,10 @@ 
     }
 
     // face un fel de merge intre ce ii dai si ce e in CRUD
+
+    /**
+     * @param string $entity
+     */
     public function syncRelations($entity)
     {
         foreach ($this->relations as $field => $relation) {

src/CrudTrait.php

Unused Use Statements -3 removed lines

@@ -1,9 +1,6 @@
 <?php namespace Backpack\CRUD;
 
-use Illuminate\Database\Eloquent\Collection;
-use Illuminate\Database\Eloquent\Model;
 use DB;
-use Lang;
 
 trait CrudTrait {
 

src/ToneCrud.php

Doc Comments +25 added lines

@@ -38,6 +38,9 @@ 
         $this->initState();
     }
 
+    /**
+     * @param string $model
+     */
     public function setModel($model)
     {
         if (!class_exists($model)) throw new \Exception('This model does not exist.', 404);
@@ -54,6 +57,10 @@ 
     }
 
     // TODO: make this work without having to specify "index"
+
+    /**
+     * @param string $route
+     */
     public function setRoute($route, $parameters = [])
     {
         if (!\Route::has($route)) throw new \Exception('This route does not exist.', 404);
@@ -259,6 +266,9 @@ 
         return $model;
     }
 
+    /**
+     * @param integer $id
+     */
     public function delete($id) // DONE
     {
         return $this->model->destroy($id);
@@ -489,6 +499,9 @@ 
         return array_filter($this->{$entity}[] = $this->syncField($field));
     }
 
+    /**
+     * @param string $entity
+     */
     private function addMultiple($entity, $field)
     {
         $this->{$entity} = array_filter(array_map([$this, 'syncField'], $fields));
@@ -506,16 +519,25 @@ 
         }
     }
 
+    /**
+     * @param string $entity
+     */
     private function remove($entity, $fields)
     {
         return array_values(array_filter($this->{$entity}, function($field) use ($fields) { return !in_array($field['name'], (array)$fields);}));
     }
 
+    /**
+     * @param string $items
+     */
     private function setSort($items, $order)
     {
         $this->sort[$items] = $order;
     }
 
+    /**
+     * @param string $items
+     */
     private function sort($items)
     {
         if (array_key_exists($items, $this->sort))
@@ -561,6 +583,9 @@ 
         return $values->get()->lists($field, $model->getKeyName())->toArray();
     }
 
+    /**
+     * @param string $entity
+     */
     private function syncRelations($entity)
     {
         foreach ($this->relations as $field => $relation) {