GTG

[gtg-refactor] DBus api is broken due to requester changes

Bug #529817 reported by Bryce Harrington
8
This bug affects 2 people
Affects Status Importance Assigned to Milestone
GTG
Fix Released
Critical
Bryce Harrington

Bug Description

From Lionel's status email today:

- DBus is broken, due to the huge change to the requester.

This affects the gtg_new_task tool and other consumers of the dbus service.

Tags: regression

Related branches

Bryce Harrington (bryce)
tags: added: regression
description: updated
Changed in gtg:
importance: Undecided → Critical
milestone: none → 0.3
Changed in gtg:
status: New → Confirmed
Revision history for this message
Philippe Le Toquin (ppmt) wrote :

not sure if it the same but I use the bzr 0.2.2 and I have the same problem. the dbus commands don't seem to work anymore.

Revision history for this message
Luca Invernizzi (invernizzi) wrote :

That's not the same bug, please open a new bug. In my copy of gtg 0.2.4 it works fine, so please check again. Mahalo.

Changed in gtg:
status: Confirmed → Triaged
Revision history for this message
Zunbeltz Izaola (zunbeltz) wrote :

I can confim that get_tasks_filtered is not working.

org.freedesktop.DBus.Python.AttributeError: Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/dbus/service.py", line 702, in _message_cb
    retval = candidate_method(self, *args, **keywords)
  File "/home/zunbeltz/gtg-bugs/GTG/viewmanager/dbuswrapper.py", line 94, in get_tasks_filtered
    tags, status, started_only, is_root)
  File "/home/zunbeltz/gtg-bugs/GTG/viewmanager/dbuswrapper.py", line 84, in get_task_ids_filtered
    ids = self.req.get_tasks_list(
AttributeError: 'Requester' object has no attribute 'get_tasks_list'

I tied to look at it but I do not understand how the filtering stuff works.

summary: - [gtg-refactor] DBus api is broken due to requestor changes
+ [gtg-refactor] DBus api is broken due to requester changes
Changed in gtg:
assignee: nobody → Luca Invernizzi (invernizzi)
status: Triaged → In Progress
Revision history for this message
Bryce Harrington (bryce) wrote :

I've updated the dbus commands on the 0.3 branch to work with the new filters system. I had to change the API a bit since some of the internal functionality has changed around.

get_tasks_filtered() and get_task_ids_filtered() are the major changes. Both routines are still present but their parameter signatures are changed - they now use filters directly. Unfortunately because of this they no longer support searching by tags, but on the plus side they now will work with any filter, so as new filters are added in the future, this will all "just work".

get_task_ids() is dropped. I decided if someone needs that, they can just use get_task_ids_filtered().

I decided to keep get_tasks() as a convenience function. Basically it just calls self.get_tasks_filtered(['all'])

Also, I've implemented a command line tool gtcli in trunk which has been quite handy in testing out the dbus API. I plan to continue maintaining this and adding to it, and patches for it are of course quite welcome!

Changed in gtg:
assignee: Luca Invernizzi (invernizzi) → Bryce Harrington (bryceharrington)
status: In Progress → Fix Committed
Changed in gtg:
milestone: 0.3 → 0.2.9
Izidor Matušov (izidor)
Changed in gtg:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.