offbyoneBB / mp-onlinevideos2

MediaPortal OnlineVideos Plugin
15 stars 12 forks source link

watch-series not showing series that end with "." #116

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Enter watch-series util
2. "Series -> C -> Cops L.A.C." or "Series -> Gomer Pyle, U.S.M.C." (not sure 
if there are more)
3. No seasons/episodes will be shown

What is the expected output? What do you see instead?
List of all seasons/episodes from that show

Please use labels and text to provide additional information.
The problem is that GetWebData of e.g. 
"http://watchseries.eu/serie/cops_l.a.c." will return the "Um, Where did the 
page go?" page for some reason.

Original issue reported on code.google.com by bgmei...@gmail.com on 8 Sep 2011 at 9:28

GoogleCodeExporter commented 9 years ago
System.Uri class has a serious bug with trailing dots. It strips them out.

Documented here: 
https://connect.microsoft.com/VisualStudio/feedback/details/386695/system-uri-in
correctly-strips-trailing-dots#

I have implemented workaround in the code in r1549. Method call is commented 
out because workaround sets static flag in the UriParser and will affect all 
System.Uri classes created after this, APPLICATION wise (complete MediaPortal, 
not just OnlineVideos). Although the fix is probably valid it might have some 
not wanted side-effects so that's why I leave the decision whether to uncomment 
the line to offbyone :)

Original comment by mitja.skuver on 8 Sep 2011 at 4:13

GoogleCodeExporter commented 9 years ago
Nice finding!
I guess we can enable this even though it changes to the global handling of 
URLs in the process, because these changes are rather a fix and might even fix 
other plugins's problems ;)

Original comment by bborgsd...@gmail.com on 13 Sep 2011 at 2:54

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r1555.

Original comment by bborgsd...@gmail.com on 15 Sep 2011 at 10:00