dlumberg’s picture

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

diff --git a/includes/ b/includes/
index 30ecf4b..ee08025 100644
--- a/includes/
+++ b/includes/
@@ -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

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

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 that also adds image styles to the possibilities.

Status:Fixed» Closed (fixed)

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