- πΊπΈUnited States aitala
Hi,
Patch seems to work...
No idea why the error was occurring - Is a new Drupal 9 site...
Eric
- π¦πΊAustralia timfletcher
I tried Patch #11 after running into this π Argument #1 ($link) must be of type Drupal\linkchecker\LinkCheckerLinkInterface, null given Needs review on the 2.x D10 version, which suppresses the error but still WSODs the page. If the cause of the error isn't the cause of the WSOD, I have no idea what may be causing it.
- Status changed to Needs review
about 1 year ago 11:50am 25 April 2024 - last update
about 1 year ago 87 pass - πΈπ°Slovakia kaszarobert
I updated the patch with proper array index checking, method argument type checking and catching Throwable to not produce WSOD if Exception or Error is thrown.
- π¬π§United Kingdom alt36
I have a site which is currently throwing this error every time cron runs, and have been trying to figure out reproducible steps without success. However, I am reasonably confident about the following.
I have linkchecker configured to update redirected links after three checks (error.action_status_code_301 = 3)
I have two nodes which contain the same link (identical urlhash value in linkchecker_link table, lid=1643 and 1720, different entity_id__target_id values). The link returns 301 and both rows in linkchecker_link have fail_count=2. The queue table has a couple of hundred rows with name=linkchecker_status_handle. I've made a clone of my Drupal installation to debug and have manually executed "truncate queue".
I run "drush cron". Manually watching the queue table I see many linkchecker_status_handle being created. The "TypeError .. null given" error is then thrown. The queue entry with the smallest item_id has data containing "s:5:"links";a:2:{i:1643;s:4:"1643";i:1720;s:4:"1720";}", i.e it will deserialize to links with lid 1643 and 1720. However, linkchecker_link no longer has the row with lid=1643 but the row for lid=1720 is still present. This presumably is what leads to the error: LinkStatusHandle::processItem() loads a null $link which gets passed unchecked to $hanlder->handle()
- π¬π§United Kingdom joehuggans Harrogate, UK
Tested patch #19 and seems to be working
- Status changed to RTBC
13 days ago 9:31am 30 June 2025 - π¦πΊAustralia VladimirAus Brisbane, Australia
Thank you for contributing and testing.
RTBC - π¦πΊAustralia VladimirAus Brisbane, Australia
Added changes from patch #19 to MR.
- π¦πΊAustralia VladimirAus Brisbane, Australia
Great work everyone.
I no longer have permissions to merge. π€·ββοΈ
Waiting for someone with permissions. -
vladimiraus β
committed d0e562cf on 2.0.x authored by
eiriksm β
Issue #3247070 by vladimiraus, eiriksm, szeberli, kaszarobert, wrd-...
-
vladimiraus β
committed d0e562cf on 2.0.x authored by
eiriksm β
- π¦πΊAustralia VladimirAus Brisbane, Australia
Thanks everyone. π§
Merged and released. -
vladimiraus β
committed 87cc476f on 2.1.x
Issue #3247070 by vladimiraus, eiriksm, szeberli, kaszarobert, wrd-...
-
vladimiraus β
committed 87cc476f on 2.1.x