! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Open and Save of .pls files goes wrong
Status:
RESOLVED: WONTFIX
Product:
Xfmedia
Component:
playlist

Comments

Description Peter de Ridder editbugs 2007-05-27 13:39:10 CEST
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.3) Gecko/20070217 Iceape/1.1.1 (Debian-1.1.1-2)
Build Identifier: 

Opening an pls file doesn't show any streams, when NumberOfEntries has casing like this.
Saving a pls file gives wrong numbers to the FileN LengthN ... it starts with 0 instead of 1.

Reproducible: Always

Steps to Reproduce:
Comment 1 Peter de Ridder editbugs 2007-05-27 13:42:51 CEST
Created attachment 1162 
Path for open/save pls file

This reads the NumberofEntries using this case as specified in http://www.assistanttools.com/articles/pls_playlist_format.shtml
And it writes pls files starting with File1 instead of File0.
Comment 2 Brian J. Tarricone (not reading bugmail) 2007-05-27 23:01:57 CEST
I've seen both all-lowercase and mixed-caps formats.  I've also seen .pls files starting from 0 instead of 1.  A correct patch needs to check for both casing formats, and first check for File0, and if it's present, assume 0-based indexing, and if not, assume 1-based indexing.
Comment 3 Peter de Ridder editbugs 2007-05-27 23:22:28 CEST
(In reply to comment #2)
> I've seen both all-lowercase and mixed-caps formats.  I've also seen .pls files
> starting from 0 instead of 1.  A correct patch needs to check for both casing
> formats, and first check for File0, and if it's present, assume 0-based
> indexing, and if not, assume 1-based indexing.
> 

In that case is might be a good idea to support both.
I could make a new patch if you don't mind.
Would you recommend to do the same lowercase check for FileN, TitleN and LengthN.
Maybe playlist, Playlist and PlayList too?
Comment 4 Peter de Ridder editbugs 2007-05-27 23:53:13 CEST
Created attachment 1165 
Patch for pls files updated

This patch takes different cases in account.
For NumberOfEntries and also File Length enz

And is check for file0 as start.
This is done checking 0 to NumberOfEntries inclusive.
Comment 5 Robby Workman editbugs 2014-12-02 21:42:19 CET
Xfmedia is unmaintained.

Bug #3294

Reported by:
Peter de Ridder
Reported on: 2007-05-27
Last modified on: 2014-12-02

People

Assignee:
Brian J. Tarricone (not reading bugmail)
CC List:
1 user

Version

Version:
unspecified
Target Milestone:
xfmedia-0.10.0

Attachments

Path for open/save pls file (1.85 KB, patch)
2007-05-27 13:42 CEST , Peter de Ridder
no flags
Patch for pls files updated (3.36 KB, patch)
2007-05-27 23:53 CEST , Peter de Ridder
no flags

Additional information