Merge lp:~camptocamp/partner-contact-management/7.0-add-partner-affiliate into lp:~partner-contact-core-editors/partner-contact-management/7.0
Proposed by
Yannick Vaucher @ Camptocamp
Status: | Merged |
---|---|
Merge reported by: | Joël Grand-Guillaume @ camptocamp |
Merged at revision: | not available |
Proposed branch: | lp:~camptocamp/partner-contact-management/7.0-add-partner-affiliate |
Merge into: | lp:~partner-contact-core-editors/partner-contact-management/7.0 |
Diff against target: |
215 lines (+195/-0) 4 files modified
res_partner_affiliate/__init__.py (+24/-0) res_partner_affiliate/__openerp__.py (+42/-0) res_partner_affiliate/res_partner.py (+37/-0) res_partner_affiliate/res_partner_view.xml (+92/-0) |
To merge this branch: | bzr merge lp:~camptocamp/partner-contact-management/7.0-add-partner-affiliate |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Guewen Baconnier @ Camptocamp | no test, code review | Approve | |
Nicolas Bessi - Camptocamp (community) | Needs Fixing | ||
Review via email: mp+144516@code.launchpad.net |
Description of the change
Add module res_partner_
It assumes other modules takes care of checking if current partner is a company or not before using parent_id
Like in openobject-
To post a comment you must log in.
Hello,
There is some styling error here and there, for example sometime there is spaces before ":" sometime not. Some indentation trouble in _columns. A quick pass into a pep8 checker would be nice.
You should not use osv.osv any more but orm.Model:
From openerp.osv import orm, fields
You should suppress commented line:
+ #'lot_count': fields. function( _count_ related, string="Lot", type='integer', multi='count_rel'),
Shouldn't you also add a form view under the kanban view you add ?
Tanks
Regards
Nicolas