Problem/Motivation

Had an error in my cronjob that was caused by not checking if the node exists before doing anything with it.

Proposed resolution

Checks if it loaded before it does anything with it.

Remaining tasks

* Add documentation about it sending a watchdog notice saying which node failed to load.

CommentFileSizeAuthor
#2 xmlsitemap-check-if-loaded.patch798 byteschristiaandev
xmlsitemap-node-load-fix.patch922 byteschristiaandev

Comments

ChristiaanDev created an issue. See original summary.

christiaandev’s picture

christiaandev commented
StatusFileSize
new xmlsitemap-check-if-loaded.patch798 bytes
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Assigned: christiaandev » Unassigned
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Title: Add check if node exists before creating and saving xmlsitemap link. » Verify the node exists before creating and saving the xmlsitemap link
poker10’s picture

poker10 commented
Status: Active » Closed (duplicate)

This should be fixed by #2986847: TypeError: Argument 1 passed to xmlsitemap_node_create_link() must be an instance of stdClass, boolean given