From #1459614: Better UI for removing / changing menu icons: Using image styles means that when the original image changes, the derivatives need to be deleted so they can be re-generated on-demand. (This would be less of a problem if the new and old files did not have the same name.)

Comments

acrollet’s picture

acrollet commented
Assigned: Unassigned » acrollet
acrollet’s picture

acrollet commented
Title: Flush image caches when images are changed » Correctly generate imagecache derivatives
Status: Active » Needs review

Upon further investigation, I did not find a problem with the derivatives being flushed. There was, however, a problem with them getting generated in the first place, which I've fixed in http://drupalcode.org/project/menu_icons.git/commit/11c9278 -- If you're able to test and verify that this fixes your issue, it would be much appreciated it. At that point, I think we'll be ready for a 3rd (and hopefully final) beta before releasing a 1.0.

Anonymous’s picture

Anonymous (not verified) commented

The patch seems to work. I noticed the image url on the menu item edit form is now reading public://path and the css is reading the full url, which makes sense - before it was the opposite. I was able to add a new menu icon and then replace the image without any issues.

acrollet’s picture

acrollet commented
Status: Needs review » Fixed

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