Comment 8 for bug 668465

Revision history for this message
Serge (sboivin) wrote :

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)