Needs review
Project:
Googtube
Version:
7.x-2.1
Component:
code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Feb 2015 at 06:10 UTC
Updated:
18 Nov 2016 at 23:55 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
azrael3x commentedintegrate expression:
at line 135 of module file
#(((http://)?)|(^./))(((www.)?)|(^./))youtube\.com/watch[?]v=([^\[\]()<.,\s\n\t\r]+)(?![^<]*)#i
with "(s)?"
#(((http(s)?://)?)|(^./))(((www.)?)|(^./))youtube\.com/watch[?]v=([^\[\]()<.,\s\n\t\r]+)(?![^<]*)#i
Comment #2
Andrzej7 commentedThanks azrael3x, but it doesn't work for Drupal6 and 7
your code is about 40 line (D6), after change
all youtube links makes a simple link.
Any help?
Comment #3
wrebis commentedthe line should be:
if (preg_match_all('#(((https?://)?)|(^./))(((www.)?)|(^./))youtube\.com/watch[?]v=([^\[\]()<.,\s\n\t\r]+)(?![^<]*</a>)#i', $text, $matches)); {Comment #4
Andrzej7 commentedThanks
it works :-)
Comment #5
federico commentedWorks for me too.
Comment #6
tikaszvince commentedI attach a patch about the modification of @wrebus.
Comment #7
bobbia commentedThis patch worked for me, also. I no longer see the https:// before the video.
I would imagine that most people will need this patch now that all of YouTube's videos start with https://.
Thanks!