Drupal 11 compatibility fixes for sitemap

Hi @naveenvalecha,

I have fixed this "Drupal 11 compatibility fixes for sitemap" and also attached patch ,
please review and verify .

@sarwan_verma
Thank you for your contribution. Please create a Merge request https://www.drupal.org/docs/develop/git/using-gitlab-to-contribute-to-dr... →

Hi@naveenvalecha,

I have created MR. please review it once.

Applied the patch in #3 and the MR in #5, both are applied successfully. But on visiting the config page/admin/config/search/sitemap getting the below error.

Moving this to NW.

MR and patch applied successfully no errors found attaching ss.

@chaitanyadessai It seems you are testing it against D10. However the issue is for D11.

I've applied the MR as a patch and the patch applied with no errors. After applying the patch the module is now compatible with Drupal 11 and it's working fine. Getting no error on visiting the module config page /admin/config/search/sitemap. Attaching the screenshots of the module config and sitemap page for reference.

Updated the issue summary.

The code in the merge request had several PHPCS and PHPStan errors; and none of the changes accounted for the Book module moving into contrib → in Drupal 11, so I had to make extensive changes to the merge request to account for that (via a new sitemap_book sub-module).

I also need to file a follow-up issue, because I couldn't figure out how to move the book-plugin-specific config into the new sitemap_book sub-module, so I'll do that before merging this.

Going to mark this as fixed; I'll create a new release after resolving some other issues.

Note that this change was released in sitemap-8.x-2.0-beta7 → , but resulted in a follow-up issue, 🐛 Error when running sitemap_update_8202 Fixed , which was released in sitemap-8.x-2.0-beta8 → .

Automatically closed - issue fixed for 2 weeks with no activity.