#5660 NORM Never A: Activity.save() throws exception if create_jobject=False

[Zarro Boogs per Child]

#5660: Activity.save() throws exception if create_jobject=False
---------------------+------------------------------------------------------
  Reporter:  cscott  |       Owner:  tomeu         
      Type:  defect  |      Status:  new           
  Priority:  normal  |   Milestone:  Never Assigned
 Component:  sugar   |     Version:                
Resolution:          |    Keywords:  update.1?     
  Verified:  0       |    Blocking:                
 Blockedby:          |  
---------------------+------------------------------------------------------

Comment(by cscott):

 A one line patch would do: just put the logging.debug statement inside the
 if -- no need to return early, since the unimplemented write_file will
 throw a NotImplementedError and everything will be fine.

 If creating objects even with no data is desired behavior, then #5655
 needs to be addressed some other way: we're creating far too many versions
 of even empty objects right now.  I haven't tracked down the ultimate
 source yet, but sugar or ohm or somebody is calling save() a lot without
 the user asking for it.

-- 
Ticket URL: <http://dev.laptop.org/ticket/5660#comment:3>
One Laptop Per Child <http://dev.laptop.org>
OLPC bug tracking system