Skip to content
Snippets Groups Projects
Commit 37117421 authored by Stefan Galinski's avatar Stefan Galinski :video_game:
Browse files

[BUGFIX] Pagination throws 404 if no data is available, because of wrong type check

parent b4ffe0da
No related branches found
No related tags found
No related merge requests found
...@@ -138,7 +138,7 @@ class ListByCategoryController extends AbstractController { ...@@ -138,7 +138,7 @@ class ListByCategoryController extends AbstractController {
$newsPerPage = (int) $this->settings['newsLimitPerPage']; $newsPerPage = (int) $this->settings['newsLimitPerPage'];
$newsCount = $this->newsRepository->newsCountByCategories($categoryUids, $tagUids, $startTime, $endTime); $newsCount = $this->newsRepository->newsCountByCategories($categoryUids, $tagUids, $startTime, $endTime);
$numberOfPages = ($newsPerPage <= 0 ? 0 : ceil($newsCount / $newsPerPage)); $numberOfPages = (int) ($newsPerPage <= 0 ? 0 : ceil($newsCount / $newsPerPage));
if ($numberOfPages !== 0 && $currentPageBrowserPage >= $numberOfPages) { if ($numberOfPages !== 0 && $currentPageBrowserPage >= $numberOfPages) {
/** @var ErrorController $errorController */ /** @var ErrorController $errorController */
$errorController = GeneralUtility::makeInstance(ErrorController::class); $errorController = GeneralUtility::makeInstance(ErrorController::class);
......
...@@ -312,7 +312,7 @@ class OverviewController extends AbstractController { ...@@ -312,7 +312,7 @@ class OverviewController extends AbstractController {
$this->highlightBestFitNews($categoryIds); $this->highlightBestFitNews($categoryIds);
$newsCount = $this->newsRepository->newsCountByCategories($categoryIds, $tagIds, $startTime, $endTime); $newsCount = $this->newsRepository->newsCountByCategories($categoryIds, $tagIds, $startTime, $endTime);
$numberOfPages = ($newsLimitPerCategory <= 0 ? 0 : ceil($newsCount / $newsLimitPerCategory)); $numberOfPages = (int) ($newsLimitPerCategory <= 0 ? 0 : ceil($newsCount / $newsLimitPerCategory));
if ($numberOfPages !== 0 && $currentPageBrowserPage >= $numberOfPages) { if ($numberOfPages !== 0 && $currentPageBrowserPage >= $numberOfPages) {
/** @var ErrorController $errorController */ /** @var ErrorController $errorController */
$errorController = GeneralUtility::makeInstance(ErrorController::class); $errorController = GeneralUtility::makeInstance(ErrorController::class);
...@@ -474,7 +474,7 @@ class OverviewController extends AbstractController { ...@@ -474,7 +474,7 @@ class OverviewController extends AbstractController {
// Check to achieve less Ajax calls. // Check to achieve less Ajax calls.
$newsCount = $this->newsRepository->newsCountByCategories([], $tagIds, $startTime, $endTime); $newsCount = $this->newsRepository->newsCountByCategories([], $tagIds, $startTime, $endTime);
$numberOfPages = ($newsLimitPerTag <= 0 ? 0 : ceil($newsCount / $newsLimitPerTag)); $numberOfPages = (int) ($newsLimitPerTag <= 0 ? 0 : ceil($newsCount / $newsLimitPerTag));
if ($numberOfPages !== 0 && $currentPageBrowserPage >= $numberOfPages) { if ($numberOfPages !== 0 && $currentPageBrowserPage >= $numberOfPages) {
/** @var ErrorController $errorController */ /** @var ErrorController $errorController */
$errorController = GeneralUtility::makeInstance(ErrorController::class); $errorController = GeneralUtility::makeInstance(ErrorController::class);
...@@ -614,7 +614,7 @@ class OverviewController extends AbstractController { ...@@ -614,7 +614,7 @@ class OverviewController extends AbstractController {
} }
$this->highlightBestFitNews($categoryIds); $this->highlightBestFitNews($categoryIds);
$numberOfPages = ($newsPerPage <= 0 ? 0 : ceil($newsCount / $newsPerPage)); $numberOfPages = (int) ($newsPerPage <= 0 ? 0 : ceil($newsCount / $newsPerPage));
if ($numberOfPages !== 0 && $currentPageBrowserPage >= $numberOfPages) { if ($numberOfPages !== 0 && $currentPageBrowserPage >= $numberOfPages) {
/** @var ErrorController $errorController */ /** @var ErrorController $errorController */
$errorController = GeneralUtility::makeInstance(ErrorController::class); $errorController = GeneralUtility::makeInstance(ErrorController::class);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment