Ticket #1241 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

ObjectChooser displays USB media files, but fails to access file (datastore traceback)

Reported by: garycmartin Owned by: alsroot
Priority: Unspecified by Maintainer Milestone: 0.86
Component: sugar-toolkit Version: Git as of bugdate
Severity: Major Keywords: r?
Cc: sascha_silbe Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

Using the ObjectChooser to open a file from external USB media fails to open a selected file with the below (datastore) traceback. Tested with ImageViewer and Write (inserting an image) using latest sugar-jhbuild.

1251299807.015424 DEBUG root: ObjectChooser.__chooser_response_cb: dbus.String(u'/media/USB/misc file backups/Getting involved icons/Developer3.png')
1251299807.047768 DEBUG root: datastore.get
1251299807.050838 DEBUG root: dbus_helpers.get_properties: /media/USB/misc file backups/Getting involved icons/Developer3.png
Traceback (most recent call last):
  File "/home/garycmartin/sugar-jhbuild/install/share/sugar/activities/Write.activity/toolbar.py", line 186, in _image_cb
    chooser.pick(what=chooser.IMAGE, cb=cb)
  File "/home/garycmartin/sugar-jhbuild/install/share/sugar/activities/Write.activity/port/chooser.py", line 49, in pick
    jobject = chooser.get_selected_object()
  File "/home/garycmartin/sugar-jhbuild/install/lib/python2.6/site-packages/sugar/graphics/objectchooser.py", line 102, in get_selected_object
    return datastore.get(self._object_id)
  File "/home/garycmartin/sugar-jhbuild/install/lib/python2.6/site-packages/sugar/datastore/datastore.py", line 144, in get
    metadata = dbus_helpers.get_properties(object_id)
  File "/home/garycmartin/sugar-jhbuild/install/lib/python2.6/site-packages/sugar/datastore/dbus_helpers.py", line 78, in get_properties
    return _get_data_store().get_properties(uid, byte_arrays=True)
  File "/usr/lib/python2.6/site-packages/dbus/proxies.py", line 140, in __call__
    **keywords)
  File "/usr/lib/python2.6/site-packages/dbus/connection.py", line 630, in call_blocking
    message, timeout)
dbus.exceptions.DBusException: org.freedesktop.DBus.Python.IOError: Traceback (most recent call last):
  File "/usr/lib/python2.6/site-packages/dbus/service.py", line 702, in _message_cb
    retval = candidate_method(self, *args, **keywords)
  File "/home/garycmartin/sugar-jhbuild/install/lib/python2.6/site-packages/carquinyol/datastore.py", line 271, in get_properties
    metadata = self._metadata_store.retrieve(uid)
  File "/home/garycmartin/sugar-jhbuild/install/lib/python2.6/site-packages/carquinyol/metadatastore.py", line 39, in retrieve
    return metadatareader.retrieve(metadata_path, properties)
IOError: Couldn't open metadata directory /home/garycmartin/.sugar/default/datastore//m//media/USB/misc file backups/Getting involved icons/Developer3.png/metadata

Attachments

sugar-1241-base.patch Download (1.2 KB) - added by alsroot 4 years ago.
sugar-1241-toolkit.patch Download (2.5 KB) - added by alsroot 4 years ago.
sugar-1241-toolkit.2.patch Download (1.5 KB) - added by alsroot 4 years ago.
sugar-1241-toolkit.3.patch Download (1.2 KB) - added by alsroot 4 years ago.

Change History

Changed 4 years ago by sascha_silbe

  • cc sascha_silbe added

Changed 4 years ago by alsroot

  • owner changed from tomeu to alsroot
  • status changed from new to assigned

Changed 4 years ago by alsroot

Changed 4 years ago by alsroot

Changed 4 years ago by alsroot

  • keywords r? added
  • component changed from sugar-datastore to sugar-toolkit

Changed 4 years ago by alsroot

  • owner changed from alsroot to tomeu

Changed 4 years ago by alsroot

  • milestone changed from Unspecified by Release Team to 0.86

Changed 4 years ago by tomeu

  • keywords r+ added; r? removed
  • owner changed from tomeu to alsroot

Looks good to me, thanks a lot!

Changed 4 years ago by alsroot

  • keywords r? added; r+ removed
  • status changed from closed to reopened
  • resolution fixed deleted

Some activities relies on existing of ds fields like 'title'

Changed 4 years ago by alsroot

Changed 4 years ago by tomeu

  • keywords r+ added; r? removed
  • status changed from reopened to assigned

Makes sense, you can see a bigger list of common properties here:

 http://wiki.laptop.org/go/Low-level_Activity_API#Meta_Data

But activities shouldn't depend on any of them really being there.

Changed 4 years ago by alsroot

  • status changed from assigned to closed
  • resolution set to fixed

Changed 4 years ago by alsroot

  • status changed from closed to reopened
  • resolution fixed deleted

Changed 4 years ago by alsroot

Changed 4 years ago by alsroot

  • keywords r? added; r+ removed

To emulate missed parts of DSObject - optional file_path removing.

Changed 4 years ago by alsroot

  • status changed from reopened to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.