openerp-community team mailing list archive

["Martin Trigaux \(OpenERP\)" <mat@xxxxxxxxxxx>]

Review: Approve

Hello,

Sorry for the long delay on this one and thanks for your work to fix the bug.

One small remark: Please do not make write calls in on_change methods. on_change should only be for user interface, no impact on the data (eg: if I close my browser after making the change in edit mode without saving).

On the bug report, you mentioned the issue that the date will be changed when confirming is late. On the other hand it is not really problematic as long as these are in draft.
What do you think ?

In the meanwhile I have merged your changes (everything except the on_change) into 7.0

Regards

revno: 9799 [merge]
revision-id: mat@xxxxxxxxxxx-20140129172009-vbp5n1nco51kaly8

-- 
https://code.launchpad.net/~openerp-community/openobject-addons/fix-1127198-import-invoice-payment-date/+merge/163006
Your team OpenERP Community is subscribed to branch lp:~openerp-community/openobject-addons/fix-1127198-import-invoice-payment-date.