After this morning test i get a bug with this patch. If we are in EDITABLE mode we NEED a save (for wizard).
How to see the problem:
Create a wizard with 1 fields(field1), create 1 buttons.
write 'first press' in field, click button (print in debug self.datas[ids[0]]['field1'])
result, print 'first press'
write 'second press' in field, click button (print in debug self.datas[ids[0]]['field1'])
result, print 'first press'
Like you see, data not saved when we do second click, because now data have a id
(if (not(params.button) and params.editable) or not params.id:)
not True and True or not True
So never save after the first use
So,
if (not(params.button) and params.editable) or not params.id:
Change by
if params.editable
(always save in editable mode)
Doh !!!
After this morning test i get a bug with this patch. If we are in EDITABLE mode we NEED a save (for wizard).
How to see the problem:
Create a wizard with 1 fields(field1), create 1 buttons.
write 'first press' in field, click button (print in debug self.datas[ ids[0]] ['field1' ])
result, print 'first press'
write 'second press' in field, click button (print in debug self.datas[ ids[0]] ['field1' ])
result, print 'first press'
Like you see, data not saved when we do second click, because now data have a id
(if (not(params.button) and params.editable) or not params.id:)
not True and True or not True
So never save after the first use
So,
if (not(params.button) and params.editable) or not params.id:
Change by
if params.editable
(always save in editable mode)