afrittella / back-project

src/app/Traits/HasOneAttachment.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
      */
     public function attachments()
     {
-        return $this->morphToMany( Attachment::class, 'attachable' )->orderBy('is_main', 'desc');
+        return $this->morphToMany(Attachment::class, 'attachable')->orderBy('is_main', 'desc');
     }
 
     public function getAttachment() {

src/app/Traits/HasManyAttachments.php

Spacing +1 added lines, -1 removed lines

@@ -19,6 +19,6 @@
 
     public function attachments()
     {
-        return $this->morphToMany( Attachment::class, 'attachable' )->orderBy('is_main', 'desc');
+        return $this->morphToMany(Attachment::class, 'attachable')->orderBy('is_main', 'desc');
     }
 }

src/app/Repositories/Base.php

Spacing +5 added lines, -5 removed lines

@@ -48,7 +48,7 @@ 
         return $this->model->create($data);
     }
 
-    public function update(array $data, $id, $attribute="id")
+    public function update(array $data, $id, $attribute = "id")
     {
         $model_data = $this->model->where($attribute, '=', $id)->first();
 
@@ -123,7 +123,7 @@ 
         return $this;
     }
 
-    public function skipCriteria($status = true){
+    public function skipCriteria($status = true) {
         $this->skipCriteria = $status;
         return $this;
     }
@@ -143,11 +143,11 @@ 
     }
 
     public function  applyCriteria() {
-        if($this->skipCriteria === true)
+        if ($this->skipCriteria === true)
             return $this;
 
-        foreach($this->getCriteria() as $criteria) {
-            if($criteria instanceof Criteria)
+        foreach ($this->getCriteria() as $criteria) {
+            if ($criteria instanceof Criteria)
                 $this->model = $criteria->apply($this->model, $this);
         }
 

Doc Comments +6 added lines

@@ -29,6 +29,9 @@ 
         $this->makeModel();
     }
 
+    /**
+     * @return string
+     */
     abstract public function model();
 
     public function all($columns = ['*'])
@@ -96,6 +99,9 @@ 
         return $this->model->findOrCreate($data);
     }
 
+    /**
+     * @param string $attribute
+     */
     public function findBy($attribute, $value, $columns = array('*'))
     {
         $this->applyCriteria();

Braces +9 added lines, -6 removed lines

@@ -107,8 +107,9 @@ 
         $model = $this->app->make($this->model());
         //$model = app()->make($this->model());
 
-        if (!$model instanceof Model)
-            throw new RepositoryException("Class {$this->model()} must be an instance of Illuminate\\Database\\Eloquent\\Model");
+        if (!$model instanceof Model) {
+                    throw new RepositoryException("Class {$this->model()} must be an instance of Illuminate\\Database\\Eloquent\\Model");
+        }
 
         return $this->model = $model;
     }
