If an user type a YouTube URL inside an tag, the Googtube regex will process it anyway, leaving a broken tag and no video.

Solution: correct regex.

CommentFileSizeAuthor
#2 googtube-551924.patch1000 bytesfranz
#3 googtube-551924.patch1.24 KBfranz

Comments

josesanmartin’s picture

josesanmartin commented

my bad :-)

If an user type a YouTube URL inside an <a href=""> tag, the Googtube regex will process it anyway, leaving a broken tag and no video.

Solution: correct regex.

...which I'm trying to do now

franz’s picture

franz
any
Location Montréal
commented
StatusFileSize
new googtube-551924.patch1000 bytes

Please, ignore this patch (explained below).

franz’s picture

franz
any
Location Montréal
commented
StatusFileSize
new googtube-551924.patch1.24 KB

Sorry, forgot to include a new regex for google videos too. Please, consider this patch.

alexandreracine’s picture

alexandreracine commented
Status: Active » Needs review

Can someone test this? Thanks.

franz’s picture

franz
any
Location Montréal
commented

bump?

maedi’s picture

maedi commented

subscribing

alexandreracine’s picture

alexandreracine commented

Frank, do you have a website witch is currently working with your patch? Thanks.

franz’s picture

franz
any
Location Montréal
commented

I fixed it for a site on my company, but can't remember which. I tested this on a local site.

alexandreracine’s picture

alexandreracine commented
Status: Needs review » Fixed

This is now comitted to the dev6 branch.

alexandreracine’s picture

alexandreracine commented

Thanks! :)

Status: Fixed » Closed (fixed)

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