#3151 HIGH First D: removing bundle from journal should uninstall it

[Zarro Boogs per Child]

#3151: removing bundle from journal should uninstall it
-------------------------------+--------------------------------------------
  Reporter:  danw              |       Owner:  danw                  
      Type:  defect            |      Status:  new                   
  Priority:  high              |   Milestone:  First Deployment, V1.0
 Component:  journal-activity  |     Version:                        
Resolution:                    |    Keywords:  review?               
  Verified:  0                 |  
-------------------------------+--------------------------------------------
Changes (by danw):

  * keywords:  => review?


Comment:

 Actually, involving the datastore at all gets very complicated, because
 the dbus signals are asynchronous, so whichever of the two possibilities
 (BeforeDelete signal, pass metadata to Delete signal) we use, the object
 will have been destroyed before the journal gets the signal, and there's
 no enough data in the metadata alone to find and delete the installed
 version of the bundle. (You need to extract some information out of the
 bundle zipfile itself.)

 So instead, I just patched the journal to delete the installed bundle
 *before* sending the delete request to the datastore. This means that if
 the bundle were deleted by some other process, it wouldn't delete the
 installed activity/content, but there doesn't seem to be any easy fix for
 that. (Other than moving the special bundle handling from the journal to
 the datastore.)

 OK to commit?

-- 
Ticket URL: <https://dev.laptop.org/ticket/3151#comment:5>
One Laptop Per Child <https://dev.laptop.org>
OLPC bug tracking system