I appologize if one of the other image style related issues covers this, they seem unrelated.

To reproduce on a clean install:

1) Create a feature with a new image style exported into it.
2) Override in the UI
3) Create a new override feature containing the overridden image style
4) drush cc all
5) drush fl

The original feature will still be overridden and the diff contains the changes.

This is using core 7.20 and Features 7.x-1.0, but the same results with 7.x-2.0-beta1.

CommentFileSizeAuthor
#5 reset_effect_keys-1950614-5.patch721 bytesmikran
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tbenice’s picture

Priority: Normal » Major

So does nobody care that image style overrides cannot be exported with this module? Seems important to me.

preksha’s picture

I am facing the same issue... Image Style overrides is not working... It's important for me to use it.

jduhls’s picture

Same issue. Please fix.

mikran’s picture

Issue summary: View changes

I've found a workaround. If you set weight of image style effect to 0 instead of 1 it somehow works. My use case only has 1 effect so I don't know if this works with multiple effects.

mikran’s picture

Status: Active » Needs review
FileSize
721 bytes

And here is my attempt at fixing this. image_features_override_component_overrides_alter() resets the keys before overrides are exported so this patch exports the same reset mechanism to export itself.

segovia94’s picture

Patch in #5 is working for me so far.

kehan’s picture

Status: Needs review » Reviewed & tested by the community

works for me to

BarisW’s picture

Works great. Please commit.

  • mpotter committed eaf4e56 on 7.x-2.x authored by mikran
    Issue #1950614 by mikran: Image style overrides remain overridden in the...
mpotter’s picture

Status: Reviewed & tested by the community » Fixed

Committed in eaf4e56.

Status: Fixed » Closed (fixed)

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