Browse not installing .xol bundles

[Charlie]

As mentioned here, Browse-92 can download .xol files but does not install them into /home/olpc/Library. This issue was previously masked by #7427. (If one can't download, how can one install?)

In addition, attempting to open .xol files from the Journal (either from a USB stick or after downloading through Browse) has no visible effect -- nothing is added to /home/olpc/Library, and there is no response from Journal or Sugar indicating that something may have changed. The bundle does not appear in Browse even after rebooting the XO.

[dsd]

This probably isn't a browse bug but I'm not sure where it should be filed. The relevant line from the journal log is:

1216044454.290063 WARNING root: No activity can open this object, application/vnd.olpc-content.

I also got this error in my journal logs, I think it occurred immediately after the .xol download in browse completed:

1216045203.795985 ERROR dbus.connection: Exception in handler for D-Bus signal:
Traceback (most recent call last):
  File "/usr/lib/python2.5/site-packages/dbus/connection.py", line 214, in maybe_handle_message
    self._handler(*args, **kwargs)
  File "/usr/share/sugar/activities/Journal.activity/journalactivity.py", line 251, in __data_store_updated_cb
    self._check_for_bundle(jobject)
  File "/usr/share/sugar/activities/Journal.activity/journalactivity.py", line 265, in _check_for_bundle
    bundle = misc.get_bundle(jobject)
  File "/usr/share/sugar/activities/Journal.activity/misc.py", line 100, in get_bundle
    return ContentBundle(jobject.file_path)
  File "/usr/lib/python2.5/site-packages/sugar/bundle/contentbundle.py", line 52, in __init__
    info_file = self._get_file('library/library.info')
AttributeError: ContentBundle instance has no attribute '_get_file'

[dsd]

patch to fix the journal contentbundle exception

[dsd]

With that patch, libraries are successfully unpacked. The library index page regeneration is broken, separate issue, see #7523

[tomeu]

The patch looks good to me. Please add a test case following the link below and then set the flags accordingly afterwards. Thanks!

http://wiki.sugarlabs.org/go/DevelopmentTeam/CodeReview#Patch_submission

[dsd]

|TestCase|

• Go to this URL in browse: http://wiki.laptop.org/images/d/df/Bible-en.xol
• Wait a few seconds for it to download and install
• Check that /home/olpc/Library/bible-en exists on the filesystem

[mstone]

We need a sugar package, then a build, then a test run.

[dsd]

This is fixed in joyride-2201