Fix for latest Youtube change

GitHubWMC / macrotube-providers

Automatically exported from code.google.com/p/macrotube-providers

0 stars 0 forks source link

Fix for latest Youtube change #4

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago

Hey, guys,
This is Jack Luminous, dev for TV Library. The bug that is killing Macrotube 
right now is in the flash vars regex: 

private static readonly Regex _swfvarsMatch = new 
Regex(@"^*flashvars=.*&amp;s.*?"" ", RegexOptions.Compiled | 
RegexOptions.Multiline);

This may be a Unicode issue. Change "&amp" to "u0026amp".

Original issue reported on code.google.com by Departme...@gmail.com on 29 Jan 2012 at 8:01

Merged into: #1

GoogleCodeExporter commented 8 years ago

Thanks for your suggestion. The regex match is failing as you say, although the 
"&"  is valid, it is actually the match string from s.* that appears to have 
changed. I'm testing an alternative match, as there are instances where a 
youtube page can contain more than one flashvars declaration. Please comment 
under issue 1 and I'll close this issue.

Original comment by sbe...@iventris.net on 30 Jan 2012 at 9:51

Changed state: Duplicate

GoogleCodeExporter commented 8 years ago

Thanks a ton for this fix!

Original comment by RudiHend...@gmail.com on 18 Feb 2012 at 1:50