Code review comment for lp:~openerp-dev/openobject-server/trunk-extended-view-inheritance-xmo

> - could we add a small @help for the new `mode` field to make its meaning crystal clear (you know, with the view_type/view_mode conspiracy ;-))

done

> - wouldn't _check_mode() be simpler to implement as a pure SQL constraint (_sql_constraints)?

and done