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

Revision history for this message
Michael Telahun Makonnen (mmakonnen) wrote :

Ok. I hadn't thought of that.
My main point was that it should be searchable. Otherwise, it's broken.

Which way is the recommended way?
1. store=False and use fnct_search
    or
2. store={...} (and have the value in the db updated when the related
field changes).

I'm just wondering for my own future reference.

Regards,
Mike.

« Back to merge proposal