Avoid throwing an exception on getting references when migrating content with changed data types (16)

[AndyButland]

Forward port of #20079 for Umbraco 16.

See linked PR for details.

[Copilot]

Pull Request Overview

This PR forward ports a fix from Umbraco 15 to prevent exceptions when retrieving references during content migration with changed data types. The changes focus on improving error handling in block editors by adding logging capabilities and avoiding fatal exceptions when deserializing invalid block data.

• Adds logger parameter to block editor constructors to enable proper error logging
• Replaces exception-throwing deserialization with safe parsing that logs warnings
• Improves resilience during content migration scenarios where data format mismatches occur

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
BlockEditorPropertyValueEditor.cs	Main implementation with new constructor overload, logger field, and safe parsing method with warning logs
BlockListPropertyEditorBase.cs	Updates constructor call to pass logger parameter to base class
BlockGridPropertyEditorBase.cs	Updates constructor call to pass logger parameter to base class

[kjac]

@AndyButland I reintroduced the generic catch to avoid a potential functional breaking change here. Other than that it looks good 👍

I'll leave it up to you to merge, in case you have any objections with my change 😄