Changes between Initial Version and Version 10 of Ticket #8550


Ignore:
Timestamp:
Mar 2, 2011, 10:08:07 PM (13 years ago)
Author:
osimons
Comment:

Perhaps the Agilo people could chime in if they have an opinion on the matter, but seeing protocols can be arbitrary we need to stick to lowest common denominator for the supported type system.

There is no point in serializing an object and passing it down the wire to a client, if the same object can't be described as XML or JSON in a simple way for other clients requesting that. And there is no point in requesting ticket data and returning some complex type that clients like Mylyn can't ever be expected to make sense of.

That is why the wrapper/type system in use by Agilo should learn how to quack like ducks... A wrapper around a string by subclassing str and adding some __new__() magic, or whatever other way would allow it to behave like the underlying type.

I haven't looked at Agilo code, nor have I any plans to do so. Most of my arguments are of course highly speculative when I don't know what they actually do or why they do it...

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #8550

    • Property Owner changed from osimons to Andrea Tomasini
  • Ticket #8550 – Description

    initial v10  
    11Calling function ticket.get() defined tickets in Agilo return an error. Below Traceback output from log:
    22
    3 
     3{{{
    442011-03-01 10:39:37,852 Trac[web_ui] ERROR: RPC(XML-RPC) Unhandled protocol error
    55Traceback (most recent call last):
     
    1919    raise TypeError, "cannot marshal %s objects" % type(value)
    2020TypeError: cannot marshal <class 'agilo.ticket.model.TicketValueWrapper'> objects
     21}}}