Updated: Comment #0
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).
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.
Review patch
| 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