Exaile displays negative time for .wav files

Bug #455115 reported by Micah Gersten
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Exaile
Fix Released
Medium
reacocard
exaile (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: exaile

When I open the file, it shows a negative play time. Screenshot attached.

ProblemType: Bug
Architecture: amd64
Date: Sun Oct 18 21:52:34 2009
DistroRelease: Ubuntu 9.10
Package: exaile 0.3.0.1-0ubuntu6
PackageArchitecture: all
ProcEnviron:
 PATH=(custom, no user)
 LANG=en_US.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.31-14.48-generic
SourcePackage: exaile
Uname: Linux 2.6.31-14-generic x86_64

Revision history for this message
Micah Gersten (micahg) wrote :
Revision history for this message
Olivier Kaloudoff (kalou) wrote :

Hello,

  I think the problem is laying in xl/metadata/wav.py, the type_map'ing is not working as expected for wav files.
I played with pdb, and it turns out that the code goes straight to the exception.

Removing the type_map as below fixes the problem;

< opener = type_map[sndhdr.what(self.loc)]
< f = opener.open(self.loc, "rb")
---
> f = wave.open(self.loc, "rb")

Kalou

Changed in exaile (Ubuntu):
status: New → Confirmed
Changed in exaile:
assignee: nobody → Johannes Sasongko (sjohannes)
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Laurent (laurent-pouillet) wrote :

The patch proposed by Kalou seems to work for me. It also solves #153929.

After discussing with Aren on #exaile the patch removes the support of aiff, snd and au file formats.
A specific wav module might be needed.

Changed in exaile:
assignee: Johannes Sasongko (sjohannes) → nobody
Revision history for this message
reacocard (reacocard) wrote :

should be fixed in trunk/2714. detection is now based on file extension instead of sndhdr.

Changed in exaile:
assignee: nobody → Aren Olson (reacocard)
milestone: none → 0.3.1
status: Triaged → Fix Committed
Revision history for this message
Laurent (laurent-pouillet) wrote :

The fix committed in 2714 seems to work for me.
Thanks

Revision history for this message
Micah Gersten (micahg) wrote :

Marking this Triaged in Ubuntu since upstream is fixing it.

Changed in exaile (Ubuntu):
importance: Undecided → Low
status: Confirmed → Triaged
reacocard (reacocard)
Changed in exaile:
status: Fix Committed → Fix Released
Changed in exaile (Ubuntu):
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.