diff --git a/core/modules/image/image.admin.inc b/core/modules/image/image.admin.inc index 851b009..4e28881 100644 --- a/core/modules/image/image.admin.inc +++ b/core/modules/image/image.admin.inc @@ -87,12 +87,12 @@ function image_style_form($form, &$form_state, $style) { if (isset($effect['form callback'])) { $links['edit'] = array( 'title' => t('edit'), - 'href' => 'admin/config/media/image-styles/edit/' . $style->id() . '/effects/' . $key, + 'href' => 'admin/config/media/image-styles/manage/' . $style->id() . '/effects/' . $key, ); } $links['delete'] = array( 'title' => t('delete'), - 'href' => 'admin/config/media/image-styles/edit/' . $style->id() . '/effects/' . $key . '/delete', + 'href' => 'admin/config/media/image-styles/manage/' . $style->id() . '/effects/' . $key . '/delete', ); $form['effects'][$key]['operations'] = array( '#type' => 'operations', @@ -101,13 +101,13 @@ function image_style_form($form, &$form_state, $style) { $form['effects'][$key]['configure'] = array( '#type' => 'link', '#title' => t('edit'), - '#href' => 'admin/config/media/image-styles/edit/' . $style->id() . '/effects/' . $key, + '#href' => 'admin/config/media/image-styles/manage/' . $style->id() . '/effects/' . $key, '#access' => isset($effect['form callback']), ); $form['effects'][$key]['remove'] = array( '#type' => 'link', '#title' => t('delete'), - '#href' => 'admin/config/media/image-styles/edit/' . $style->id() . '/effects/' . $key . '/delete', + '#href' => 'admin/config/media/image-styles/manage/' . $style->id() . '/effects/' . $key . '/delete', ); } } @@ -170,7 +170,7 @@ function image_style_form_add_submit($form, &$form_state) { // Load the configuration form for this option. if (isset($effect['form callback'])) { - $path = 'admin/config/media/image-styles/edit/' . $style->id() . '/add/' . $form_state['values']['new']; + $path = 'admin/config/media/image-styles/manage/' . $style->id() . '/add/' . $form_state['values']['new']; $form_state['redirect'] = array($path, array('query' => array('weight' => $form_state['values']['weight']))); } // If there's no form, immediately add the image effect. @@ -213,7 +213,7 @@ function image_style_form_submit($form, &$form_state) { if ($status == SAVED_UPDATED) { drupal_set_message(t('Changes to the style have been saved.')); } - $form_state['redirect'] = 'admin/config/media/image-styles/edit/' . $style->id(); + $form_state['redirect'] = 'admin/config/media/image-styles/manage/' . $style->id() . '/edit'; } /** @@ -256,7 +256,7 @@ function image_style_add_form_submit($form, &$form_state) { )); $style->save(); drupal_set_message(t('Style %name was created.', array('%name' => $style->label()))); - $form_state['redirect'] = 'admin/config/media/image-styles/edit/' . $style->id(); + $form_state['redirect'] = 'admin/config/media/image-styles/manage/' . $style->id() . '/edit'; } /** @@ -324,7 +324,7 @@ function image_effect_form($form, &$form_state, $style, $effect) { // If there's no configuration for this image effect, return to // the image style page. if (!isset($effect['form callback'])) { - drupal_goto('admin/config/media/image-styles/edit/' . $style->id()); + drupal_goto('admin/config/media/image-styles/manage/' . $style->id() . '/edit'); } $form_state['image_style'] = $style; $form_state['image_effect'] = $effect; @@ -365,7 +365,7 @@ function image_effect_form($form, &$form_state, $style, $effect) { $form['actions']['cancel'] = array( '#type' => 'link', '#title' => t('Cancel'), - '#href' => 'admin/config/media/image-styles/edit/' . $style->id(), + '#href' => 'admin/config/media/image-styles/manage/' . $style->id() . '/edit', ); return $form; @@ -382,7 +382,7 @@ function image_effect_form_submit($form, &$form_state) { image_effect_save($style, $effect); drupal_set_message(t('The image effect was successfully applied.')); - $form_state['redirect'] = 'admin/config/media/image-styles/edit/' . $style->id(); + $form_state['redirect'] = 'admin/config/media/image-styles/manage/' . $style->id() . '/edit'; } /** @@ -400,7 +400,7 @@ function image_effect_delete_form($form, &$form_state, $style, $effect) { $form_state['image_effect'] = $effect; $question = t('Are you sure you want to delete the @effect effect from the %style style?', array('%style' => $style->label(), '@effect' => $effect['label'])); - return confirm_form($form, $question, 'admin/config/media/image-styles/edit/' . $style->id(), '', t('Delete')); + return confirm_form($form, $question, 'admin/config/media/image-styles/manage/' . $style->id() . '/edit', '', t('Delete')); } /** @@ -412,7 +412,7 @@ function image_effect_delete_form_submit($form, &$form_state) { image_effect_delete($style, $effect); drupal_set_message(t('The image effect %name has been deleted.', array('%name' => $effect['label']))); - $form_state['redirect'] = 'admin/config/media/image-styles/edit/' . $style->id(); + $form_state['redirect'] = 'admin/config/media/image-styles/manage/' . $style->id() . '/edit'; } /** diff --git a/core/modules/image/image.module b/core/modules/image/image.module index b9ad53c..b49ac78 100644 --- a/core/modules/image/image.module +++ b/core/modules/image/image.module @@ -62,10 +62,10 @@ function image_help($path, $arg) { return $output; case 'admin/config/media/image-styles': return '

' . t('Image styles commonly provide thumbnail sizes by scaling and cropping images, but can also add various effects before an image is displayed. When an image is displayed with a style, a new file is created and the original image is left unchanged.') . '

'; - case 'admin/config/media/image-styles/edit/%/add/%': + case 'admin/config/media/image-styles/manage/%/add/%': $effect = image_effect_definition_load($arg[7]); return isset($effect['help']) ? ('

' . $effect['help'] . '

') : NULL; - case 'admin/config/media/image-styles/edit/%/effects/%': + case 'admin/config/media/image-styles/manage/%/effects/%': $effect = ($arg[5] == 'add') ? image_effect_definition_load($arg[6]) : image_effect_load($arg[6], $arg[4]); return isset($effect['help']) ? ('

' . $effect['help'] . '

') : NULL; } @@ -134,7 +134,7 @@ function image_menu() { 'weight' => 2, 'file' => 'image.admin.inc', ); - $items['admin/config/media/image-styles/edit/%image_style'] = array( + $items['admin/config/media/image-styles/manage/%image_style/edit'] = array( 'title' => 'Edit style', 'description' => 'Configure an image style.', 'page callback' => 'drupal_get_form', @@ -142,7 +142,7 @@ function image_menu() { 'access arguments' => array('administer image styles'), 'file' => 'image.admin.inc', ); - $items['admin/config/media/image-styles/delete/%image_style'] = array( + $items['admin/config/media/image-styles/manage/%image_style/delete'] = array( 'title' => 'Delete style', 'description' => 'Delete an image style.', 'load arguments' => array(NULL, (string) IMAGE_STORAGE_NORMAL), @@ -151,7 +151,7 @@ function image_menu() { 'access arguments' => array('administer image styles'), 'file' => 'image.admin.inc', ); - $items['admin/config/media/image-styles/edit/%image_style/effects/%image_effect'] = array( + $items['admin/config/media/image-styles/manage/%image_style/effects/%image_effect'] = array( 'title' => 'Edit image effect', 'description' => 'Edit an existing effect within a style.', 'load arguments' => array(5, (string) IMAGE_STORAGE_EDITABLE), @@ -160,7 +160,7 @@ function image_menu() { 'access arguments' => array('administer image styles'), 'file' => 'image.admin.inc', ); - $items['admin/config/media/image-styles/edit/%image_style/effects/%image_effect/delete'] = array( + $items['admin/config/media/image-styles/manage/%image_style/effects/%image_effect/delete'] = array( 'title' => 'Delete image effect', 'description' => 'Delete an existing effect from a style.', 'load arguments' => array(5, (string) IMAGE_STORAGE_EDITABLE), @@ -169,7 +169,7 @@ function image_menu() { 'access arguments' => array('administer image styles'), 'file' => 'image.admin.inc', ); - $items['admin/config/media/image-styles/edit/%image_style/add/%image_effect_definition'] = array( + $items['admin/config/media/image-styles/manage/%image_style/add/%image_effect_definition'] = array( 'title' => 'Add image effect', 'description' => 'Add a new effect to a style.', 'load arguments' => array(5), @@ -201,9 +201,6 @@ function image_theme() { ), // Theme functions in image.admin.inc. - 'image_style_list' => array( - 'variables' => array('styles' => NULL), - ), 'image_style_effects' => array( 'render element' => 'form', ), diff --git a/core/modules/image/lib/Drupal/image/Plugin/Core/Entity/ImageStyle.php b/core/modules/image/lib/Drupal/image/Plugin/Core/Entity/ImageStyle.php index eda9f5c..d820050 100644 --- a/core/modules/image/lib/Drupal/image/Plugin/Core/Entity/ImageStyle.php +++ b/core/modules/image/lib/Drupal/image/Plugin/Core/Entity/ImageStyle.php @@ -2,7 +2,7 @@ /** * @file - * Definition of Drupal\image\Plugin\Core\Entity\ImageStyle. + * Contains \Drupal\image\Plugin\Core\Entity\ImageStyle. */ namespace Drupal\image\Plugin\Core\Entity; diff --git a/core/modules/image/lib/Drupal/image/Tests/ImageAdminStylesTest.php b/core/modules/image/lib/Drupal/image/Tests/ImageAdminStylesTest.php index 6461317..d3dc442 100644 --- a/core/modules/image/lib/Drupal/image/Tests/ImageAdminStylesTest.php +++ b/core/modules/image/lib/Drupal/image/Tests/ImageAdminStylesTest.php @@ -68,7 +68,7 @@ function testStyle() { // Setup a style to be created and effects to add to it. $style_name = strtolower($this->randomName(10)); $style_label = $this->randomString(); - $style_path = 'admin/config/media/image-styles/edit/' . $style_name; + $style_path = 'admin/config/media/image-styles/manage/' . $style_name; $effect_edits = array( 'image_resize' => array( 'data[width]' => 100, @@ -112,7 +112,7 @@ function testStyle() { // Add each sample effect to the style. foreach ($effect_edits as $effect => $edit) { // Add the effect. - $this->drupalPost($style_path, array('new' => $effect), t('Add')); + $this->drupalPost($style_path . '/edit', array('new' => $effect), t('Add')); if (!empty($edit)) { $this->drupalPost(NULL, $edit, t('Add effect')); } @@ -179,13 +179,13 @@ function testStyle() { $image_path = $this->createSampleImage($style); $this->assertEqual($this->getImageCount($style), 1, format_string('Image style %style image %file successfully generated.', array('%style' => $style->label(), '%file' => $image_path))); - $this->drupalPost($style_path, $edit, t('Update style')); + $this->drupalPost($style_path . '/edit', $edit, t('Update style')); // Note that after changing the style name, the style path is changed. - $style_path = 'admin/config/media/image-styles/edit/' . $style_name; + $style_path = 'admin/config/media/image-styles/manage/' . $style_name; // Check that the URL was updated. - $this->drupalGet($style_path); + $this->drupalGet($style_path . '/edit'); $this->assertResponse(200, format_string('Image style %original renamed to %new', array('%original' => $style->label(), '%new' => $style_name))); // Check that the image was flushed after updating the style. @@ -233,7 +233,7 @@ function testStyle() { // Style deletion form. // Delete the style. - $this->drupalPost('admin/config/media/image-styles/delete/' . $style_name, array(), t('Delete')); + $this->drupalPost($style_path . '/delete', array(), t('Delete')); // Confirm the style directory has been removed. $directory = file_default_scheme() . '://styles/' . $style_name; @@ -252,7 +252,7 @@ function testStyleReplacement() { $style_label = $this->randomString(); $style = entity_create('image_style', array('name' => $style_name, 'label' => $style_label)); $style->save(); - $style_path = 'admin/config/media/image-styles/edit/' . $style_name; + $style_path = 'admin/config/media/image-styles/manage/'; // Create an image field that uses the new style. $field_name = strtolower($this->randomName(10)); @@ -280,7 +280,7 @@ function testStyleReplacement() { 'name' => $new_style_name, 'label' => $new_style_label, ); - $this->drupalPost('admin/config/media/image-styles/edit/' . $style_name, $edit, t('Update style')); + $this->drupalPost($style_path . $style_name . '/edit', $edit, t('Update style')); $this->assertText(t('Changes to the style have been saved.'), format_string('Style %name was renamed to %new_name.', array('%name' => $style_name, '%new_name' => $new_style_name))); $this->drupalGet('node/' . $nid); $this->assertRaw(image_style_url($new_style_name, file_load($node->{$field_name}[LANGUAGE_NOT_SPECIFIED][0]['fid'])->uri), 'Image displayed using style replacement style.'); @@ -289,7 +289,7 @@ function testStyleReplacement() { $edit = array( 'replacement' => 'thumbnail', ); - $this->drupalPost('admin/config/media/image-styles/delete/' . $new_style_name, $edit, t('Delete')); + $this->drupalPost($style_path . $new_style_name . '/delete', $edit, t('Delete')); $message = t('Style %name was deleted.', array('%name' => $new_style_label)); $this->assertRaw($message);