[Bugs] #2229 UNSP: metadata-only update sets filesize property to 0 (was: Using Journal to rename an entry sets entry size to "Empty" when sort mode is "View by size")

[Sugar Labs Bugs]

#2229: metadata-only update sets filesize property to 0
------------------------------------------+---------------------------------
    Reporter:  garycmartin                |          Owner:  aa      
        Type:  defect                     |         Status:  assigned
    Priority:  Unspecified by Maintainer  |      Milestone:  0.90    
   Component:  sugar-datastore            |        Version:  0.90.x  
    Severity:  Major                      |       Keywords:  dextrose
Distribution:                             |   Status_field:  Assigned
------------------------------------------+---------------------------------
Changes (by sascha_silbe):

  * keywords:  dextrose, r? => dextrose
  * distribution:  Unspecified =>
  * version:  Unspecified => 0.90.x
  * component:  journal => sugar-datastore


Comment:

 Replying to [comment:6 aa]:

 > The default_file path is '', and so the filesize is updated with an
 inexistent file.
 That's a bug in the data store then. Passing file_path='' for an entry
 that already has data means updating the metadata only and leaving the
 data unchanged (see [http://git.sugarlabs.org/projects/sugar-
 datastore/repos/mainline/blobs/master/src/carquinyol/filestore.py#line61
 carquinyol.filestore.FileStore.store()]; especially notice the commented
 out part).
 If we change the API to not support metadata-only updates anymore, more
 than just the Journal will break.

-- 
Ticket URL: <http://bugs.sugarlabs.org/ticket/2229#comment:7>
Sugar Labs <http://sugarlabs.org/>
Sugar Labs bug tracking system