Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use cache key filter when scaling an image after download #529

Closed
wants to merge 1 commit into from

Conversation

Tylerc230
Copy link

We were using the raw url as the key when trying to determine if an
image was an @2x image or not. This was a problem for me because I was
pulling different sized image from a server which did not append the @2x
to the larger images. I want to the cache key filter to append @2x to
the cache key.

We were using the raw url as the key when trying to determine if an
image was an @2x image or not. This was a problem for me because I was
pulling different sized image from a server which did not append the @2x
to the larger images. I want to the cache key filter to append @2x to
the cache key.
@dzenbot
Copy link

dzenbot commented Nov 8, 2013

+1! Please merge.

@Fl0p
Copy link

Fl0p commented Dec 3, 2013

👍

VoteUp for this

semi fix for #569

@bpoplauschi bpoplauschi added this to the 3.7.0 milestone Jun 15, 2014
bpoplauschi added a commit that referenced this pull request Jun 19, 2014
…when computing the `scaledImageForKey`. For most of the developers that did not set `cacheKeyFilter`, the code will work exactly the same
@bpoplauschi
Copy link
Member

Replaced by commit eb91fdd. The pull request was not mergeable. Thanks @Tylerc230

devedup pushed a commit to FilmFlexMovies/SDWebImage that referenced this pull request Sep 10, 2014
…s ignored when computing the `scaledImageForKey`. For most of the developers that did not set `cacheKeyFilter`, the code will work exactly the same
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants