Dbus method new_task does not work

[Zunbeltz Izaola]

new_task method accepts only string for the duedate, startdate and donedate arguments. But the set_due_date method in task.py (line 192) and related methods only accept instances of Date. So the following crash message appears:

org.freedesktop.DBus.Python.AssertionError: 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 "/hmi/gtg/GTG/core/dbuswrapper.py", line 110, in new_task
    nt.set_due_date(duedate)
  File "/hmi/gtg/GTG/core/task.py", line 193, in set_due_date
    assert(isinstance(fulldate, Date))
AssertionError

Conversion from string should be provided in the set_due_date (and related methods). Not only for the real dates, but also for fuzzy dates and no_dates.

[Lionel Dricot (ploum-deactivatedaccount)]

fixed in rev. 463

 DBUS interface still uses our old dates. We should have unit
tests for that, really. Thanks Zunbeltz for being our first DBus user! Sorry
for the bugs, we will correct that. Please continue to report bug, we appreciate it :-)

[Zunbeltz Izaola (zunbeltz)]

Thanks for the quick fix. I would continue testint. I already suspect that i found another bug :-)

[Zunbeltz Izaola (zunbeltz)]

I think this commit was reverted in r509.
So it should be reopened.

[Zunbeltz Izaola (zunbeltz)]

I look in the code changes of rev460 and rev509. I do not understand why it was reverted,
the only explanation is in the commit message of rev509. Any idea what is the problem?

[Lionel Dricot (ploum-deactivatedaccount)]

I checked and it works fine. Also, the code was not reverted at all (rev 509 only change the way modules are loaded).

tested with new_task("Active", "dbus task", "now", "2009-12-31", '', '', "blablabla", '') and it worked fine.

Maybe you have a different issue? If so, please open a new bug and attach the dbus script that trigger the bug