FIX: fix few bugs in routes

This commit is contained in:
Michal Kváček
2017-08-11 04:13:33 +02:00
parent ef432702b4
commit 7d847cbab9
9 changed files with 30 additions and 22 deletions
+14 -13
View File
@@ -8,6 +8,7 @@ use App\CMS\Contracts\Widgets\WidgetRenderer;
use App\CMS\Widgets\Exceptions\MissingParameterException; use App\CMS\Widgets\Exceptions\MissingParameterException;
use App\Http\Requests\Request; use App\Http\Requests\Request;
use App\Models\Content\Content; use App\Models\Content\Content;
use Illuminate\Support\Facades\Cache;
use Mcamara\LaravelLocalization\Facades\LaravelLocalization; use Mcamara\LaravelLocalization\Facades\LaravelLocalization;
use Modules\Blog\Article\Handlers\Article; use Modules\Blog\Article\Handlers\Article;
@@ -15,7 +16,6 @@ class Menu implements PublicWidget
{ {
/** /**
* Name getter * Name getter
*
* @return string * @return string
*/ */
public static function getName() public static function getName()
@@ -25,7 +25,6 @@ class Menu implements PublicWidget
/** /**
* Getter for description * Getter for description
*
* @return string * @return string
*/ */
public static function getDescription() public static function getDescription()
@@ -35,7 +34,6 @@ class Menu implements PublicWidget
/** /**
* Getter for presenter/renderer of this widget * Getter for presenter/renderer of this widget
*
* @return WidgetRenderer * @return WidgetRenderer
*/ */
public static function getPresenter() public static function getPresenter()
@@ -75,16 +73,19 @@ class Menu implements PublicWidget
$allowedStatuses[] = IContent::STATUS_PROTECTED; $allowedStatuses[] = IContent::STATUS_PROTECTED;
} }
$pages = Content::where('language', LaravelLocalization::getCurrentLocale()) return Cache::remember('widget-menu-content-pages', 5, function () use ($ids, $allowedStatuses) {
->whereIn('id', $ids) $pages = Content::where('language', LaravelLocalization::getCurrentLocale())
->whereIn('status', $allowedStatuses) ->whereIn('id', $ids)
->where('moduleHandler', '!=', Article::class) ->whereIn('status', $allowedStatuses)
->get(); ->where('moduleHandler', '!=', Article::class)
$used = []; ->get();
foreach ($pages as $page) {
$used[$page->id] = $page;
}
return $used; $used = [];
foreach ($pages as $page) {
$used[$page->id] = $page;
}
return $used;
});
} }
} }
@@ -9,6 +9,12 @@ class Dashboard extends Controller
{ {
public function index(Request $request) public function index(Request $request)
{ {
// $data = unserialize(file_get_contents('/home/michal/decoded.bin'));
//
// dd($data);
return redirect()->route('content.overview'); return redirect()->route('content.overview');
} }
} }
+2 -3
View File
@@ -1,9 +1,9 @@
<?php <?php
namespace App\Http\Controllers; namespace App\Http\Controllers\Visitor;
use App\CMS\Contracts\Repositories\Content; use App\CMS\Contracts\Repositories\Content;
use App\Http\Requests; use App\Http\Controllers\Controller;
class Sitemap extends Controller class Sitemap extends Controller
{ {
@@ -24,7 +24,6 @@ class Sitemap extends Controller
/** /**
* Get sitemap for all content pages * Get sitemap for all content pages
*
* @return mixed * @return mixed
*/ */
public function getMap() public function getMap()
@@ -38,7 +38,7 @@ class AdminMenuRegistrator
// try to register menus // try to register menus
try { try {
call_user_func([$module->handler, 'registerAdminMenu'], $user); // call_user_func([$module->handler, 'registerAdminMenu'], $user);
} catch (\Exception $e) { } catch (\Exception $e) {
Log::debug('Error in ' . $module->handler . '::registerAdminMenu: ' . $e->getMessage()); Log::debug('Error in ' . $module->handler . '::registerAdminMenu: ' . $e->getMessage());
} }
Executable → Regular
View File
@@ -1,11 +1,11 @@
<?php <?php
namespace Modules\Blog\Article\Http\Controllers\Admin; namespace Modules\Blog\Article\Http\Controllers\Admin;
use App\Http\Controllers\Admin\Controller;
use Illuminate\Support\Facades\Event; use Illuminate\Support\Facades\Event;
use Modules\Blog\Article\Entities\Comment; use Modules\Blog\Article\Entities\Comment;
use Modules\Blog\Article\Events\ApprovedComment; use Modules\Blog\Article\Events\ApprovedComment;
use Modules\Blog\Article\Events\DeletedComment; use Modules\Blog\Article\Events\DeletedComment;
use Pingpong\Modules\Routing\Controller;
class EditComment extends Controller class EditComment extends Controller
{ {
@@ -2,11 +2,11 @@
namespace Modules\ContactForm\Http\Controllers\Visitor; namespace Modules\ContactForm\Http\Controllers\Visitor;
use App\CMS\Helpers\Template; use App\CMS\Helpers\Template;
use App\Http\Controllers\Controller;
use App\Models\Content\Content; use App\Models\Content\Content;
use Illuminate\Mail\Message; use Illuminate\Mail\Message;
use Illuminate\Support\Facades\Mail; use Illuminate\Support\Facades\Mail;
use Modules\ContactForm\Http\Requests\SendNewEmailRequest; use Modules\ContactForm\Http\Requests\SendNewEmailRequest;
use Pingpong\Modules\Routing\Controller;
class CreateNewMessage extends Controller class CreateNewMessage extends Controller
{ {
-3
View File
@@ -6,9 +6,6 @@ Route::pattern('id', '\d+');
Route::pattern('url', '[0-9\-a-z]+'); Route::pattern('url', '[0-9\-a-z]+');
/*******************************************************************************/ /*******************************************************************************/
Auth::routes();
$this->get('logout', 'Auth\LoginController@logout')
->name('logout');
/** /**
* This route group serves to handle request in admin * This route group serves to handle request in admin
+5
View File
@@ -12,6 +12,11 @@ Route::pattern('url', '[0-9\-a-z]+');
/*******************************************************************************/ /*******************************************************************************/
Route::group(['prefix' => LaravelLocalization::setLocale(), 'middleware' => ['bindings', 'web']], function () { Route::group(['prefix' => LaravelLocalization::setLocale(), 'middleware' => ['bindings', 'web']], function () {
Auth::routes();
$this->get('logout', 'Auth\LoginController@logout')
->name('logout');
// Homepage // Homepage
Route::get('/', 'Visitor\Homepage@getIndex') Route::get('/', 'Visitor\Homepage@getIndex')
->name('page.index'); ->name('page.index');