Add wizard to proteus

[udono, 2010-10-11 21:53:18]

Looks good.

I test it tomorrow.

Thanks a lot!

http://codereview.appspot.com/2440041/diff/1/proteus/__init__.py
File proteus/__init__.py (right):

http://codereview.appspot.com/2440041/diff/1/proteus/__init__.py#newcode560
proteus/__init__.py:560: def __init__(self, name, models=None, config=None,
context=None):
Why using models (plural) instead of model?

[ced, 2010-10-12 07:56:57]

http://codereview.appspot.com/2440041/diff/1/proteus/__init__.py
File proteus/__init__.py (right):

http://codereview.appspot.com/2440041/diff/1/proteus/__init__.py#newcode560
proteus/__init__.py:560: def __init__(self, name, models=None, config=None,
context=None):
On 2010/10/11 21:53:18, udono wrote:
> Why using models (plural) instead of model?

Because some wizards run on a list of model

[udono, 2010-10-12 08:38:26]

http://codereview.appspot.com/2440041/diff/1/proteus/__init__.py
File proteus/__init__.py (right):

http://codereview.appspot.com/2440041/diff/1/proteus/__init__.py#newcode575
proteus/__init__.py:575: 'model': models[0].__class__.__name__,
Why only take the first model in models, and not all in a loop?

[ced, 2010-10-12 08:47:27]

http://codereview.appspot.com/2440041/diff/1/proteus/__init__.py
File proteus/__init__.py (right):

http://codereview.appspot.com/2440041/diff/1/proteus/__init__.py#newcode575
proteus/__init__.py:575: 'model': models[0].__class__.__name__,
On 2010/10/12 08:38:26, udono wrote:
> Why only take the first model in models, and not all in a loop?

Because I just need to have the model _name

[udono, 2010-10-12 09:03:26]

Tested the patch. My first shot works perfect. I like the api for wizard control
and form filling. It is very nice and death simple. Thanks a lot!

Today I continue testing several wizards via proteus and keep you informed.

Cheers Udo

[Paul J Stevens, 2010-10-16 09:47:37]

The new wizard interface for proteus will trigger an exception when using it
over xmlrpc.

http://codereview.appspot.com/2440041/diff/1/proteus/tests/test_wizard.py
File proteus/tests/test_wizard.py (right):

http://codereview.appspot.com/2440041/diff/1/proteus/tests/test_wizard.py#new...
proteus/tests/test_wizard.py:10: config.set_trytond(':memory:',
database_type='sqlite')
Running this test with xmlrpc will trigger an exception:

Fault: <Fault 1: "<type 'exceptions.OverflowError'>:int exceeds XML-RPC limits">

[ced, 2010-10-18 21:00:38]

http://codereview.appspot.com/2440041/diff/1/proteus/tests/test_wizard.py
File proteus/tests/test_wizard.py (right):

http://codereview.appspot.com/2440041/diff/1/proteus/tests/test_wizard.py#new...
proteus/tests/test_wizard.py:10: config.set_trytond(':memory:',
database_type='sqlite')
On 2010/10/16 09:47:37, Paul J Stevens wrote:
> Running this test with xmlrpc will trigger an exception:
> 
> Fault: <Fault 1: "<type 'exceptions.OverflowError'>:int exceeds XML-RPC
limits">

I can not reproduce. Could you give a complete example?

[Paul J Stevens, 2010-10-19 09:32:33]

Please check http://codereview.appspot.com/2533042 for patch against
test_wizard.py which fails for me.

[ced, 2010-10-19 20:12:35]

On 2010/10/19 09:32:33, Paul J Stevens wrote:
> Please check http://codereview.appspot.com/2533042 for patch against
> test_wizard.py which fails for me.

Could not reproduce. I think it is not directly linked to this patch but to a
general issue with XML-RPC.
Could you fill an issue in roundup with the full traceback?

[paul_nfg.nl, 2010-10-20 13:43:12]

On 10/19/2010 10:12 PM, cedric.krier@b2ck.com wrote:
> On 2010/10/19 09:32:33, Paul J Stevens wrote:
>> Please check http://codereview.appspot.com/2533042 for patch against
>> test_wizard.py which fails for me.
>
> Could not reproduce. I think it is not directly linked to this patch but
> to a general issue with XML-RPC.
> Could you fill an issue in roundup with the full traceback?
>
> http://codereview.appspot.com/2440041/
>

done

-- 
   ________________________________________________________________
   Paul Stevens                                      paul at nfg.nl
   NET FACILITIES GROUP                     GPG/PGP: 1024D/11F8CD31
   The Netherlands________________________________http://www.nfg.nl