Provide reference to original content node when generating sitemap nodes

[patrickdemooij9]

Discussed in #141

^{Originally posted by ianleeder December 29, 2022}
We would like the ability to hide individual content nodes from the sitemap, not just document types. I've added a flag to my master document type.

I'm looking at the GenerateSitemapNodeNotification, but the event object doesn't provide an easy way to reference the original content node (it includes a URL, but I can't see a ContentService.GetByUrl method), so I can't read my flag.

I've tracked the event source down to the GetSelfAndChildren method in SitemapGenerator, and each sitemap node is created from a content node (IPublishedContent). Would it be feasible (particularly with respect to performance) to include the source IPublishedContent in the SitemapNodeItem? Then in my notification event I could simply write:

notification.Node.HideFromSitemap |= notification.Node.SourceContent.GetPropertyValue<bool>("hideFromSitemap");

This also seems like it would be more performant than having to do a Content lookup on each node (eg by URL or id).

Or am I going about this the wrong way?

[patrickdemooij9]

Fixed in latest release