Some like embed, some like iframe, I like Floatbox, others like Colorbox.
To please all, I added settings to Googtube to make things configurable.

There are now four methods to show video:

  • Embedded
  • Iframe
  • Colorbox (needs Colorbox module to be installed)
  • Floatbox (needs Floatbox module to be installed)

There are settings for default width and height.

I implemented some options:

  • Use height and width from videoinfo when available When available (Vimeo) height and width from videoinfo will be used. No/Yes
  • Enable fullscreen button No/Yes
  • Default autoplay setting No/Yes
  • Related videos setting Show "related videos"? Not all video formats support this setting No/Yes
  • Removed videos setting Show removed videos? No/Yes

Attached are versions for Drupal 6 and Drupal 7.

Regards
Luch Klooster
CONVENT app & web design & development

Comments

Luch’s picture

Status: Active » Needs review
Luch’s picture

Version: 7.x-1.x-dev » 6.x-1.x-dev
StatusFileSize
new10.89 KB

New upload of D6 version. Corrected missing default values in variable_get functions.

alexandreracine’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev

Hi Luch,

I am testing the googtube D7, the basic is working, that is, one youtube video already there is still working.

But I can't seem to find any interface to configure the new settings.
It should be at least in www.website.com/admin/index I guess?

Let me know what you think.

Cristian.Andrei’s picture

StatusFileSize
new32.76 KB

Here's a patch for the d7 module made out of Luch's code. No other changes have been performed. Thanks Luch for the extra functionality.

Cristian.Andrei’s picture

StatusFileSize
new35.74 KB

And the respective d6 module patch

Cristian.Andrei’s picture

StatusFileSize
new32.76 KB

D7 patch not working with build script, trying new file name (same contents)

Cristian.Andrei’s picture

StatusFileSize
new32.7 KB

Removed trailing whitespaces that broke build script. Please use this patch and disregard the other ones when patching d7 module

artis.bajars’s picture

StatusFileSize
new18.94 KB

Still having trouble with getting the patch to apply cleanly. Rerolled.

fenda’s picture

This patch is not applying for me. What's up?

alexandreracine’s picture

Not working for me too...

# patch googtube.module 1538446-make-googtube-configurable-3.patch
patching file googtube.module
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file googtube.module.rej
patching file googtube.module
Reversed (or previously applied) patch detected!  Assume -R? [n]
Apply anyway? [n]
Skipping patch.
2 out of 2 hunks ignored -- saving rejects to file googtube.module.rej
alexandreracine’s picture

Status: Needs review » Needs work
artis.bajars’s picture

How are you applying it? Running "patch -p1 < 1538446-make-googtube-configurable-3.patch" from the module's directory works cleanly for me.

alexandreracine’s picture

I'll try that and give feedback, thanks.

alexandreracine’s picture

Status: Needs work » Closed (fixed)

The version 7.x-2 is now online! Check it out!

sittard’s picture

On the D6 version I'm still getting 'Missing argument 2 for variable_get()' when embedding YouTube video's seems to work for Vimeo videos.

Luch’s picture

Right. The patch is based on my original attachment of D6 version.
The corrections I made in #2 are not within the patch.

alexandreracine’s picture

Luch, could you create/patch a 6.x-2.0 release please? Thanks.

Luch’s picture

Version: 7.x-1.x-dev » 6.x-1.x-dev
StatusFileSize
new35.85 KB

Patch 6.x-2.0 including my last corrections from #2.

sittard’s picture

Please can we have a release of this version with the patch applied? I'm struggling to apply the patch #18 cleanly against 6.x-1.1 and I can't locate an official 6.x-2.0 to patch against.

Thanks.

alexandreracine’s picture

It's coming...

alexandreracine’s picture

Everything is now online. :)

gfushion’s picture

Version: 6.x-1.x-dev » 7.x-2.x-dev
Category: task » bug

how do you solve this?
Warning: get_headers() [function.get-headers]: URL file-access is disabled in the server configuration in googtube_youtube() (line 159 of /home/youita/public_html/sites/all/modules/googtube/googtube.module).
Warning: get_headers() [function.get-headers]: This function may only be used against URLs. in googtube_youtube() (line 159 of /home/youita/public_html/sites/all/modules/googtube/googtube.module).
Warning: get_headers() [function.get-headers]: URL file-access is disabled in the server configuration in googtube_youtube() (line 159 of /home/youita/public_html/sites/all/modules/googtube/googtube.module).
Warning: get_headers() [function.get-headers]: This function may only be used against URLs. in googtube_youtube() (line 159 of /home/youita/public_html/sites/all/modules/googtube/googtube.module).
Warning: get_headers() [function.get-headers]: URL file-access is disabled in the server configuration in googtube_youtube() (line 159 of /home/youita/public_html/sites/all/modules/googtube/googtube.module).
Warning: get_headers() [function.get-headers]: This function may only be used against URLs. in googtube_youtube() (line 159 of /home/youita/public_html/sites/all/modules/googtube/googtube.module).
Warning: get_headers() [function.get-headers]: URL file-access is disabled in the server configuration in googtube_youtube() (line 159 of /home/youita/public_html/sites/all/modules/googtube/googtube.module).
Warning: get_headers() [function.get-headers]: This function may only be used against URLs. in googtube_youtube() (line 159 of /home/youita/public_html/sites/all/modules/googtube/googtube.module).

alexandreracine’s picture

Version: 7.x-2.x-dev » 6.x-1.x-dev
Category: bug » feature

Please reopen a new bug report, you are in luck if I saw this.

alexandreracine’s picture

Issue summary: View changes

Typo's corrected.