Skip to content
This repository has been archived by the owner on Feb 2, 2023. It is now read-only.

ASNetworkImageNode can NOT load animatedData sometime #2871

Closed
IvanChan opened this issue Jan 7, 2017 · 5 comments
Closed

ASNetworkImageNode can NOT load animatedData sometime #2871

IvanChan opened this issue Jan 7, 2017 · 5 comments

Comments

@IvanChan
Copy link

IvanChan commented Jan 7, 2017

[ASDK 2.0.1]

I realized ASImageDownloaderProtocol & ASImageCacheProtocol for ASNetworkImageNode

static image works well

load gif data sometimes just got a blank imageNode for display

the api below is called normally

- (nullable id <ASAnimatedImageProtocol>)animatedImageWithData:(NSData *)animatedImageData
{
    return (id <ASAnimatedImageProtocol>)[[PINAnimatedImage alloc] initWithAnimatedImageData:animatedImageData];
}

@IvanChan IvanChan changed the title ASNetworkImageNode can NOT load animatedData from cache ASNetworkImageNode can NOT load animatedData sometime Jan 7, 2017
@hannahmbanana
Copy link
Contributor

@IvanChan - are you implementing a custom image downloader? I don't think we have tested this feature with anything except PINRemoteImage (what we use at Pinterest). Is there any reason you aren't able to use PINRemoteImage?

Without a example project that includes the downloader you are using, realistically we won't be able to prioritize this issue since 95% of our users are on PINRemoteImage. I suggest you use FLAnimatedImage as a workaround.

@IvanChan
Copy link
Author

IvanChan commented Jan 13, 2017

YES, I have a lot of stat for image download, so I implementing a custom image downloader with SDWebImage.

But I set breakpoint for downloader callback, the data is indeed gif-data, and passed to animatedImageWithData , I currently work around with some other logic to play gif instead of passing data to ASNetworkImageNode.

I can post a sample project later , sorry for recently busy.

By the way, It's not friendly for user to manage cache for ASNetworkImageNode with default downloader?

@garrettmoon
Copy link
Contributor

@IvanChan GIFs are currently broken in ASDK master sadly, it's fixed in PINRemoteImage master though. Can you check to see if that fixes the issue for you?

@ay8s
Copy link
Contributor

ay8s commented Feb 13, 2017

@garrettmoon Looks like it's fixed with the latest master PINRemoteImage for us. 👍

@garrettmoon
Copy link
Contributor

Going to close this out, yay!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants