Part of #2285413: [Meta] Standardize entity route names. See there for details.

Holy crap this is an entity? :-)

CommentFileSizeAuthor
route-names-responsiveimagemapping.patch3.98 KBCrell
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community 
+++ b/core/modules/responsive_image/src/ResponsiveImageMappingForm.php
@@ -132,7 +132,7 @@ public function save(array $form, FormStateInterface $form_state) {
+        'route_name' => 'entity.responsive_image_mapping.edit_form',
         'route_parameters' => array(
           'responsive_image_mapping' => $responsive_image_mapping->id(),
         ),

This especially makes it easier to get the right parameter

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

  • webchick committed 1bdb496 on 8.0.x 
    Issue #2314881 by Crell: Standardize responsive image mapping entity...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.