Just some nitpickings:
l.419 self.pool.get('project.issue') can be written self.pool['project.issue']
l.421-422 assign_to.user_id and assign_to.user_id.id or None
can simply be written
assign_to.user_id.id
As assign_to.user_id if not defined will return browse_null and browse_null.id will return None
http://bazaar.launchpad.net/~openerp/openobject-server/7.0/view/head:/openerp/osv/orm.py#L266
Otherwise LGTM
(can't check pt.po)
« Back to merge proposal
Just some nitpickings:
l.419 get('project. issue') can be written self.pool[ 'project. issue']
self.pool.
l.421-422 to.user_ id.id or None
assign_to.user_id and assign_
can simply be written
assign_ to.user_ id.id
As assign_to.user_id if not defined will return browse_null and browse_null.id will return None
http:// bazaar. launchpad. net/~openerp/ openobject- server/ 7.0/view/ head:/openerp/ osv/orm. py#L266
Otherwise LGTM
(can't check pt.po)