Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Change record: https://www.drupal.org/node/2270941
Comment | File | Size | Author |
---|---|---|---|
#3 | xmlsitemap-watchdog_is_now_logging-2455217-3.patch | 7.22 KB | camoa |
#1 | xmlsitemap-watchdog-2455217-1.patch | 1.78 KB | Michelle |
Comments
Comment #1
MichelleA couple caveats:
1) I was not able to test this change because I kept running into other errors while trying to run it a second time. Rather than just sit on the fix, I decided to make a patch of it and will comment when I am able to test it (unless someone beats me to it).
2) There are watchdog references in the tests and in the submodules that I have not addressed in this patch.
Comment #2
tadityar CreditAttribution: tadityar commentedThank you for noticing and working on this!
After searching for "watchdog(" I found other occurrences of the function. Might want to change them all in this issue, so I'm setting this to Needs Work.
Comment #3
camoa CreditAttribution: camoa at Camo Advanced Tech commentedDid a quick replacement of the watchdog calls, needs testing.
Comment #4
swentel CreditAttribution: swentel commentedstill applies, and looks good.
Comment #6
Dave ReidCommitted #1 to 8.x-1.x.