[TASK] Some code review

Classes/Controller/OverviewController.php

@@ -375,16 +375,13 @@ class OverviewController extends AbstractController {
 	 * Renders the news in a paginated list
 	 *
 	 * @param array $newsMetaData
-	 * @param array $newsFilter
+	 * @param array|null $newsFilter
 	 * @return void
 	 * @throws \InvalidArgumentException
 	 * @throws \TYPO3\CMS\Extbase\Persistence\Exception\InvalidQueryException
+	 * @throws \TYPO3\CMS\Extbase\Mvc\Exception\NoSuchArgumentException
 	 */
 	protected function overviewWithoutCategoriesAction(array $newsMetaData = [], array $newsFilter = NULL) {
-		if (empty($newsFilter)) {
-			$newsFilter = $this->request->getArguments()['newsFilter'];
-		}
-
 		// remember selection of the filter values, if any
 		$selectedTag = $this->tagRepository->findByUid((int) $newsFilter['tag']);
 		$selectedCategory = $this->categoryRepository->findByUid((int) $newsFilter['category']);
@@ -454,7 +451,7 @@ class OverviewController extends AbstractController {
 			$nextPage = $currentPageBrowserPage + 1;
 			if ($nextPage <= $numberOfPages) {
 				GeneralUtility::_GETset(['tx_sgnews_pagebrowser' => ['currentPage' => $nextPage]]);
-				$this->overviewWithoutCategoriesAction($newsMetaData, $this->request->getArguments()['newsFilter']);
+				$this->overviewWithoutCategoriesAction($newsMetaData, $newsFilter);
 				return;
 			}
 		}