nicklockwood / AsyncImageView

[DEPRECATED]
http://charcoaldesign.co.uk/source/cocoa#asyncimageview
Other
906 stars 186 forks source link

AsyncImageView hangs on setting imageURL #3

Closed jogi closed 13 years ago

jogi commented 13 years ago

Hi I am trying to load an image based on a NSURL. Anything I might be doing wrong?

nicklockwood commented 13 years ago

I see you've closed the issue - did you manage to fix it?

jogi commented 13 years ago

Well there was nothing to fix because I was supplying the wrong imageURL. To reproduce, try this URL - http://chart.apis.google.com/chart?chs=300x300&cht=qr&chl={%22ssid%22:%22connect+me+if+you+can%22,%22secret%22:%22secret%22,%22security%22:%22WPA/WPA2%22}

upon using this url the view hangs. May be there is something that needs to be fixed.

nicklockwood commented 13 years ago

This works for me, although I'm not sure if I fixed something without realising it. When I tried copying and pasting the URL directly into code it returned a nil NSURL, suggesting that it is malformed somehow, but when I put it into Safari and then copied it out again it worked fine.

Anyway, In the latest build I've added a demo app that lets you easily test out URLs by adding them to a plist. Invalid URLs should fail gracefully, but if you find that you can hang the demo app, let me know.

jogi commented 13 years ago

I found the issue. The original URL has { in the url which is considered invalid. After pasting it into Safari, Safari changed it to %7B% and thats the reason it worked after copying-pasting the url back from Safari. Thanks! :)