afrittella / back-project

src/app/Http/ViewComposers/AdminMenuComposer.php

Spacing +1 added lines, -1 removed lines

@@ -91,7 +91,7 @@
           <a href="%s"><i class="%s text-red"></i><span>%s</span></a>
         ' // url, icon, title
         ];
-        $traverse = function ($rows) use (&$traverse, $templates, $user) {
+        $traverse = function($rows) use (&$traverse, $templates, $user) {
             $menuString = "";
             $hasActive = false;
             foreach ($rows as $menu) {

Indentation +18 added lines, -18 removed lines

@@ -12,7 +12,7 @@ 
 
     public function __construct(MenuRepository $menuRepository)
     {
-      $this->menuRepository = $menuRepository;
+        $this->menuRepository = $menuRepository;
     }
 
     public function compose(View $view)
@@ -95,22 +95,22 @@ 
             $menuString = "";
             $hasActive = false;
             foreach ($rows as $menu) {
-              if (!empty($menu->permission) and !$user->hasPermission($menu->permission)) {
-                  continue;
-              }
+                if (!empty($menu->permission) and !$user->hasPermission($menu->permission)) {
+                    continue;
+                }
 
-              $hasActive = false;
-              $link = sprintf($templates['menu_link'],
+                $hasActive = false;
+                $link = sprintf($templates['menu_link'],
                 (!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
-              );
+                );
 
-              $submenu = "";
-              $authorized = true;
+                $submenu = "";
+                $authorized = true;
 
-              if ($menu->children->count() > 0) {
+                if ($menu->children->count() > 0) {
                 list($submenuString, $hasActive) = $traverse($menu->children);
                 $submenu = "";
                 if (!empty($submenuString)) {
@@ -118,19 +118,19 @@ 
                 } else {
                     $authorized = false;
                 }
-              }
+                }
 
-              $class = (!empty($submenu) ? 'treeview' : '');
-              $current_url = \Route::current()->uri();
+                $class = (!empty($submenu) ? 'treeview' : '');
+                $current_url = \Route::current()->uri();
 
-              if ($authorized) {
-                  $menuString .= sprintf($templates['menu_row'], $class, $link.$submenu);
-              }
+                if ($authorized) {
+                    $menuString .= sprintf($templates['menu_row'], $class, $link.$submenu);
+                }
         }
 
         return [
-              $menuString,
-              $hasActive
+                $menuString,
+                $hasActive
             ];
         };
 

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/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('users.confirm');
         Route::auth();
 
@@ -8,7 +8,7 @@ 
         Route::get('auth/{provider}/callback', 'Auth\SocialLoginController@handleProviderCallback')->name('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('admin.dashboard');
         Route::get('account', 'UsersController@account')->name('admin.account');
         Route::put('account', 'UsersController@accountStore')->name('admin.add-account');
@@ -18,7 +18,7 @@ 
         Route::get('attachments/{attachment}/main', 'AttachmentsController@setMain')->name('attachments.main');
         Route::resource('attachments', 'AttachmentsController', ['except' => ['destroy', 'show']]);
         // Users
-        Route::group(['middleware' => 'role:administrator'], function () {
+        Route::group(['middleware' => 'role:administrator'], function() {
             Route::get('users/{user}/delete', 'UsersController@delete')->name('users.delete'); // Implementing delete avoiding DELETE method
             Route::resource('users', 'UsersController', ['except' => ['destroy', 'show']]);
 

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/Attachable.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php namespace Afrittella\BackProject\Traits;
 
 use Afrittella\BackProject\Models\Attachment;
-use Illuminate\Support\Facades\Storage;
 
 trait Attachable
 {

src/app/BackProjectServiceProvider.php

Spacing +14 added lines, -14 removed lines

@@ -33,13 +33,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($router);
@@ -63,11 +63,11 @@ 
             return new BackProject($app);
         });*/
 
-        $this->app->singleton('attachments-manager', function ($app) {
+        $this->app->singleton('attachments-manager', function($app) {
             return new MediaManager();
         });
 
-        $this->app->singleton('back-project', function ($app) {
+        $this->app->singleton('back-project', function($app) {
             return new BackProject();
         });
 
@@ -104,15 +104,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');
@@ -127,7 +127,7 @@ 
         //$router->aliasMiddleware('guest', \Afrittella\BackProject\Http\Middleware\RedirectIfAuthenticated::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()
@@ -137,8 +137,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');
             }
         }
@@ -178,7 +178,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);