Performance: Added request cache to media type retrieval in media picker validation (#20405)

* Added request cache to media type retrieval in media picker validation.

* Applied suggestions from code review.

Author: AndyButland

src/Umbraco.Infrastructure/PropertyEditors/MediaPicker3PropertyEditor.cs

@@ -82,7 +82,8 @@ public MediaPicker3PropertyValueEditor(
             IDataTypeConfigurationCache dataTypeReadCache,
             ILocalizedTextService localizedTextService,
             IMediaTypeService mediaTypeService,
-            IMediaNavigationQueryService mediaNavigationQueryService)
+            IMediaNavigationQueryService mediaNavigationQueryService,
+            AppCaches appCaches)
             : base(shortStringHelper, jsonSerializer, ioHelper, attribute)
         {
             _jsonSerializer = jsonSerializer;
@@ -95,7 +96,7 @@ public MediaPicker3PropertyValueEditor(
             var validators = new TypedJsonValidatorRunner<List<MediaWithCropsDto>, MediaPicker3Configuration>(
                 jsonSerializer,
                 new MinMaxValidator(localizedTextService),
-                new AllowedTypeValidator(localizedTextService, mediaTypeService, _mediaService),
+                new AllowedTypeValidator(localizedTextService, mediaTypeService, _mediaService, appCaches),
                 new StartNodeValidator(localizedTextService, mediaNavigationQueryService));
 
             Validators.Add(validators);
@@ -401,18 +402,22 @@ public IEnumerable<ValidationResult> Validate(
         /// </summary>
         internal sealed class AllowedTypeValidator : ITypedJsonValidator<List<MediaWithCropsDto>, MediaPicker3Configuration>
         {
+            private const string MediaTypeCacheKeyFormat = nameof(AllowedTypeValidator) + "_MediaTypeKey_{0}";
+
             private readonly ILocalizedTextService _localizedTextService;
             private readonly IMediaTypeService _mediaTypeService;
             private readonly IMediaService _mediaService;
+            private readonly AppCaches _appCaches;
 
             /// <summary>
             /// Initializes a new instance of the <see cref="AllowedTypeValidator"/> class.
             /// </summary>
-            public AllowedTypeValidator(ILocalizedTextService localizedTextService, IMediaTypeService mediaTypeService, IMediaService mediaService)
+            public AllowedTypeValidator(ILocalizedTextService localizedTextService, IMediaTypeService mediaTypeService, IMediaService mediaService, AppCaches appCaches)
             {
                 _localizedTextService = localizedTextService;
                 _mediaTypeService = mediaTypeService;
                 _mediaService = mediaService;
+                _appCaches = appCaches;
             }
 
             /// <inheritdoc/>
@@ -452,9 +457,20 @@ public IEnumerable<ValidationResult> Validate(
 
                 foreach (var typeAlias in distinctTypeAliases)
                 {
-                    IMediaType? type = _mediaTypeService.Get(typeAlias);
+                    // Cache media type lookups since the same media type is likely to be used multiple times in validation,
+                    // particularly if we have multiple languages and blocks.
+                    var cacheKey = string.Format(MediaTypeCacheKeyFormat, typeAlias);
+                    string? typeKey = _appCaches.RequestCache.GetCacheItem<string?>(cacheKey);
+                    if (typeKey is null)
+                    {
+                        typeKey = _mediaTypeService.Get(typeAlias)?.Key.ToString();
+                        if (typeKey is not null)
+                        {
+                            _appCaches.RequestCache.Set(cacheKey, typeKey);
+                        }
+                    }
 
-                    if (type is null || allowedTypes.Contains(type.Key.ToString()) is false)
+                    if (typeKey is null || allowedTypes.Contains(typeKey) is false)
                     {
                         return
                         [

tests/Umbraco.Tests.UnitTests/Umbraco.Core/PropertyEditors/DataValueReferenceFactoryCollectionTests.cs

@@ -41,7 +41,8 @@ public class DataValueReferenceFactoryCollectionTests
                  Mock.Of<IDataTypeConfigurationCache>(),
                  Mock.Of<ILocalizedTextService>(),
                  Mock.Of<IMediaTypeService>(),
-                 Mock.Of<IMediaNavigationQueryService>()));
+                 Mock.Of<IMediaNavigationQueryService>(),
+                 AppCaches.Disabled));
 
     private IIOHelper IOHelper { get; } = Mock.Of<IIOHelper>();
 

tests/Umbraco.Tests.UnitTests/Umbraco.Core/PropertyEditors/MediaPicker3ValueEditorValidationTests.cs

@@ -218,7 +218,8 @@ private static (MediaPicker3PropertyEditor.MediaPicker3PropertyValueEditor Value
             Mock.Of<IDataTypeConfigurationCache>(),
             Mock.Of<ILocalizedTextService>(),
             mediaTypeServiceMock.Object,
-            mediaNavigationQueryServiceMock.Object)
+            mediaNavigationQueryServiceMock.Object,
+            AppCaches.Disabled)
         {
             ConfigurationObject = new MediaPicker3Configuration()
         };