@@ -143,12 +144,14 @@ 
     }
 
     public function  applyCriteria() {
-        if($this->skipCriteria === true)
-            return $this;
+        if($this->skipCriteria === true) {
+                    return $this;
+        }
 
         foreach($this->getCriteria() as $criteria) {
-            if($criteria instanceof Criteria)
-                $this->model = $criteria->apply($this->model, $this);
+            if($criteria instanceof Criteria) {
+                            $this->model = $criteria->apply($this->model, $this);
+            }
         }
 
         return $this;

src/app/Http/Controllers/AttachmentsController.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Afrittella\BackProject\Http\Controllers;
 
-use Afrittella\BackProject\Exceptions\NotFoundException;
 use Afrittella\BackProject\Http\Requests\AttachmentAdd;
 use Afrittella\BackProject\Repositories\Attachments;
 use Afrittella\BackProject\Repositories\Criteria\Attachments\ByUser;

src/app/Traits/Sluggable.php

Doc Comments +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
 
     /**
      * Get similar slugs
-     * @param $slug
+     * @param string $slug
      * @param $id
      * @return mixed
      */

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
         $i = 1;
 
         while ($completed == false) {
-            $newSlug = $slug .'-'.$i;
+            $newSlug = $slug.'-'.$i;
             if (!$relatedSlugs->contains('slug', $newSlug)) {
                 return $newSlug;
             }

src/app/Services/MediaManager.php

Indentation +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     {
         //@TODO implement google or s3 cloud storage support
         //if (config('filesystems.default') == 'google') {
-          //  $this->media_path = Storage::disk()->getDriver()->getAdapter()->getStorageApiUri() . '/' . Storage::disk()->getDriver()->getAdapter()->getBucket()->name();
+            //  $this->media_path = Storage::disk()->getDriver()->getAdapter()->getStorageApiUri() . '/' . Storage::disk()->getDriver()->getAdapter()->getBucket()->name();
         //} else {
         //$this->media_path = Storage::disk()->getDriver()->getAdapter()->getPathPrefix();
         //}

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
     {
         $format = $this->chooseFormat($format);
 
-        return '<img src="'.$this->getCachedImageUrl($format, $image->name).'" alt="'.$image->description.'" '.$attributes. '>';
+        return '<img src="'.$this->getCachedImageUrl($format, $image->name).'" alt="'.$image->description.'" '.$attributes.'>';
     }
 
     protected function chooseFormat($format)

src/app/Http/ViewComposers/AdminMenuComposer.php

Spacing +3 added lines, -3 removed lines

@@ -70,7 +70,7 @@ 
         ' // url, icon, title
         ];
 
-        $traverse = function ($rows) use (&$traverse, $templates, $user) {
+        $traverse = function($rows) use (&$traverse, $templates, $user) {
             $menuString = "";
             $hasActive = false;
             foreach ($rows as $menu) {
@@ -97,14 +97,14 @@ 
                     (!empty($menu->route) ? url($menu->route) : '#'),
                     (!empty($menu->icon) ? $menu->icon : 'fa fa-circle-o'),
                     //trans('back-project::base.dashboard')
-                    \Lang::has('back-project::menu.' . $menu->title) ? __('back-project::menu.' . $menu->title) : $menu->title,
+                    \Lang::has('back-project::menu.'.$menu->title) ? __('back-project::menu.'.$menu->title) : $menu->title,
                     $menu_caret
                 );
                 $class = (!empty($submenu) ? 'treeview' : '');
                 $current_url = \Route::current()->uri();
 
                 if ($authorized) {
-                    $menuString .= sprintf($templates['menu_row'], $class, $link . $submenu);
+                    $menuString .= sprintf($templates['menu_row'], $class, $link.$submenu);
                 }
             }
 

src/app/BackProjectServiceProvider.php

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 use Afrittella\BackProject\Services\BackProject;
 use Afrittella\BackProject\Services\SlugGenerator;
 use Illuminate\Support\ServiceProvider;
-use Illuminate\Routing\Router;
 use function Symfony\Component\HttpKernel\Tests\controller_func;
 
 

Spacing +13 added lines, -13 removed lines

@@ -37,13 +37,13 @@ 
         // - first the published views (in case they have any changes)
         $this->loadViewsFrom(resource_path('views/vendor/back-project/base'), 'back-project');
         // - then the stock views that come with the package, in case a published view might be missing
-        $this->loadViewsFrom(__DIR__ . '/../resources/views', 'back-project');
+        $this->loadViewsFrom(__DIR__.'/../resources/views', 'back-project');
         // Load Translations
-        $this->loadTranslationsFrom(__DIR__ . '/../resources/lang', 'back-project');
+        $this->loadTranslationsFrom(__DIR__.'/../resources/lang', 'back-project');
 
         // use the vendor configuration file as fallback
         $this->mergeConfigFrom(
-            __DIR__ . '/../config/config.php', 'back-project'
+            __DIR__.'/../config/config.php', 'back-project'
         );
 
         $this->routes();
@@ -62,7 +62,7 @@ 
 
     public function register()
     {
-        $this->app->singleton('media-manager', function ($app) {
+        $this->app->singleton('media-manager', function($app) {
             return new MediaManager();
         });
 
@@ -105,15 +105,15 @@ 
     public function publishFiles()
     {
         // publish config file
-        $this->publishes([__DIR__ . '/../config/config.php' => config_path() . '/back-project.php'], 'config');
+        $this->publishes([__DIR__.'/../config/config.php' => config_path().'/back-project.php'], 'config');
         // publish lang files
-        $this->publishes([__DIR__ . '/../resources/lang' => resource_path('lang/vendor/back-project')], 'lang');
+        $this->publishes([__DIR__.'/../resources/lang' => resource_path('lang/vendor/back-project')], 'lang');
         // publish public BackProject assets
-        $this->publishes([__DIR__ . '/../public' => public_path('vendor/back-project')], 'public');
+        $this->publishes([__DIR__.'/../public' => public_path('vendor/back-project')], 'public');
         // publish views
-        $this->publishes([__DIR__ . '/../resources/views' => resource_path('views/vendor/back-project')], 'views');
+        $this->publishes([__DIR__.'/../resources/views' => resource_path('views/vendor/back-project')], 'views');
         // publish error views
-        $this->publishes([__DIR__ . '/../resources/error_views' => resource_path('views/errors')], 'errors');
+        $this->publishes([__DIR__.'/../resources/error_views' => resource_path('views/errors')], 'errors');
         // publish public AdminLTE assets
         $this->publishes(['vendor/almasaeed2010/adminlte/bootstrap' => public_path('vendor/adminlte/bootstrap')], 'adminlte');
         $this->publishes(['vendor/almasaeed2010/adminlte/dist' => public_path('vendor/adminlte/dist')], 'adminlte');
@@ -128,7 +128,7 @@ 
         //$router->aliasMiddleware('admin', \Afrittella\BackProject\Http\Middleware\Admin::class);
         //$router->aliasMiddleware('role', \Afrittella\BackProject\Http\Middleware\Role::class);
 
-        $this->loadRoutesFrom(__DIR__ . '/../routes/web.php');
+        $this->loadRoutesFrom(__DIR__.'/../routes/web.php');
     }
 
     public function handleMigrations()
@@ -138,8 +138,8 @@ 
                 $timestamp = date('Y_m_d_His', time());
 
                 $this->publishes([
-                    __DIR__ . '/../database/migrations/' . $file . '.php' =>
-                        database_path('migrations/' . $timestamp . '_' . $file . '.php')
+                    __DIR__.'/../database/migrations/'.$file.'.php' =>
+                        database_path('migrations/'.$timestamp.'_'.$file.'.php')
                 ], 'migrations');
             }
         }
@@ -179,7 +179,7 @@ 
     public function loadHelpers()
     {
         foreach ($this->helpers as $helper):
-            $file = __DIR__ . '/Helpers/' . $helper . '.php';
+            $file = __DIR__.'/Helpers/'.$helper.'.php';
 
             if (file_exists($file)) {
                 require_once($file);

src/routes/web.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
-Route::group(['namespace' => '\Afrittella\BackProject\Http\Controllers'], function () {
-    Route::group(['middleware' => 'web'], function () {
+Route::group(['namespace' => '\Afrittella\BackProject\Http\Controllers'], function() {
+    Route::group(['middleware' => 'web'], function() {
         Route::get('confirm/{code}/{user}', 'Auth\RegisterController@confirm')->name('bp.users.confirm');
 
         if (empty(config('back-project.use_custom_auth_routes'))) {
@@ -11,7 +11,7 @@ 
         Route::get('auth/{provider}/callback', 'Auth\SocialLoginController@handleProviderCallback')->name('bp.social_callback');
     });
 
-    Route::group(['middleware' => 'web', 'prefix' => config('back-project.route_prefix')], function () {
+    Route::group(['middleware' => 'web', 'prefix' => config('back-project.route_prefix')], function() {
         Route::get('dashboard', 'AdminController@dashboard')->name('bp.admin.dashboard');
         Route::get('account', 'UsersController@account')->name('bp.admin.account');
         Route::put('account', 'UsersController@accountStore')->name('bp.admin.add-account');
@@ -21,7 +21,7 @@ 
         Route::get('attachments/{attachment}/main', 'AttachmentsController@setMain')->name('bp.attachments.main');
         Route::resource('attachments', 'AttachmentsController', ['except' => ['destroy', 'show'], 'as' => 'bp']);
         // Users
-        Route::group(['middleware' => 'role:administrator'], function () {
+        Route::group(['middleware' => 'role:administrator'], function() {
             Route::get('users/{user}/delete', 'UsersController@delete')->name('bp.users.delete'); // Implementing delete avoiding DELETE method
             Route::resource('users', 'UsersController', ['except' => ['destroy', 'show'], 'as' => 'bp']);