Closing tab on a new form reload another record after not saving and don't close the tab.

Linked to https://hg.tryton.org/tryton/rev/5b4d41c301a3.

To reproduce (tryton >=5.8.3):

• Open a new tab (sales for instance)
• Create a new record
• Close the tab
• At the confirmation to save, click on "No"
• Another record will be displayed in form view instead of closing tab.

added tryton type::behavior + 1 deleted label

As discussed on the irc this is linked to #9881 (closed). We should keep the behaviour introduced on this issue but just for some actions.

The new, switch view, next, forward and close actions should be executed without comparing the ids while others should not be executed if the record does not save the changes.

added 1 deleted label and removed 1 deleted label

I just tested it on 5.8 series and the problem is just with the new records. For me the behaviour seems correct when the record is already saved.

Could we just invert order of testing ?

https://hg.tryton.org/tryton/file/default/tryton/gui/window/form.py#l669

                if self.sig_reload(test_modified=False):
                    if self.screen.current_record:
                        return record_id == self.screen.current_record.id
                    elif record_id < 0:
                        return True

by

                if self.sig_reload(test_modified=False):
                    if record_id < 0:
                        return True
                    elif self.screen.current_record:
                        return record_id == self.screen.current_record.id

assigned to @maxx

added 1 deleted label and removed 1 deleted label

Sounds like a reasonable solution but should be tested with a report or and action related

added 1 deleted label and removed 1 deleted label

Hum, too quick... It's not working when clicking on report

New review363421002 at https://codereview.tryton.org/363421002/#ps352051002

I checked in 5.7 : the behavior is the same as my patch when clicking on report on a new record that should not be saved, it loads another record. Don't know what should be done in this case because it was the original behavior.

Since #9881 (closed) we should not execute a report nor a different record when the changes are discarded. We should do nothing in this cas.

Probably it is need to add a new parameter to the modified_save to skip the test for some actions (i.e: report, action, relate) and keep the current behaviour for other (i.e: close tab)

There's something not logical in the returned value in method "modified_save":

• sig_close (https://hg.tryton.org/tryton/file/default/tryton/gui/window/form.py#l677) needs a True value for closing which is the value returned when i don't want to save a new record.

• But action (https://hg.tryton.org/tryton/file/default/tryton/gui/window/form.py#l683) needs the opposite value to bypass the action

added 1 deleted label and removed 1 deleted label

added 1 deleted label and removed 1 deleted label

I do not agree sig_close must return True for the tab being closed.

The problem is the special case of new record for which the user decides to not save. For this case we want to close the tab but we do not want to launch an action. So we need a third returned value which could be None. The only ambiguous case is the ko. I think we must return True if the current record is still the same and otherwise return None. Then the sig_close must return True if modified_save returns True or None. (sig_action does not need to be changed).

review363421002 updated at https://codereview.tryton.org/363421002/#ps342441003

The solution of msg67928 can not be implemented in sao because Promise has only two states. I guess we could resolve the promise with the value True or Null and test it in __close_allowed.

I'm not comfortable with javascript and the promises. In the python script, when record_id is < 0, return is None. So in javascript it should return a jQuery.Deferred().reject() ? Should i pass the value in the "reject" ? jQuery.Deferred().reject(true) ?

Yes you must pass an argument so the _close_allowed can return a resolved or failed promise.

review363421002 updated at https://codereview.tryton.org/363421002/#ps352121002

review363421002 updated at https://codereview.tryton.org/363421002/#ps365581005

review363421002 updated at https://codereview.tryton.org/363421002/#ps369461003

New changeset bea021af96d5 by Maxime Richez in branch 'default':
Close tab when cancelling save on new record
https://hg.tryton.org/sao/rev/bea021af96d5

added 1 deleted label and removed 1 deleted label

closed

New changeset 941b90300b84 by Maxime Richez in branch 'default':
Close tab when cancelling save on new record
https://hg.tryton.org/tryton/rev/941b90300b84

New changeset 4df7e301f1df by Maxime Richez in branch 'default':
Close tab when cancelling save on new record
https://hg.tryton.org/tryton-env/rev/4df7e301f1df

New changeset b0b231e9ed9a by Maxime Richez in branch '6.0':
Close tab when cancelling save on new record
https://hg.tryton.org/sao/rev/b0b231e9ed9a

New changeset 279f4dca4fbe by Maxime Richez in branch '5.8':
Close tab when cancelling save on new record
https://hg.tryton.org/sao/rev/279f4dca4fbe

New changeset 86070e4180f1 by Maxime Richez in branch '5.0':
Close tab when cancelling save on new record
https://hg.tryton.org/sao/rev/86070e4180f1

New changeset 1a549e8dda5c by Maxime Richez in branch '6.0':
Close tab when cancelling save on new record
https://hg.tryton.org/tryton/rev/1a549e8dda5c

New changeset e3a715ee875f by Maxime Richez in branch '5.8':
Close tab when cancelling save on new record
https://hg.tryton.org/tryton/rev/e3a715ee875f

New changeset c7477c90e6be by Maxime Richez in branch '5.0':
Close tab when cancelling save on new record
https://hg.tryton.org/tryton/rev/c7477c90e6be