ps3mediaserver/ps3mediaserver — Issue #17: media container not filled in media parser v1
Author: gzsombor
Created: 2012-02-01 01:29:52+00:00
State: open

It seems that the v1 media parser is not filling the container field for the DLNAMediaInfo, which cause problems, because that mediainfo is put it into the database, and if a renderer use the v2 media parser, in :
https://github.com/ps3mediaserver/ps3mediaserver/blob/master/src/main/java/net/pms/dlna/RealFile.java#L60
will call 'run' which will retrieve a DLNAMediaInfo here:
https://github.com/ps3mediaserver/ps3mediaserver/blob/master/src/main/java/net/pms/dlna/RealFile.java#L164
First time, it will be that DLNAMediaInfo which created by the v1 parser, but this will result in a 'file badly parsed' here:
https://github.com/ps3mediaserver/ps3mediaserver/blob/master/src/main/java/net/pms/dlna/RealFile.java#L67 because the getContainer() is null.

I'm not sure what could be the correct fix, and how the two different parser outputs is compatible, but we can
1, ensure that media container is filled in v1 too
2, if the container field in the db container is null, we should skip that, re-parse with v2, insert the new record, and delete the old

Thoughts?