Opened 12 years ago

Closed 11 years ago

Last modified 8 years ago

#1198 closed task (fixed)

use sugar.datastore.* instead of direct DBus access

Reported by: sascha_silbe Owned by: tomeu
Priority: Unspecified by Maintainer Milestone:
Component: Sugar Version: Git as of bugdate
Severity: Unspecified Keywords: r+
Cc: Distribution/OS: Unspecified
Bug Status: New

Description

As discussed on IRC this patch replaces direct datastore DBus API access with calls to sugar.datastore.*. This is the sugar part of the patch, see #1197 for the sugar-toolkit part it depends on.

Attachments (2)

sugar-0001-use-sugar.datastore.datastore-or-at-least-sugar.data.patch (21.0 KB) - added by sascha_silbe 12 years ago.
use sugar.datastore.* instead of direct DBus access
0001-use-sugar.datastore.-instead-of-direct-DBus-access.patch (21.2 KB) - added by sascha_silbe 12 years ago.
use sugar.datastore.* instead of direct DBus access (updated)

Download all attachments as: .zip

Change History (8)

Changed 12 years ago by sascha_silbe

use sugar.datastore.* instead of direct DBus access

Changed 12 years ago by sascha_silbe

use sugar.datastore.* instead of direct DBus access (updated)

comment:1 Changed 12 years ago by sascha_silbe

With the updated patch file transfer works as well (in both directions, including cancelling/declining transfer and watching the transferring file in the Journal details view).
It requires #1201, though.

comment:2 Changed 12 years ago by tomeu

  • Keywords r! added; r? removed

As discussed in IRC, I would prefer to fix bugs at this point of the release cycle. This is a big change that is likely to introduce new bugs and doesn't have a strong enough need AFAICS.

comment:3 Changed 11 years ago by tomeu

  • Milestone changed from 0.86 to 0.88

comment:4 Changed 11 years ago by erikos

  • Keywords r? added; r! removed

comment:5 Changed 11 years ago by erikos

  • Keywords r+ added; r? removed
  • Resolution set to fixed
  • Status changed from new to closed

comment:6 Changed 8 years ago by dnarvaez

  • Milestone 0.88 deleted

Milestone 0.88 deleted

Note: See TracTickets for help on using tickets.