X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=src%2FWallabag%2FCoreBundle%2FController%2FEntryController.php;h=56759adba0625ad5b71b74680bde2dbb1ad980d6;hb=074110ca2dd3ba0d58f2fa93076933d06b46df77;hp=6e56c2373313dde73d3bf3cd52f6937ce672a6d8;hpb=9b5552290445b404e6de0595d7d27a2de824230a;p=github%2Fwallabag%2Fwallabag.git diff --git a/src/Wallabag/CoreBundle/Controller/EntryController.php b/src/Wallabag/CoreBundle/Controller/EntryController.php index 6e56c237..56759adb 100644 --- a/src/Wallabag/CoreBundle/Controller/EntryController.php +++ b/src/Wallabag/CoreBundle/Controller/EntryController.php @@ -532,13 +532,17 @@ class EntryController extends Controller $repository = $this->get('wallabag_core.entry_repository'); $searchTerm = (isset($request->get('search_entry')['term']) ? $request->get('search_entry')['term'] : ''); $currentRoute = (null !== $request->query->get('currentRoute') ? $request->query->get('currentRoute') : ''); - $direction = (null !== $request->query->get('entry_sort')['sortOrder'] ? $request->query->get('entry_sort')['sortOrder'] : 'asc'); - - // defined as null by default because each repository method have the right field as default value too - // like `getBuilderForStarredByUser` will have `starredAt` sort by default + $direction = 'asc'; $sortBy = null; - if (\in_array($request->get('entry_sort')['sortType'], ['id', 'title', 'createdAt', 'updatedAt', 'starredAt', 'archivedAt'], true)) { - $sortBy = $request->get('entry_sort')['sortType']; + + if (null !== ($request->query->get('entry_sort'))) { + $direction = (null !== $request->query->get('entry_sort')['sortOrder'] ? $request->query->get('entry_sort')['sortOrder'] : 'asc'); + + // defined as null by default because each repository method have the right field as default value too + // like `getBuilderForStarredByUser` will have `starredAt` sort by default + if (\in_array($request->get('entry_sort')['sortType'], ['id', 'title', 'createdAt', 'updatedAt', 'starredAt', 'archivedAt'], true)) { + $sortBy = $request->get('entry_sort')['sortType']; + } } switch ($type) {