diff --git a/core/modules/picture/lib/Drupal/picture/PictureMappingFormController.php b/core/modules/picture/lib/Drupal/picture/PictureMappingFormController.php index 4b93dc548a6..3c3ce344beb 100644 --- a/core/modules/picture/lib/Drupal/picture/PictureMappingFormController.php +++ b/core/modules/picture/lib/Drupal/picture/PictureMappingFormController.php @@ -144,7 +144,7 @@ class PictureMappingFormController extends EntityFormController { // another breakpoint group. if (!$picture_mapping->hasMappings()) { $uri = $picture_mapping->uri(); - $form_state['redirect'] = $uri['path'] . '/edit'; + $form_state['redirect'] = $uri['path']; } else { $form_state['redirect'] = 'admin/config/media/picturemapping'; diff --git a/core/modules/picture/lib/Drupal/picture/Tests/PictureAdminUITest.php b/core/modules/picture/lib/Drupal/picture/Tests/PictureAdminUITest.php index 114e30cc560..e61cac6c036 100644 --- a/core/modules/picture/lib/Drupal/picture/Tests/PictureAdminUITest.php +++ b/core/modules/picture/lib/Drupal/picture/Tests/PictureAdminUITest.php @@ -102,7 +102,7 @@ class PictureAdminUITest extends WebTestBase { $this->assertText('mapping_one'); // Edit the group. - $this->drupalGet('admin/config/media/picturemapping/mapping_one/edit'); + $this->drupalGet('admin/config/media/picturemapping/mapping_one'); $this->assertFieldByName('label', 'Mapping One'); $this->assertFieldByName('breakpointGroup', 'atestset'); @@ -122,8 +122,8 @@ class PictureAdminUITest extends WebTestBase { 'mappings[custom.user.medium][1x]' => 'medium', 'mappings[custom.user.large][1x]' => 'large', ); - $this->drupalPost('admin/config/media/picturemapping/mapping_one/edit', $edit, t('Save')); - $this->drupalGet('admin/config/media/picturemapping/mapping_one/edit'); + $this->drupalPost('admin/config/media/picturemapping/mapping_one', $edit, t('Save')); + $this->drupalGet('admin/config/media/picturemapping/mapping_one'); $this->assertFieldByName('mappings[custom.user.small][1x]', 'thumbnail'); $this->assertFieldByName('mappings[custom.user.small][2x]', ''); $this->assertFieldByName('mappings[custom.user.medium][1x]', 'medium'); diff --git a/core/modules/picture/picture.module b/core/modules/picture/picture.module index 680f05c9302..6863bfb58af 100644 --- a/core/modules/picture/picture.module +++ b/core/modules/picture/picture.module @@ -64,7 +64,7 @@ function picture_menu() { 'route_name' => 'picture_mapping_page_add', 'type' => MENU_LOCAL_ACTION, ); - $items['admin/config/media/picturemapping/%picture_mapping/edit'] = array( + $items['admin/config/media/picturemapping/%picture_mapping'] = array( 'title' => 'Edit picture mapping', 'route_name' => 'picture_mapping_page_edit', ); diff --git a/core/modules/picture/picture.routing.yml b/core/modules/picture/picture.routing.yml index c9530291f4b..e01bb81049a 100644 --- a/core/modules/picture/picture.routing.yml +++ b/core/modules/picture/picture.routing.yml @@ -13,7 +13,7 @@ picture_mapping_page_add: _permission: 'administer pictures' picture_mapping_page_edit: - pattern: '/admin/config/media/picturemapping/{picture_mapping}/edit' + pattern: '/admin/config/media/picturemapping/{picture_mapping}' defaults: _entity_form: 'picture_mapping.edit' requirements: