[mythtv-commits] Re: Ticket #341: Music database update does not reread metadata

[MythTV]

#341: Music database update does not reread metadata
------------------------------------+---------------------------------------
 Reporter:  eskil <myth at eskil.org>  |        Owner:  ijr
     Type:  patch                   |       Status:  new
 Priority:  minor                   |    Milestone:     
Component:  mythmusic               |      Version:     
 Severity:  medium                  |   Resolution:     
------------------------------------+---------------------------------------
Comment (by eskil <myth at eskil.org>):

 The attached patch fixes the behaviour.

 Metadata writes date_added and date_modified, containing date and
 time.

 I've changed the behaviour or Decoder::getMetadata slightly. All
 Decoders implement a ::readMetadata as well, the default
 implementation of ::getMetadata calls ::readMetadata, all the existing
 decoders have implementing a new ::readMetadata.

 Additionally, ::getMetadata does *not* save the metadata to the db,
 seems like a very undesired sideeffect of a method called
 "getMetadata".

 main.cpp is now responsible for calling Metadata::dumpToDatabase in
 CheckFile, and it compares the file's mtime to the date_modified in
 the db.

-- 
Ticket URL: <http://cvs.mythtv.org/trac/ticket/341>
MythTV <http://www.mythtv.org/>
MythTV