TYPO3 Core

Bug #90006

Updated by Bostjan Kristl over 4 years ago

In this bug fix 61657: [BUGFIX] Do not fall back to routes on invalid “id” 
 for fall back is now error page when we use eId or type (typeNum) pages and *id* as *query parameter* ! (for eId script i haven't checked) 

 --------------------------------------------------------------------------------------------------------- 
 *Before fix it was:* 

 $requestId = (string)($request->getQueryParams()['id'] ?? ''); 
             if (!empty($requestId) && !empty($page = $this->resolvePageId($requestId))) { 

 --------------------------------------------------------------------------------------------------------- 
 *with buxfix is:* 

 $requestId = (string)($request->getQueryParams()['id'] ?? ''); 
             if (!empty($requestId)) { 
                 $page = $this->resolvePageId($requestId); 
                 if ($page === null) { 
                     return GeneralUtility::makeInstance(ErrorController::class)->pageNotFoundAction( 
                         $request, 
                         'The requested page does not exist', 
                         ['code' => PageAccessFailureReasons::PAGE_NOT_FOUND] 
                     ); 
                 } 

 --------------------------------------------------------------------------------------------------------- 
 *BUT is supposed to be:* 


 $requestId = (string)($request->getQueryParams()['id'] ?? ''); 


             if (!empty($requestId)) { 

                
                 

                 $requestEID = (string)($request->getQueryParams()['eID'] ?? null); 
                 $requestType = (int)($request->getQueryParams()['type'] ?? null); 

                 if ($requestType ($requestEID !== null && $requestType !== null && $this->resolvePageId($requestId) === null) { 
                     return GeneralUtility::makeInstance(ErrorController::class)->pageNotFoundAction( 
                         $request, 
                         'The requested page does not exist', 
                         ['code' => PageAccessFailureReasons::PAGE_NOT_FOUND] 
                     ); 
                 } 



 Because we need to check if is eID script or type (typeNum) page. 

 Regards, 
 Bostjan 

Back