The proposal to merge lp:~openerp-dev/openobject-server/trunk-reexport-thu into
lp:openobject-server has been updated.
Description changed to:
Time for some bikeshedding:
# 0
from openerp import *
class res_user(Model):
_columns = { 'name': Char() }
# 1
from openerp import exceptions, fields, models
class res_user(models.Model):
_columns = { 'name': fields.Char() }
# 2
import openerp
class res_user(openerp.Model):
_columns = { 'name': openerp.Char() }
# 3
import openerp
class res_user(openerp.models.Model):
_columns = { 'name': openerp.fields.Char() }
I think I like #1: openerp is not just an orm, so having openerp re-exporting
the (admitedly commonly used) Model, Char, ... symbols seems to stress too much
the orm side.
We also have openerp.exceptions which provides Warning, AccessDenied,
AccessError. I like those to be on the same level as Model, Char, ... (which is
the case when openerp.models lives side by side with openerp.fields).
But openerp is also about the workflows, the WSGI stack, import/export,
reporting, maybe about process monitoring (I wish it was not), ... and really
the use of models.Model, and fields.Char is just a few more characters among
thousands of lines.
Thoughts?
For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-reexport-thu/+merge/146372
--
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-reexport-thu/+merge/146372
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openobject-server/trunk-reexport-thu.
_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help : https://help.launchpad.net/ListHelp
