Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Updated: Comment #0
Problem/Motivation
The 'Delete' operation in the list of image styles is no longer available, after commit of #2167641: EntityInterface::uri() should use route name and not path. Also, the 'Delete' link in the image style edit form is not working (in fact it was not working before either).
Proposed resolution
Add the 'delete-form' link in the annotation of ImageStyle. Also, use the opportunity to convert the 'cancel' link in effect edit to use route_name and route_parameter instead of href.
Remaining tasks
Review patch
User interface changes
API changes
Comment | File | Size | Author |
---|---|---|---|
#3 | image_style_delete-2187335-3.patch | 3.27 KB | mondrake |
#3 | image_style_delete-2187335-3-test_only.patch | 1.6 KB | mondrake |
#1 | image_style_delete-2187335-1.patch | 1.67 KB | mondrake |
Comments
Comment #1
mondrakePatch.
Comment #2
tim.plunkettNice catch, but we should add a test.
Comment #3
mondrakeTest added. Test_only patch should fail, the other is combining patch and test and should be OK. Test_only patch equals the interdiff.
Comment #6
mondrakeComment #7
tim.plunkett@mondrake++
Comment #8
catch