Code review comment for lp:~openerp-dev/openobject-addons/trunk-bug-782168-uco

Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

I agree that this patch is incorrect, but I'm afraid the other suggestions are not correct either.
In fact, after discussing with R&D management, we are more inclined to close the bug as invalid, because it is really an exception case.
As always with OpenERP, we don't want or plan to handle automatically all exception cases, because there is no safe way of doing that. If we enforce one behavior (e.g. automatic recompute), there will be other users who will need the opposite behavior, and will have more trouble *after the fix* than before. See more details in the bug comments.

review: Disapprove

« Back to merge proposal