Bug #101243
closedFix in linkvalidator PagesRepository::doesRootLineContainHiddenPages
100%
Description
PagesRepository::doesRootlineContainHiddenPages() calls itself. It fetches fields from the DB without "pid" and passes the results to itself, then checks the value of pid, but this is always empty.
42 public function doesRootLineContainHiddenPages(array $pageInfo): bool 43 { 44 $pid = (int)($pageInfo['pid'] ?? 0); .... 59 $row = $queryBuilder 60 ->select('uid', 'title', 'hidden', 'extendToSubpages') .... 72 return $this->doesRootLineContainHiddenPages($row);
Also, it is not necessary to get the title here
Updated by Gerrit Code Review 10 months ago
- Status changed from New to Under Review
Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/79724
Updated by Gerrit Code Review 10 months ago
Patch set 2 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/79724
Updated by Gerrit Code Review 10 months ago
Patch set 3 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/79724
Updated by Gerrit Code Review 10 months ago
Patch set 4 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/79724
Updated by Sybille Peters 9 months ago
- Related to Bug #93895: Linkvalidator report doesn't list discovered bad links if the page is hidden added
Updated by Gerrit Code Review 9 months ago
Patch set 5 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/79724
Updated by Gerrit Code Review 9 months ago
Patch set 6 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/79724
Updated by Gerrit Code Review 5 months ago
Patch set 7 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/79724
Updated by Gerrit Code Review 5 months ago
Patch set 1 for branch 12.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/81831
Updated by Sybille Peters 5 months ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 67ef7f14939e599ac2df81cb5d51891ef43441ad.