Item with ID cannot be deleted because it is not found in Sitecore 9.2+

[markgibbons25]

We are getting our logs spammed with messages like

WARN Item with ID={C100A1CF-6205-4F05-A636-0831DBADE1D0} cannot be deleted because it is not found

I traced it back to here

Glass.Mapper/Source/Glass.Mapper.Sc/RenderingParametersModelFactory.cs

Line 50 in 2bc3e94

item.Delete(); //added for clean up

I added a log statement

Sitecore.Diagnostics.Log.Info("GLASS DELETE?", this); item.Delete(); //added for clean up

And confirmed logs:

17368 12:51:06 INFO GLASS DELETE?
17368 12:51:06 WARN Item with ID={C100A1CF-6205-4F05-A636-0831DBADE1D0} cannot be deleted because it is not found
17368 12:51:06 INFO GLASS DELETE?
17368 12:51:06 WARN Item with ID={88CC0F4C-EB54-4013-8228-83E558CF35AC} cannot be deleted because it is not found
17368 12:51:06 INFO GLASS DELETE?
17368 12:51:06 WARN Item with ID={B82C7AC0-46BD-4CD5-9D58-E4D9627646E6} cannot be deleted because it is not found
17368 12:51:07 INFO GLASS DELETE?

It looks like Sitecore in 9.2+ has changed that Delete() to log if the item is not found.
Looks like in this case it is an in memory item only.
Is it safe to just remove that item.Delete() line and let the GC do its thing?
Happy to open a PR if so.

[swissworx]

@mikeedwards83 any thoughts on @markgibbons25 suggestion? We are facing the same problem and this is filling up our prod trace logs massively. Thanks for your help!

[mikeedwards83]

@swissworx Sorry for not responding quicker.

This is indeed annoying, my only worry is that the item just hangs around in memory and cause a memory leak if delete isn't called. Is there a way to tell if the item is removed from memory without the delete?

[markgibbons25]

It looks to me like it is going to get cleaned up by the GC just fine. Which leads me to think that we could maybe improve performance here by pooling and reusing these fake items to save the GC some work.
I'll do some benchmarking...

[simonethreechillies]

Hi @markgibbons25

We've seen the same behaviour in one of our projects. When could we expect this update to be published on NuGet?

Thanks!

[markgibbons25]

Hi @markgibbons25

We've seen the same behaviour in one of our projects. When could we expect this update to be published on NuGet?

Thanks!

I would love to but it's up to @mikeedwards83

[swissworx]

@mikeedwards83 it looks fine to me, I quickly cracked the Sitecore dll open and I couldn't find anything being allocated that is not cleaned up by the GC.

[Glass-Build-Github]

Thanks, I will make the change later today and push a beta nuget out.

[mikeedwards83]

I have pushed a beta release with this fix, it also contains a flag in the config for backward compatibility. By default it will no longer delete the item, but setting the flag will restore the original logic.

74ff5e8#diff-112f7122b703ee4f85649982cc0c249eR74