Changes needed:

1. Here is something I don't do everyday, I'm going to ask you to to un-nice 
your solution (but to my defense it's for simplicity)

I have thought about the passback methods in interfaces.base.* - I think your 
solution is nice and everything, but I don't think we should extend the 
interfaces.base.Base* classes just to copy an id.
So I suggest a smaller not as nice solution for now:

http://pastebin.com/raw.php?i=GyPMSTBV

I know you meen to make the nicest solution I'm just worried about extending 
the interfaces.base.Base* classes much more - they are bloated enough already 
and these are exactly the classes we want to attract other people to use for 
contributing new interfaces.

2. please remove the methodname argument in call_method() of the dispatcher it 
is redundant cause the methodname _MUST_ recide in req_dict["methodname"] - we 
have to stick to some kind of format in req_dict and res_dict as they are very 
central in Ladon's design.
That also means you have to change your jsonrpc10 interface class so the method 
name doesn't end up in req_dict["method"].

Best Regards Jakob

-- 
https://code.launchpad.net/~roger-lp/ladon/trunk/+merge/131028
Your team Ladon Developer is subscribed to branch lp:ladon.

-- 
Mailing list: https://launchpad.net/~ladon-dev-team
Post to     : ladon-dev-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ladon-dev-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to