some cleanup of routes

app/Providers/AppServiceProvider.php

@@ -33,9 +33,5 @@ class AppServiceProvider extends ServiceProvider
         $this->app->bind('App\Services\AircraftFareService', function($app) {
             return new \App\Services\AircraftFareService();
         });
-
-        if (in_array($this->app->environment(), ['local', 'dev', 'unittest'])) {
-            $this->app->register(\Bpocallaghan\Generators\GeneratorsServiceProvider::class);
-        }
     }
 }

app/Providers/RouteServiceProvider.php

@@ -36,10 +36,7 @@ class RouteServiceProvider extends ServiceProvider
     public function map()
     {
         $this->mapWebRoutes();
-
         $this->mapApiRoutes();
-
-        //
     }
 
     /**

composer.json

@@ -29,8 +29,7 @@
         "phpunit/phpunit": "~5.7",
         "symfony/css-selector": "3.1.*",
         "symfony/dom-crawler": "3.1.*",
-        "laravel/homestead": "^5.4",
+        "laravel/homestead": "^5.4"
-        "bpocallaghan/generators": "^4.0"
     },
     "autoload": {
         "classmap": [

config/app.php

@@ -110,7 +110,7 @@ return [
         'Flash' => Laracasts\Flash\Flash::class,
         'Yaml' => Symfony\Component\Yaml\Yaml::class,
         'Geotools' => Toin0u\Geotools\Facade\Geotools::class,
-        'Setting' => 'anlutro\LaravelSettings\Facade',
+        'Setting' => anlutro\LaravelSettings\Facade::class,
     ],
 
 ];

routes/admin.php

@@ -1,21 +1,31 @@
 <?php
 /**
- * Admin routes
+ * Admin Routes
  */
 
-/*Route::get('/admin', [
-    'middleware' => ['role:admin'],
-    'uses' => 'Http\Controllers\Admin\DashboardController@index'
-]);
-
 Route::group([
-    'namespace' => 'Http\Controllers\Admin',
+     'namespace' => 'Admin', 'prefix' => 'admin', 'as' => 'admin.',
-    //'middleware' => ['role:admin'],
+     'middleware' => ['role:admin'],
-    'prefix' => 'admin',
+ ], function () {
-], function () {
     Route::resource('airports', 'AirportController');
     Route::resource('airlines', 'AirlinesController');
-    Route::resource('aircraft', 'AircraftController');
     Route::resource('aircraftclasses', 'AircraftClassController');
     Route::resource('fares', 'FareController');
-});*/
+
+    # aircraft and fare associations
+    Route::resource('aircraft', 'AircraftController');
+    Route::match(['get', 'post', 'put', 'delete'], 'aircraft/{id}/fares', 'AircraftController@fares');
+
+    # flights and aircraft associations
+    Route::resource('flights', 'FlightController');
+    Route::match(['get', 'post', 'put', 'delete'], 'flights/{id}/aircraft', 'FlightController@aircraft');
+
+    # view/update settings
+    Route::match(['get'], 'settings', 'SettingsController@index');
+    Route::match(['post', 'put'], 'settings', 'SettingsController@update');
+
+    # defaults
+    Route::get('', ['uses' => 'DashboardController@index']);
+    Route::get('/', ['uses' => 'DashboardController@index']);
+    Route::get('/dashboard', ['uses' => 'DashboardController@index', 'name' => 'dashboard']);
+});

routes/web.php

@@ -4,6 +4,9 @@ Route::get('/', 'HomeController@index');
 Route::get('/home', 'HomeController@index');
 
 
+/**
+ * These are only visible to a logged in user
+ */
 Route::group([
     'namespace' => 'Frontend', 'prefix' => 'frontend', 'as' => 'frontend.',
     'middleware' => ['role:admin|user'],
@@ -13,33 +16,4 @@ Route::group([
 
 Auth::routes();
 
-/**
+require base_path('routes/admin.php');
- * Admin Routes
- */
-
-Route::group([
-    'namespace' => 'Admin', 'prefix' => 'admin', 'as' => 'admin.',
-    'middleware' => ['role:admin'],
-], function () {
-    Route::resource('airports', 'AirportController');
-    Route::resource('airlines', 'AirlinesController');
-    Route::resource('aircraftclasses', 'AircraftClassController');
-    Route::resource('fares', 'FareController');
-
-    # aircraft and fare associations
-    Route::resource('aircraft', 'AircraftController');
-    Route::match(['get', 'post', 'put', 'delete'], 'aircraft/{id}/fares', 'AircraftController@fares');
-
-    # flights and aircraft associations
-    Route::resource('flights', 'FlightController');
-    Route::match(['get', 'post', 'put', 'delete'], 'flights/{id}/aircraft', 'FlightController@aircraft');
-
-    # view/update settings
-    Route::match(['get'], 'settings', 'SettingsController@index');
-    Route::match(['post', 'put'], 'settings', 'SettingsController@update');
-
-    # defaults
-    Route::get('', ['uses' => 'DashboardController@index']);
-    Route::get('/', ['uses' => 'DashboardController@index']);
-    Route::get('/dashboard', ['uses' => 'DashboardController@index','name' => 'dashboard']);
-});