Reply to thread

Message

<blockquote data-quote="enco" data-source="post: 207627" data-attributes="member: 57728">Hi,making my first steps in C# (I'm a Delphi and scripting guy), I adressed the bugs reported <a href="https://forum.team-mediaportal.com/mymusic_bass_player_doesnt_play_songs_special-t28278.html" target="_blank">here</a> and <a href="https://forum.team-mediaportal.com/m3u_playlist_problems_foreign_characters-t5347.html?t=5347" target="_blank">here</a>, being:- tracks containing special characters in .m3u playlists were skipped as well as not displayed properly- track lengths in .pls playlists were off by a factor of 1000 seconds- relative paths in .pls playlists didn't workHere's what I changed:Core.PlayList.PlayListM3uIO.cs:Line 59, old code:[code]using (file = new StreamReader(playlistFileName))[/code]new code:[code]Encoding fileEncoding = Encoding.Default;FileStream stream = File.Open(playlistFileName, FileMode.Open, FileAccess.Read, FileShare.Read);using (file = new StreamReader(stream, fileEncoding, true))[/code]This took care of the problem with special chars in .m3u playlists.Core.PlayList.PlayListPLSIO.cs:Commented out line 138[code]//duration *= 1000;[/code]This fixed the faulty track length display.Lines 142-146, old code:[code]if (tmp.IndexOf(&quot;http:&quot;) &lt; 0 &amp;&amp; tmp.IndexOf(&quot;mms:&quot;) &lt; 0 &amp;&amp; tmp.IndexOf(&quot;rtp:&quot;) &lt; 0){&nbsp; MediaPortal.Util.Utils.GetQualifiedFilename(basePath, ref fileName);&nbsp; newItem.Type = PlayListItem.PlayListItemType.AudioStream;}[/code]New code (just inserted one LOC inbetween):[code]if (tmp.IndexOf(&quot;http:&quot;) &lt; 0 &amp;&amp; tmp.IndexOf(&quot;mms:&quot;) &lt; 0 &amp;&amp; tmp.IndexOf(&quot;rtp:&quot;) &lt; 0){&nbsp; MediaPortal.Util.Utils.GetQualifiedFilename(basePath, ref fileName);&nbsp; newItem.FileName = fileName;&nbsp; newItem.Type = PlayListItem.PlayListItemType.AudioStream;}[/code]This fixed the handling of relative paths.BTW I'm not sure if the assignment of the newItem.Type is correct in this last piece of code. But since it's been working for me so far, I left it alone. Someone with more inside knowledge may verify this.Cheers...</blockquote>

[QUOTE="enco, post: 207627, member: 57728"] Hi, making my first steps in C# (I'm a Delphi and scripting guy), I adressed the bugs reported [url=https://forum.team-mediaportal.com/mymusic_bass_player_doesnt_play_songs_special-t28278.html]here[/url] and [url=https://forum.team-mediaportal.com/m3u_playlist_problems_foreign_characters-t5347.html?t=5347]here[/url], being: - tracks containing special characters in .m3u playlists were skipped as well as not displayed properly - track lengths in .pls playlists were off by a factor of 1000 seconds - relative paths in .pls playlists didn't work Here's what I changed: [B]Core.PlayList.PlayListM3uIO.cs[/B]: Line 59, old code: [code]using (file = new StreamReader(playlistFileName))[/code] new code: [code]Encoding fileEncoding = Encoding.Default; FileStream stream = File.Open(playlistFileName, FileMode.Open, FileAccess.Read, FileShare.Read); using (file = new StreamReader(stream, fileEncoding, true)) [/code] This took care of the problem with special chars in .m3u playlists. [B]Core.PlayList.PlayListPLSIO.cs[/B]: Commented out line 138 [code]//duration *= 1000;[/code] This fixed the faulty track length display. Lines 142-146, old code: [code]if (tmp.IndexOf("http:") < 0 && tmp.IndexOf("mms:") < 0 && tmp.IndexOf("rtp:") < 0) { MediaPortal.Util.Utils.GetQualifiedFilename(basePath, ref fileName); newItem.Type = PlayListItem.PlayListItemType.AudioStream; }[/code] New code (just inserted one LOC inbetween): [code]if (tmp.IndexOf("http:") < 0 && tmp.IndexOf("mms:") < 0 && tmp.IndexOf("rtp:") < 0) { MediaPortal.Util.Utils.GetQualifiedFilename(basePath, ref fileName); newItem.FileName = fileName; newItem.Type = PlayListItem.PlayListItemType.AudioStream; }[/code] This fixed the handling of relative paths. BTW I'm not sure if the assignment of the newItem.Type is correct in this last piece of code. But since it's been working for me so far, I left it alone. Someone with more inside knowledge may verify this. Cheers... [/QUOTE]

Verification