Crossposting with media_youtube #1180386: Use of copy() and simplexml_load_file() rather than drupal_http_request() causes Media:YouTube thumbnails to be empty since these two modules do this much the same in getMRSS().

CommentFileSizeAuthor
#3 media-vimeo-drupal-http-1504050-3.patch1.5 KBRobW

Comments

drewish’s picture

drewish commented

So I guess we wait until a solution it decided on over there then you want to roll a patch?

steinmb’s picture

steinmb commented

That was my plan, perhaps help out and test it in my env. to see if it breaks and help it get committed.

RobW’s picture

RobW commented
Title: Use of copy() and simplexml_load_file() rather than drupal_http_request() causes Media:Vimeo thumbnails to be empty » Use drupal_http_request() instead of of copy() and simplexml_load_file(), or else Media:Vimeo thumbnails may be empty
Status: Active » Needs review
StatusFileSize
new media-vimeo-drupal-http-1504050-3.patch1.5 KB

#1180386: Use of copy() and simplexml_load_file() rather than drupal_http_request() causes Media:YouTube thumbnails to be empty has been tested and committed. Here's a patch that copies the changes to Media: Vimeo.

vthirteen’s picture

vthirteen commented

I patched the september 7, 2012 7.x-1.x-dev release (the current dev release) with #3 but it doen't seem to work.
I still get
Warning: filesize(): stat failed for vimeo://v/[video-ID] in file_save() (line 570 of /usr/share/drupal7/includes/file.inc).
and consecuently the following error, due to impossibility to manipulate a non existing image:
Unable to generate the derived image located at public://styles/[name-of-the-style]/public/media-vimeo/[video-ID].jpg.

fuzzy76’s picture

fuzzy76 commented

Any news on this?

devin carlson’s picture

devin carlson commented
Version: 7.x-1.x-dev » 7.x-2.x-dev
Issue summary: View changes
Status: Needs review » Fixed

This was fixed as part of a general cleanup of the Vimeo Handler.

Status: Fixed » Closed (fixed)

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