Hi,

Thanks for the effort to make this module. I installed media_youku yesterday, however it can't get the image thumbnail from youku. Also, after doing a bit research in the code, I found that every time each thumbnail is displayed, it issues a 'file_get_contents' request fetching a web page from youku. So if there's 50 thumbnails is displayed, the server will connect youku 50 times.
I've made a patch to fix these two issues. Please help to test, Thanks!

Andy
http://drupaler.cn

CommentFileSizeAuthor
#1 media_youku.diff1011 bytesandyhu
media_youku.patch1.12 KBandyhu

Comments

andyhu’s picture

StatusFileSize
new1011 bytes

Please try this updated patch, the previous one might not working in the right way.

lugir’s picture

Status: Active » Reviewed & tested by the community

#1 works
Thanks

ycshen’s picture

Status: Reviewed & tested by the community » Fixed

hi andyhu ,
I have tested this patch.
patch commited in 9f43b4c
thank you for your contribution.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Issue summary: View changes

change wording