Issue #2573221 by Jelle_S: Keyed mapping static cache is not rebuilt when...

Issue #2573221 by Jelle_S: Keyed mapping static cache is not rebuilt when overwriting an existing mapping

core/modules/responsive_image/src/Entity/ResponsiveImageStyle.php

@@ -112,6 +112,7 @@ public function addImageStyleMapping($breakpoint_id, $multiplier, array $image_s
           'breakpoint_id' => $breakpoint_id,
           'multiplier' => $multiplier,
         ) + $image_style_mapping;
+        $this->keyedImageStyleMappings = NULL;
         return $this;
       }
     }

core/modules/responsive_image/tests/src/Unit/ResponsiveImageStyleConfigEntityUnitTest.php

@@ -252,6 +252,19 @@ public function testGetKeyedImageStyleMappings() {
       'image_mapping' => 'medium',
     );
     $this->assertEquals($expected, $entity->getKeyedImageStyleMappings());
+
+    // Overwrite a mapping to ensure keyed mapping static cache is rebuilt.
+    $entity->addImageStyleMapping('test_breakpoint2', '2x', array(
+      'image_mapping_type' => 'image_style',
+      'image_mapping' => 'large',
+    ));
+    $expected['test_breakpoint2']['2x'] = array(
+      'breakpoint_id' => 'test_breakpoint2',
+      'multiplier' => '2x',
+      'image_mapping_type' => 'image_style',
+      'image_mapping' => 'large',
+    );
+    $this->assertEquals($expected, $entity->getKeyedImageStyleMappings());
   }
 
   /**