Bug #85328
closedlinkvalidator still checks for tt_news in 8.7 + breaks with certain links
100%
Description
mod.linkvalidator {
searchFields {
pages = media,url
tt_content = bodytext,header_link,records
tt_news = bodytext,links
}
linktypes = db,file,external
checkhidden = 0
showCheckLinkTab = 1
mail {
fromname = Linkvalidator
fromemail = Linkvalidator@example.com
replytoname =
replytoemail =
subject = TYPO3 Linkvalidator report
}
linkhandler.reportHiddenRecords = 0
}
searchFields for linkvalidator still includes tt_news which results in the following error, while using the linkvalidator check in TYPO3 8.7:
Argument 3 passed to TYPO3\CMS\Core\Database\Query\QueryBuilder::select() must be of the type string, null given, called in [...]/typo3/sysext/linkvalidator/Classes/LinkAnalyzer.php on line 189
I think removing the line tt_news = bodytext,links would be OK, because tx_news is adding their own fields automatically.
Furthermore, when scanning links like http://www.chemmedchem.org/ I get this error:
Call to a member function getStatusCode() on null
Error thrown in file
[...]/typo3/sysext/linkvalidator/Classes/Linktype/ExternalLinktype.php in line 93.
16 TYPO3\CMS\Linkvalidator\Linktype\ExternalLinktype::checkLink("http://www.chemmedchem.org/", array, TYPO3\CMS\Linkvalidator\LinkAnalyzer)
Updated by m s over 6 years ago
- Subject changed from linkvalidator still checks for tt_news in 8.7 to linkvalidator still checks for tt_news in 8.7 + breaks with certain links
- Description updated (diff)
Updated by Georg Ringer over 6 years ago
what is strange that there is this check in LinkAnalyzer
:
// If table is not configured, assume the extension is not installed // and therefore no need to check it if (!is_array($GLOBALS['TCA'][$table])) { continue; }
so that should never happen
Updated by Gerrit Code Review over 6 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57350
Updated by Georg Ringer over 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 5daf6d83dda04d06026f765d51b2325a6893583f.
Updated by Gerrit Code Review over 6 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57485
Updated by Georg Ringer over 6 years ago
- Status changed from Under Review to Resolved
Applied in changeset 99fd3dd473fa7ef70988372bd858483607a47dc4.