Comments

dlumberg’s picture

Naming convention mismatch() get_uri vs getUri(). This is vs 7.x-1.x in git

diff --git a/includes/MediaYouTubeStyles.inc b/includes/MediaYouTubeStyles.inc
index 30ecf4b..ee08025 100644
--- a/includes/MediaYouTubeStyles.inc
+++ b/includes/MediaYouTubeStyles.inc
@@ -34,12 +34,12 @@ class MediaYouTubeStyles extends FileStyles {
   }
   function video($effect) {
     $variables = array(
-      'uri' => $this->get_uri(),
-      'width' => $this->get_width(),
-      'height' => $this->get_height(),
+      'uri' => $this->getUri(),
+      'width' => $this->getWidth(),
+      'height' => $this->getHeight(),
       'autoplay' => $this->get_autoplay(),
       'fullscreen' => $this->get_fullscreen(),
     );
-    $this->set_output(theme('media_youtube_video', $variables));
+    $this->setOutput(theme('media_youtube_video', $variables));
   }
}
dlumberg’s picture

StatusFileSize
new770 bytes

This is vs 7.x-1.0-alpha4.

madeby’s picture

Thanks, I tried to do the same to the vimeo module and it seems to work.

This will of course be included in future versions of the module?

g76’s picture

StatusFileSize
new28.89 KB
new34.09 KB

here's the patched modules if it makes it easier for anyone. I haven't tested yet, just quickly patched.

aaron’s picture

Status:Active» Fixed

thanks everyone! i committed a version of this at http://drupalcode.org/project/media_youtube.git/commit/7358b9e that also adds image styles to the possibilities.

Status:Fixed» Closed (fixed)

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