796302 - [media] [music] App does not display audio files with no manifest data

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia, defect, P1)

Description

[GH to BZ]

[GitHub issue by dclarke on 2012-08-21T23:20:21Z, https://github.com/mozilla-b2g/gaia/issues/3666]
otoro daily build 08/22/2012

Music App does not display / play audio files with no manifest data. 

With the gallery app this might be fatal, but we are ok with not having manifest data in audio, and displaying under Unknown Album.

Comment 1

[GH to BZ]

[GitHub comment by dclarke on 2012-08-21T23:22:01Z]
@davidflanagan

Comment 2

[GH to BZ]

[GitHub comment by autonome on 2012-09-10T15:15:51Z]
Do you mean metadata?

Comment 3

[GH to BZ]

[GitHub comment by davidflanagan on 2012-09-10T15:47:42Z]
I missed this one when it first got filed... 

With the new metadata parsing code that just landed this bug might not still be reproducible.

This might be a mediadb issue rather than a Music app issue.  If so, you can reassign it to me, @dominickuo.  There may be code in mediadb that treats lack of metadata as a sign of file corruption, and purposely ignores such files.

Comment 4

[GH to BZ]

[GitHub comment by dominickuo on 2012-09-10T21:21:21Z]
Still can be reproducible after I merge the new metadata parsing code, let me find out it's an issue of Music app or mediadb.

Comment 5

[GH to BZ]

[GitHub comment by autonome on 2012-09-25T03:22:58Z]
hi @dominickuo, any update on this?

Comment 6

[GH to BZ]

[GitHub comment by davidflanagan on 2012-09-25T03:39:32Z]
When @dominickuo or I updates Music to use the new mediadb, we should ensure that this bug gets fixed at the same time.

Comment 7

[GH to BZ]

[GitHub comment by dominickuo on 2012-09-26T11:36:00Z]
@autonome @davidflanagan I am working on converting music to new mediadb, and I will send a patch soon.

Comment 8

[Dominic Kuo [:dkuo]]

Music has already updated to the the new mediadb, can anyone verify this?

Comment 9

[Tim Guan-tin Chien [:timdream] (please needinfo)]

dclarke, can you verify with your original STR? Thanks.

Comment 10

[[e-mail] [:onecyrenus]]

got it, thanks Tim, hadn't seen this. 

Possibly changing to resolved fixed, or adding keyword to "qawanted" would help

Comment 11

[Tim Guan-tin Chien [:timdream] (please needinfo)]

(In reply to [e-mail]  [:onecyrenus] from comment #10)
> got it, thanks Tim, hadn't seen this. 
> 
> Possibly changing to resolved fixed, or adding keyword to "qawanted" would
> help

Is this correct? I am new to Bugzilla process.

Comment 12

[[e-mail] [:onecyrenus]]

Tim, 
  So I do know when a bug moves to resolved fixed, I look at it, and I know that qawanted is being watched by the entire QA team.

Comment 13

[John Shih]

Verified
The music without enough information will show Unknown artist right now
Build info
2012-10-23
gaia master: 608ba8a6a931322c96ac1cea7e02f4c4bf9d70fd
gecko: 167d5a73c6aa564d1abcae4654467aa18aba9a3e