Simplifying attribute value cache

OrchardCMS · Dec 7, 2024 · 3ca230d · 3ca230d
1 parent 5185ed0
commit 3ca230d
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/src/Orchard.Web/Modules/Orchard.MediaProcessing/Filters/MediaProcessingHtmlFilter.cs b/src/Orchard.Web/Modules/Orchard.MediaProcessing/Filters/MediaProcessingHtmlFilter.cs
@@ -25,8 +25,7 @@ public class MediaProcessingHtmlFilter : IHtmlFilter {
         private MediaHtmlFilterSettingsPart _settingsPart;
         private static readonly Regex _imageTagRegex = new Regex(@"<img\b[^>]*>", RegexOptions.IgnoreCase | RegexOptions.Compiled);
         private static readonly ConcurrentDictionary<string, Regex> _attributeRegexes = new ConcurrentDictionary<string, Regex>();
-        private static readonly ConcurrentDictionary<string, ConcurrentDictionary<string, string>> _attributeValues =
-            new ConcurrentDictionary<string, ConcurrentDictionary<string, string>>();
+        private static readonly ConcurrentDictionary<string, string> _attributeValues = new ConcurrentDictionary<string, string>();
         private static readonly Dictionary<string, string> _validExtensions = new Dictionary<string, string> {
             { ".jpeg", "jpg" }, // For example: .jpeg supports compression (quality), format to 'jpg'.
             { ".jpg", "jpg"  },
@@ -177,8 +176,7 @@ private string ProcessImageAltContent(string imgTag) {
 
         private string GetAttributeValue(string tag, string attributeName) =>
             _attributeValues
-                .GetOrAdd(tag, _ => new ConcurrentDictionary<string, string>())
-                .GetOrAdd(attributeName, _ => {
+                .GetOrAdd($"{tag}_{attributeName}", _ => {
                     var match = GetAttributeRegex(attributeName).Match(tag);
                     return match.Success ? match.Groups[1].Value : null;
                 });