We had duplicate definitions, which led to different implementations
being used in different settings.

In models.py, we used the ones defined in models.py. When called from
the top level `osv` module (e.g. our API), we used the one in
model_helpers.py