Browse Source

Fix pep8

pull/6/head
Sandy Carter 10 years ago
parent
commit
7155f88102
  1. 8
      account_partner_merge/__openerp__.py
  2. 5
      account_partner_merge/partner_merge.py
  3. 21
      base_address_category/__openerp__.py
  4. 14
      base_address_category/base_address.py
  5. 26
      base_continent/__openerp__.py
  6. 2
      base_location/company.py
  7. 31
      base_partner_merge/base_partner_merge.py
  8. 89
      base_partner_merge/validate_email.py
  9. 2
      base_partner_sequence/__openerp__.py
  10. 40
      firstname_display_name_trigger/__openerp__.py
  11. 13
      firstname_display_name_trigger/res_partner.py
  12. 4
      firstname_display_name_trigger/tests/__init__.py
  13. 22
      firstname_display_name_trigger/tests/test_display_name.py
  14. 38
      partner_address_ldap/__openerp__.py
  15. 274
      partner_address_ldap/address.py
  16. 25
      partner_address_ldap/company.py
  17. 7
      partner_address_ldap/partner.py
  18. 120
      partner_address_ldap/wizard/wiz_import_adresses.py
  19. 1
      partner_contact_address_detailed/__init__.py
  20. 29
      partner_contact_address_detailed/__openerp__.py
  21. 35
      partner_firstname/__openerp__.py
  22. 4
      partner_firstname/partner.py
  23. 4
      partner_firstname/res_user.py
  24. 4
      partner_firstname/tests/__init__.py
  25. 26
      partner_firstname/tests/test_partner_firstname.py
  26. 8
      portal_partner_merge/__openerp__.py
  27. 7
      portal_partner_merge/wizard/portal_wizard.py
  28. 41
      res_partner_affiliate/__openerp__.py
  29. 4
      res_partner_affiliate/res_partner.py

8
account_partner_merge/__openerp__.py

@ -18,14 +18,14 @@
# along with this program. If not, see <http://www.gnu.org/licenses/>. # along with this program. If not, see <http://www.gnu.org/licenses/>.
# #
############################################################################## ##############################################################################
{'name' : 'Account Partner Merge',
'version' : '1.0',
{'name': 'Account Partner Merge',
'version': '1.0',
'category': 'Hidden', 'category': 'Hidden',
'description': """Update invoice commercial_partner_id""", 'description': """Update invoice commercial_partner_id""",
'author' : 'Camptocamp',
'author': 'Camptocamp',
'maintainer': 'Camptocamp', 'maintainer': 'Camptocamp',
'website': 'http://www.camptocamp.com/', 'website': 'http://www.camptocamp.com/',
'depends' : ['account_report_company', 'base_partner_merge'],
'depends': ['account_report_company', 'base_partner_merge'],
'data': ['account_partner_merge_view.xml'], 'data': ['account_partner_merge_view.xml'],
'test': [], 'test': [],
'installable': True, 'installable': True,

5
account_partner_merge/partner_merge.py

@ -18,9 +18,10 @@
# along with this program. If not, see <http://www.gnu.org/licenses/>. # along with this program. If not, see <http://www.gnu.org/licenses/>.
# #
############################################################################## ##############################################################################
from openerp.osv import osv
from openerp.osv import orm
class MergePartnerAutomatic(osv.TransientModel):
class MergePartnerAutomatic(orm.TransientModel):
_inherit = 'base.partner.merge.automatic.wizard' _inherit = 'base.partner.merge.automatic.wizard'
def _update_values(self, cr, uid, src_partners, dst_partner, context=None): def _update_values(self, cr, uid, src_partners, dst_partner, context=None):

21
base_address_category/__openerp__.py

@ -22,8 +22,8 @@
############################################################################## ##############################################################################
{ {
"name" : "Partner Address Category",
"description" : """\
"name": "Partner Address Category",
"description": """\
res.partner.address.category res.partner.address.category
---------------------------- ----------------------------
@ -37,17 +37,16 @@
res.partner.address.category records are migrated to res.partner.address.category records are migrated to
res.partner.category records. res.partner.category records.
""", """,
"version" : "1.2",
"author" : "Camptocamp",
"category" : "Generic Modules/Base",
"version": "1.2",
"author": "Camptocamp",
"category": "Generic Modules/Base",
"website": "http://www.camptocamp.com", "website": "http://www.camptocamp.com",
"depends" : [
"base",
],
"data" : [
"depends": [
"base",
],
"data": [
"security/security.xml" "security/security.xml"
'security/ir.model.access.csv', 'security/ir.model.access.csv',
],
"active": False,
],
"installable": True "installable": True
} }

14
base_address_category/base_address.py

@ -23,7 +23,8 @@
############################################################################## ##############################################################################
from openerp.osv import osv, fields, orm
from openerp.osv import fields, orm
class ResPartnerAdressCategory(orm.Model): class ResPartnerAdressCategory(orm.Model):
def name_get(self, cr, uid, ids, context=None): def name_get(self, cr, uid, ids, context=None):
@ -47,17 +48,15 @@ class ResPartnerAdressCategory(orm.Model):
while ids: while ids:
cr.execute('select distinct parent_id ' cr.execute('select distinct parent_id '
'from res_partner_address_category ' 'from res_partner_address_category '
'where id in %s', ids)
'where id in %s', ids)
ids = [parent_id for (parent_id,) in cr.fetchall() if parent_id] ids = [parent_id for (parent_id,) in cr.fetchall() if parent_id]
if not level: if not level:
return False return False
level -= 1 level -= 1
return True return True
_description='Partner address Categories'
_name = 'res.partner.address.category' _name = 'res.partner.address.category'
_description = 'Partner address Categories'
_columns = { _columns = {
'name': fields.char('Category Name', required=True, size=64), 'name': fields.char('Category Name', required=True, size=64),
'parent_id': fields.many2one('res.partner.address.category', 'parent_id': fields.many2one('res.partner.address.category',
@ -69,7 +68,7 @@ class ResPartnerAdressCategory(orm.Model):
'child_ids': fields.one2many('res.partner.address.category', 'child_ids': fields.one2many('res.partner.address.category',
'parent_id', 'parent_id',
'Children Category'), 'Children Category'),
'active' : fields.boolean('Active'),
'active': fields.boolean('Active'),
} }
_constraints = [ _constraints = [
(_check_recursion, (_check_recursion,
@ -77,7 +76,7 @@ class ResPartnerAdressCategory(orm.Model):
['parent_id']) ['parent_id'])
] ]
_defaults = { _defaults = {
'active' : lambda *a: 1,
'active': lambda *a: 1,
} }
_order = 'parent_id,name' _order = 'parent_id,name'
@ -92,4 +91,3 @@ class ResPartnerAddress(orm.Model):
'category_id', 'category_id',
'Address categories'), 'Address categories'),
} }

26
base_continent/__openerp__.py

@ -21,22 +21,22 @@
{ {
'name': 'Continent management',
'version': '1.0',
'depends': ['base'],
'author': 'Camptocamp',
'license': 'AGPL-3',
'description': """
'name': 'Continent management',
'version': '1.0',
'depends': ['base'],
'author': 'Camptocamp',
'license': 'AGPL-3',
'description': """
This module introduces continent management. This module introduces continent management.
============================================ ============================================
Links continents to countries, Links continents to countries,
adds continent field on partner form adds continent field on partner form
""", """,
'category': 'Generic Modules/Base',
'data': [
'base_continent_view.xml',
'base_continent_data.xml',
'security/ir.model.access.csv'],
'active': False,
'installable': True,
'category': 'Generic Modules/Base',
'data': [
'base_continent_view.xml',
'base_continent_data.xml',
'security/ir.model.access.csv',
],
'installable': True,
} }

2
base_location/company.py

@ -43,5 +43,5 @@ class ResCompany(orm.Model):
_columns = { _columns = {
'better_zip_id': fields.many2one('res.better.zip', 'Location', select=1, 'better_zip_id': fields.many2one('res.better.zip', 'Location', select=1,
help=('Use the city name or the zip code' help=('Use the city name or the zip code'
' to search the location')),
' to search the location')),
} }

31
base_partner_merge/base_partner_merge.py

@ -1,4 +1,5 @@
#!/usr/bin/env python
# -*- coding: utf-8 -*-
from __future__ import absolute_import from __future__ import absolute_import
from email.utils import parseaddr from email.utils import parseaddr
import functools import functools
@ -14,7 +15,7 @@ from openerp.tools import mute_logger
from .validate_email import validate_email from .validate_email import validate_email
import openerp import openerp
from openerp.osv import osv, orm
from openerp.osv import orm
from openerp.osv import fields from openerp.osv import fields
from openerp.osv.orm import browse_record from openerp.osv.orm import browse_record
from openerp.tools.translate import _ from openerp.tools.translate import _
@ -55,7 +56,7 @@ def is_integer_list(ids):
return all(isinstance(i, (int, long)) for i in ids) return all(isinstance(i, (int, long)) for i in ids)
class ResPartner(osv.Model):
class ResPartner(orm.Model):
_inherit = 'res.partner' _inherit = 'res.partner'
_columns = { _columns = {
@ -64,7 +65,7 @@ class ResPartner(osv.Model):
} }
class MergePartnerLine(osv.TransientModel):
class MergePartnerLine(orm.TransientModel):
_name = 'base.partner.merge.line' _name = 'base.partner.merge.line'
_columns = { _columns = {
@ -77,7 +78,7 @@ class MergePartnerLine(osv.TransientModel):
_order = 'min_id asc' _order = 'min_id asc'
class MergePartnerAutomatic(osv.TransientModel):
class MergePartnerAutomatic(orm.TransientModel):
""" """
The idea behind this wizard is to create a list of potential partners to The idea behind this wizard is to create a list of potential partners to
merge. We use two objects, the first one is the wizard for the end-user. merge. We use two objects, the first one is the wizard for the end-user.
@ -326,7 +327,7 @@ class MergePartnerAutomatic(osv.TransientModel):
if parent_id and parent_id != dst_partner.id: if parent_id and parent_id != dst_partner.id:
try: try:
dst_partner.write({'parent_id': parent_id}) dst_partner.write({'parent_id': parent_id})
except (osv.except_osv, orm.except_orm):
except (orm.except_orm, orm.except_orm):
_logger.info('Skip recursive partner hierarchies for ' _logger.info('Skip recursive partner hierarchies for '
'parent_id %s of partner: %s', 'parent_id %s of partner: %s',
parent_id, dst_partner.id) parent_id, dst_partner.id)
@ -341,7 +342,7 @@ class MergePartnerAutomatic(osv.TransientModel):
return return
if len(partner_ids) > 3: if len(partner_ids) > 3:
raise osv.except_osv(
raise orm.except_orm(
_('Error'), _('Error'),
_("For safety reasons, you cannot merge more than 3 contacts " _("For safety reasons, you cannot merge more than 3 contacts "
"together. You can re-open the wizard several times if " "together. You can re-open the wizard several times if "
@ -351,7 +352,7 @@ class MergePartnerAutomatic(osv.TransientModel):
and len(set(partner.email for partner and len(set(partner.email for partner
in proxy.browse(cr, uid, partner_ids, in proxy.browse(cr, uid, partner_ids,
context=context))) > 1): context=context))) > 1):
raise osv.except_osv(
raise orm.except_orm(
_('Error'), _('Error'),
_("All contacts must have the same email. Only the " _("All contacts must have the same email. Only the "
"Administrator can merge contacts with different emails.")) "Administrator can merge contacts with different emails."))
@ -378,7 +379,7 @@ class MergePartnerAutomatic(osv.TransientModel):
[partner.id for partner [partner.id for partner
in src_partners])], in src_partners])],
context=context)): context=context)):
raise osv.except_osv(
raise orm.except_orm(
_('Error'), _('Error'),
_("Only the destination contact may be linked to existing " _("Only the destination contact may be linked to existing "
"Journal Items. Please ask the Administrator if you need to" "Journal Items. Please ask the Administrator if you need to"
@ -399,8 +400,12 @@ class MergePartnerAutomatic(osv.TransientModel):
dst_partner.message_post( dst_partner.message_post(
body='%s %s' % ( body='%s %s' % (
_("Merged with the following partners:"), _("Merged with the following partners:"),
", ".join('%s<%s>(ID %s)' % (p.name, p.email or 'n/a', p.id)
for p in src_partners)))
", ".join(
'%s<%s>(ID %s)' % (p.name, p.email or 'n/a', p.id)
for p in src_partners
)
)
)
for partner in src_partners: for partner in src_partners:
partner.unlink() partner.unlink()
@ -513,7 +518,7 @@ class MergePartnerAutomatic(osv.TransientModel):
] ]
if not groups: if not groups:
raise osv.except_osv(_('Error'),
raise orm.except_orm(_('Error'),
_("You have to specify a filter for your " _("You have to specify a filter for your "
"selection")) "selection"))
@ -555,7 +560,7 @@ class MergePartnerAutomatic(osv.TransientModel):
cr, uid, cr, uid,
current_partner_ids, current_partner_ids,
context context
)[-1].id,
)[-1].id,
'state': 'selection', 'state': 'selection',
}) })
else: else:

89
base_partner_merge/validate_email.py

@ -1,3 +1,4 @@
# -*- coding: utf-8 -*-
# RFC 2822 - style email validation for Python # RFC 2822 - style email validation for Python
# (c) 2012 Syrus Akbary <me@syrusakbary.com> # (c) 2012 Syrus Akbary <me@syrusakbary.com>
# Extended from (c) 2011 Noel Bush <noel@aitools.org> # Extended from (c) 2011 Noel Bush <noel@aitools.org>
@ -19,14 +20,15 @@
import re import re
import smtplib import smtplib
import socket
try: try:
import DNS import DNS
ServerError = DNS.ServerError ServerError = DNS.ServerError
except: except:
DNS = None DNS = None
class ServerError(Exception): pass
class ServerError(Exception):
pass
# All we are really doing is comparing the input string to one # All we are really doing is comparing the input string to one
# gigantic regular expression. But building that regexp, and # gigantic regular expression. But building that regexp, and
# ensuring its correctness, is made much easier by assembling it # ensuring its correctness, is made much easier by assembling it
@ -40,48 +42,35 @@ except:
# even when it's not strictly necessary. This way we don't forget # even when it's not strictly necessary. This way we don't forget
# when it is necessary.) # when it is necessary.)
# #
WSP = r'[ \t]' # see 2.2.2. Structured Header Field Bodies
CRLF = r'(?:\r\n)' # see 2.2.3. Long Header Fields
NO_WS_CTL = r'\x01-\x08\x0b\x0c\x0f-\x1f\x7f' # see 3.2.1. Primitive Tokens
QUOTED_PAIR = r'(?:\\.)' # see 3.2.2. Quoted characters
FWS = r'(?:(?:' + WSP + r'*' + CRLF + r')?' + \
WSP + r'+)' # see 3.2.3. Folding white space and comments
CTEXT = r'[' + NO_WS_CTL + \
r'\x21-\x27\x2a-\x5b\x5d-\x7e]' # see 3.2.3
CCONTENT = r'(?:' + CTEXT + r'|' + \
QUOTED_PAIR + r')' # see 3.2.3 (NB: The RFC includes COMMENT here
# as well, but that would be circular.)
COMMENT = r'\((?:' + FWS + r'?' + CCONTENT + \
r')*' + FWS + r'?\)' # see 3.2.3
CFWS = r'(?:' + FWS + r'?' + COMMENT + ')*(?:' + \
FWS + '?' + COMMENT + '|' + FWS + ')' # see 3.2.3
ATEXT = r'[\w!#$%&\'\*\+\-/=\?\^`\{\|\}~]' # see 3.2.4. Atom
ATOM = CFWS + r'?' + ATEXT + r'+' + CFWS + r'?' # see 3.2.4
DOT_ATOM_TEXT = ATEXT + r'+(?:\.' + ATEXT + r'+)*' # see 3.2.4
DOT_ATOM = CFWS + r'?' + DOT_ATOM_TEXT + CFWS + r'?' # see 3.2.4
QTEXT = r'[' + NO_WS_CTL + \
r'\x21\x23-\x5b\x5d-\x7e]' # see 3.2.5. Quoted strings
QCONTENT = r'(?:' + QTEXT + r'|' + \
QUOTED_PAIR + r')' # see 3.2.5
QUOTED_STRING = CFWS + r'?' + r'"(?:' + FWS + \
r'?' + QCONTENT + r')*' + FWS + \
r'?' + r'"' + CFWS + r'?'
LOCAL_PART = r'(?:' + DOT_ATOM + r'|' + \
QUOTED_STRING + r')' # see 3.4.1. Addr-spec specification
DTEXT = r'[' + NO_WS_CTL + r'\x21-\x5a\x5e-\x7e]' # see 3.4.1
DCONTENT = r'(?:' + DTEXT + r'|' + \
QUOTED_PAIR + r')' # see 3.4.1
DOMAIN_LITERAL = CFWS + r'?' + r'\[' + \
r'(?:' + FWS + r'?' + DCONTENT + \
r')*' + FWS + r'?\]' + CFWS + r'?' # see 3.4.1
DOMAIN = r'(?:' + DOT_ATOM + r'|' + \
DOMAIN_LITERAL + r')' # see 3.4.1
ADDR_SPEC = LOCAL_PART + r'@' + DOMAIN # see 3.4.1
WSP = r'[ \t]' # see 2.2.2. Structured Header Field Bodies
CRLF = r'(?:\r\n)' # see 2.2.3. Long Header Fields
NO_WS_CTL = r'\x01-\x08\x0b\x0c\x0f-\x1f\x7f' # see 3.2.1. Primitive Tokens
QUOTED_PAIR = r'(?:\\.)' # see 3.2.2. Quoted characters
FWS = r'(?:(?:{0}*{1})?{0}+)'.format(WSP, CRLF) # see 3.2.3. Folding white space and comments
CTEXT = r'[{0}\x21-\x27\x2a-\x5b\x5d-\x7e]'.format(NO_WS_CTL) # see 3.2.3
# see 3.2.3 (NB: The RFC includes COMMENT here as well, but that would be circular.)
CCONTENT = r'(?:{0}|{1})'.format(CTEXT, QUOTED_PAIR)
COMMENT = r'\((?:{0}?{1})*{0}?\)'.format(FWS, CCONTENT) # see 3.2.3
CFWS = r'(?:{0}?{1})*(?:{0}?{1}|{0})'.format(FWS, COMMENT) # see 3.2.3
ATEXT = r'[\w!#$%&\'\*\+\-/=\?\^`\{\|\}~]' # see 3.2.4. Atom
ATOM = r'{0}?{1}+{0}?'.format(CFWS, ATEXT) # see 3.2.4
DOT_ATOM_TEXT = r'{0}+(?:\.{0}+)*'.format(ATEXT) # see 3.2.4
DOT_ATOM = r'{0}?{1}{0}?'.format(CFWS, DOT_ATOM_TEXT) # see 3.2.4
QTEXT = r'[{0}\x21\x23-\x5b\x5d-\x7e]'.format(NO_WS_CTL) # see 3.2.5. Quoted strings
QCONTENT = r'(?:{0}|{1})'.format(QTEXT, QUOTED_PAIR) # see 3.2.5
QUOTED_STRING = r'{0}?"(?:{1}?{2})*{1}?"{0}?'.format(CFWS, FWS, QCONTENT)
LOCAL_PART = r'(?:{0}|{1})'.format(DOT_ATOM, QUOTED_STRING) # see 3.4.1. Addr-spec specification
DTEXT = r'[{0}\x21-\x5a\x5e-\x7e]'.format(NO_WS_CTL) # see 3.4.1
DCONTENT = r'(?:{0}|{1})'.format(DTEXT, QUOTED_PAIR) # see 3.4.1
DOMAIN_LITERAL = r'{0}?\[(?:{1}?{2})*{1}?\]{0}?'.format(CFWS, FWS, DCONTENT) # see 3.4.1
DOMAIN = r'(?:{0}|{1})'.format(DOT_ATOM, DOMAIN_LITERAL) # see 3.4.1
ADDR_SPEC = r'{0}@{1}'.format(LOCAL_PART, DOMAIN) # see 3.4.1
# A valid address will match exactly the 3.4.1 addr-spec. # A valid address will match exactly the 3.4.1 addr-spec.
VALID_ADDRESS_REGEXP = '^' + ADDR_SPEC + '$' VALID_ADDRESS_REGEXP = '^' + ADDR_SPEC + '$'
def validate_email(email, check_mx=False,verify=False):
def validate_email(email, check_mx=False, verify=False):
"""Indicate whether the given string is a valid email address """Indicate whether the given string is a valid email address
according to the 'addr-spec' portion of RFC 2822 (see section according to the 'addr-spec' portion of RFC 2822 (see section
@ -94,22 +83,28 @@ def validate_email(email, check_mx=False,verify=False):
assert re.match(VALID_ADDRESS_REGEXP, email) is not None assert re.match(VALID_ADDRESS_REGEXP, email) is not None
check_mx |= verify check_mx |= verify
if check_mx: if check_mx:
if not DNS: raise Exception('For check the mx records or check if the email exists you must have installed pyDNS python package')
if not DNS:
raise Exception('For check the mx records or check if the '
'email exists you must have installed pyDNS '
'python package')
DNS.DiscoverNameServers() DNS.DiscoverNameServers()
hostname = email[email.find('@')+1:]
hostname = email[email.find('@') + 1:]
mx_hosts = DNS.mxlookup(hostname) mx_hosts = DNS.mxlookup(hostname)
for mx in mx_hosts: for mx in mx_hosts:
try: try:
smtp = smtplib.SMTP() smtp = smtplib.SMTP()
smtp.connect(mx[1]) smtp.connect(mx[1])
if not verify: return True
if not verify:
return True
status, _ = smtp.helo() status, _ = smtp.helo()
if status != 250: continue
if status != 250:
continue
smtp.mail('') smtp.mail('')
status, _ = smtp.rcpt(email) status, _ = smtp.rcpt(email)
if status != 250: return False
if status != 250:
return False
break break
except smtplib.SMTPServerDisconnected: #Server not permits verify user
except smtplib.SMTPServerDisconnected: # Server not permits verify user
break break
except smtplib.SMTPConnectError: except smtplib.SMTPConnectError:
continue continue
@ -119,5 +114,5 @@ def validate_email(email, check_mx=False,verify=False):
# import sys # import sys
# sys.modules[__name__],sys.modules['validate_email_module'] = validate_email,sys.modules[__name__]
# sys.modules[__name__], sys.modules['validate_email_module'] = validate_email, sys.modules[__name__]
# from validate_email_module import * # from validate_email_module import *

2
base_partner_sequence/__openerp__.py

@ -46,7 +46,7 @@
"data": [ "data": [
'partner_sequence.xml', 'partner_sequence.xml',
'partner_view.xml', 'partner_view.xml',
],
],
"demo": [], "demo": [],
"active": False, "active": False,
"installable": True "installable": True

40
firstname_display_name_trigger/__openerp__.py

@ -18,24 +18,24 @@
# along with this program. If not, see <http://www.gnu.org/licenses/>. # along with this program. If not, see <http://www.gnu.org/licenses/>.
# #
############################################################################## ##############################################################################
{'name': 'Link module if partner_lastname and account_report_company are installed',
'version': '1.0',
'author': 'Camptocamp',
'maintainer': 'Camptocamp',
'category': 'Hidden',
'complexity': 'normal', # easy, normal, expert
'depends': [
'account_report_company',
'partner_firstname',
],
'description': """
Adapt the computation of display name so that it gets visible in tree and kanban views.
""",
'website': 'http://www.camptocamp.com',
'data': [],
'installable': True,
'images': [],
'auto_install': True,
'license': 'AGPL-3',
'application': False}
{
'name': 'Link module if partner_lastname and account_report_company are installed',
'version': '1.0',
'author': 'Camptocamp',
'maintainer': 'Camptocamp',
'category': 'Hidden',
'license': 'AGPL-3',
'depends': [
'account_report_company',
'partner_firstname',
],
'description': """
Adapt the computation of display name so that it gets visible in tree and kanban views.
""",
'website': 'http://www.camptocamp.com',
'data': [],
'installable': True,
'auto_install': True,
'application': False,
}

13
firstname_display_name_trigger/res_partner.py

@ -40,21 +40,20 @@ class ResPartner(orm.Model):
names = (record.lastname, record.firstname) names = (record.lastname, record.firstname)
name = u" ".join([s for s in names if s]) name = u" ".join([s for s in names if s])
if record.parent_id and not record.is_company: if record.parent_id and not record.is_company:
name = "%s, %s" % (record.parent_id.name, name)
name = "%s, %s" % (record.parent_id.name, name)
if context.get('show_address'): if context.get('show_address'):
name = name + "\n" + self._display_address(cr, uid, record, without_company=True, context=context) name = name + "\n" + self._display_address(cr, uid, record, without_company=True, context=context)
name = name.replace('\n\n','\n')
name = name.replace('\n\n','\n')
name = name.replace('\n\n', '\n')
name = name.replace('\n\n', '\n')
if context.get('show_email') and record.email: if context.get('show_email') and record.email:
name = "%s <%s>" % (name, record.email) name = "%s <%s>" % (name, record.email)
res.append((record.id, name)) res.append((record.id, name))
return res return res
_display_name_store_triggers = { _display_name_store_triggers = {
'res.partner': (lambda self,cr,uid,ids,context=None: self.search(cr, uid, [('id','child_of',ids)]),
'res.partner': (lambda self, cr, uid, ids, context=None: self.search(cr, uid, [('id', 'child_of', ids)]),
['parent_id', 'is_company', 'name', 'firstname', 'lastname'], 10) ['parent_id', 'is_company', 'name', 'firstname', 'lastname'], 10)
}
}
# indirection to avoid passing a copy of the overridable method when declaring the function field # indirection to avoid passing a copy of the overridable method when declaring the function field
_display_name = lambda self, *args, **kwargs: self._display_name_compute(*args, **kwargs) _display_name = lambda self, *args, **kwargs: self._display_name_compute(*args, **kwargs)
@ -62,4 +61,4 @@ class ResPartner(orm.Model):
_columns = { _columns = {
# extra field to allow ORDER BY to match visible names # extra field to allow ORDER BY to match visible names
'display_name': fields.function(_display_name, type='char', string='Name', store=_display_name_store_triggers), 'display_name': fields.function(_display_name, type='char', string='Name', store=_display_name_store_triggers),
}
}

4
firstname_display_name_trigger/tests/__init__.py

@ -1,5 +1,7 @@
# -*- coding: utf-8 -*-
import test_display_name import test_display_name
checks = [ checks = [
test_display_name test_display_name
]
]

22
firstname_display_name_trigger/tests/test_display_name.py

@ -1,31 +1,39 @@
# -*- coding: utf-8 -*-
import unittest2 import unittest2
import openerp.tests.common as common import openerp.tests.common as common
class test_display_name(common.TransactionCase): class test_display_name(common.TransactionCase):
def setUp(self): def setUp(self):
super(test_display_name,self).setUp()
cr, uid = self.cr, self.uid
super(test_display_name, self).setUp()
self.res_partner = self.registry('res.partner') self.res_partner = self.registry('res.partner')
def test_00_create_res_partner(self): def test_00_create_res_partner(self):
""" Test if the display name has been correctly set """ """ Test if the display name has been correctly set """
cr, uid = self.cr, self.uid cr, uid = self.cr, self.uid
partner_id = self.res_partner.create(cr, uid, {'lastname': 'Lastname', 'firstname': 'Firstname', 'is_company': True})
partner_id = self.res_partner.create(cr, uid, {
'lastname': 'Lastname',
'firstname': 'Firstname',
'is_company': True,
})
partner_records = self.res_partner.browse(cr, uid, [partner_id]) partner_records = self.res_partner.browse(cr, uid, [partner_id])
p1 = partner_records[0] p1 = partner_records[0]
self.assertEqual(p1.display_name, 'Lastname Firstname', 'Partner display_name incorect')
self.assertEqual(p1.display_name, 'Lastname Firstname', 'Partner display_name incorrect')
def test_01_res_partner_write_lastname(self): def test_01_res_partner_write_lastname(self):
""" Test if the display name has been correctly set """ """ Test if the display name has been correctly set """
cr, uid = self.cr, self.uid cr, uid = self.cr, self.uid
partner_id = self.res_partner.create(cr, uid, {'lastname': 'Lastname', 'firstname': 'Firstname', 'is_company': True})
partner_id = self.res_partner.create(cr, uid, {
'lastname': 'Lastname',
'firstname': 'Firstname',
'is_company': True
})
partner_records = self.res_partner.browse(cr, uid, [partner_id]) partner_records = self.res_partner.browse(cr, uid, [partner_id])
p1 = partner_records[0] p1 = partner_records[0]
self.res_partner.write(cr, uid, partner_id, {'lastname': 'Last'}) self.res_partner.write(cr, uid, partner_id, {'lastname': 'Last'})
self.assertEqual(p1.display_name, 'Last Firstname', 'Partner display_name incorect')
self.assertEqual(p1.display_name, 'Last Firstname', 'Partner display_name incorrect')
if __name__ == '__main__': if __name__ == '__main__':
unittest2.main() unittest2.main()

38
partner_address_ldap/__openerp__.py

@ -32,37 +32,41 @@
############################################################################## ##############################################################################
{ {
"name" : "Partner synchronization from OpenERP to ldap",
"version" : "1.2",
"author" : "Camptocamp",
"depends" : ["base"],
"category" : "Generic Modules/Misc",
"name": "Partner synchronization from OpenERP to ldap",
"version": "1.2",
"author": "Camptocamp",
"depends": ["base"],
"category": "Generic Modules/Misc",
"website": "http://www.camptocamp.com", "website": "http://www.camptocamp.com",
"description": """ "description": """
Live partner address synchronization through a LDAP module (inetOrgPerson). Live partner address synchronization through a LDAP module (inetOrgPerson).
OpenERP becomes the master of the LDAP. Each time an address is deleted, created or updated the same is done in the ldap (a new record is pushed).
The LDAP configuration is done in the company view. There can be one different LDAP per company. Do not forget to activate
OpenERP becomes the master of the LDAP. Each time an address is deleted,
created or updated the same is done in the ldap (a new record is pushed).
The LDAP configuration is done in the company view. There can be one different
LDAP per company. Do not forget to activate
the LDAP link in the configuration. the LDAP link in the configuration.
The used LDAP depends on the current user company. The used LDAP depends on the current user company.
This module does not allows bulk batching synchronisation into the LDAP and is thus not suitable for an instant use with an existing LDAP.
In order to use it with an existing LDAP you have to alter the uid of contact in your LDAP. The uid should be terp_ plus the OpenERP
This module does not allows bulk batching synchronisation into the LDAP and is
thus not suitable for an instant use with an existing LDAP.
In order to use it with an existing LDAP you have to alter the uid of contact
in your LDAP. The uid should be terp_ plus the OpenERP
contact id (for example terp_10). contact id (for example terp_10).
N.B: N.B:
The module requires the python-ldap library The module requires the python-ldap library
Unicode support --> As python ldap does not support unicode we try to decode string if it fails we transliterate values.
Unicode support --> As python ldap does not support unicode we try to decode
string if it fails we transliterate values.
Active Directory Support for Windows server 2003, try 2008 at your own risk Active Directory Support for Windows server 2003, try 2008 at your own risk
(AD support not tested for Version 6 of OpenERP looking for active dir test infra) (AD support not tested for Version 6 of OpenERP looking for active dir test infra)
""", """,
"init_xml" : ["security/security.xml"],
"update_xml":['company_view.xml',
'address_view.xml',
"wizard.xml"],
"demo_xml" : [],
"active": False,
"data": [
"security/security.xml"
'company_view.xml',
'address_view.xml',
"wizard.xml",
],
"installable": False "installable": False
} }

274
partner_address_ldap/address.py

@ -9,10 +9,10 @@
# Contribution : Joel Grand-Guillaume # Contribution : Joel Grand-Guillaume
# #
# WARNING: This program as such is intended to be used by professional # WARNING: This program as such is intended to be used by professional
# programmers who take the whole responsability of assessing all potential
# programmers who take the whole responsibility of assessing all potential
# consequences resulting from its eventual inadequacies and bugs # consequences resulting from its eventual inadequacies and bugs
# End users who are looking for a ready-to-use solution with commercial # End users who are looking for a ready-to-use solution with commercial
# garantees and support are strongly adviced to contract a Free Software
# guarantees and support are strongly advised to contract a Free Software
# Service Company # Service Company
# #
# This program is Free Software; you can redistribute it and/or # This program is Free Software; you can redistribute it and/or
@ -30,22 +30,24 @@
# Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. # Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
# #
############################################################################## ##############################################################################
#TODO FInd why company parameter are cached
# TODO Find why company parameter are cached
import re import re
import unicodedata import unicodedata
import netsvc
from openerp import netsvc
try: try:
import ldap import ldap
import ldap.modlist import ldap.modlist
except :
print 'python ldap not installed please install it in order to use this module'
except:
print('python ldap not installed please install it in order to use this module')
from osv import osv, fields
from tools.translate import _
from openerp.osv import orm, fields
from openerp.tools.translate import _
logger = netsvc.Logger() logger = netsvc.Logger()
class LdapConnMApper(object): class LdapConnMApper(object):
"""LdapConnMApper: push specific fields from the Terp Partner_contacts to the """LdapConnMApper: push specific fields from the Terp Partner_contacts to the
LDAP schema inetOrgPerson. Ldap bind options are stored in company.r""" LDAP schema inetOrgPerson. Ldap bind options are stored in company.r"""
@ -61,7 +63,7 @@ class LdapConnMApper(object):
self.connexion = '' self.connexion = ''
self.ACTIVDIR = False self.ACTIVDIR = False
#Reading ldap pref
# Reading ldap pref
user = osv_obj.pool.get('res.users').browse(cursor, uid, uid, context=context) user = osv_obj.pool.get('res.users').browse(cursor, uid, uid, context=context)
company = osv_obj.pool.get('res.company').browse(cursor, company = osv_obj.pool.get('res.company').browse(cursor,
uid, uid,
@ -75,26 +77,26 @@ class LdapConnMApper(object):
self.OU = company.ounit self.OU = company.ounit
self.ACTIVDIR = company.is_activedir self.ACTIVDIR = company.is_activedir
mand = (self.USER_DN, self.CONTACT_DN, self.LDAP_SERVER , self.PASS, self.OU)
mand = (self.USER_DN, self.CONTACT_DN, self.LDAP_SERVER, self.PASS, self.OU)
if company.ldap_active: if company.ldap_active:
for param in mand: for param in mand:
if not param: if not param:
raise osv.except_osv(_('Warning !'),
_('An LDAP parameter is missing for company %s') % (company.name,))
raise orm.except_orm(_('Warning !'),
_('An LDAP parameter is missing for company %s') % (company.name, ))
def get_connexion(self): def get_connexion(self):
"""create a new ldap connexion""" """create a new ldap connexion"""
logger.notifyChannel("LDAP Address", netsvc.LOG_DEBUG, logger.notifyChannel("LDAP Address", netsvc.LOG_DEBUG,
_('connecting to server ldap %s') % (self.LDAP_SERVER,))
if self.PORT :
_('connecting to server ldap %s') % (self.LDAP_SERVER, ))
if self.PORT:
self.connexion = ldap.open(self.LDAP_SERVER, self.PORT) self.connexion = ldap.open(self.LDAP_SERVER, self.PORT)
else :
else:
self.connexion = ldap.open(self.LDAP_SERVER) self.connexion = ldap.open(self.LDAP_SERVER)
self.connexion.simple_bind_s(self.USER_DN, self.PASS) self.connexion.simple_bind_s(self.USER_DN, self.PASS)
return self.connexion return self.connexion
class LDAPAddress(osv.osv):
class LDAPAddress(orm.Model):
"""Override the CRUD of the objet in order to dynamically bind to ldap""" """Override the CRUD of the objet in order to dynamically bind to ldap"""
_inherit = 'res.partner.address' _inherit = 'res.partner.address'
ldapMapper = None ldapMapper = None
@ -108,7 +110,7 @@ class LDAPAddress(osv.osv):
'column name to lastname ;')) 'column name to lastname ;'))
cr.execute('ALTER TABLE res_partner_address RENAME column name to lastname ;') cr.execute('ALTER TABLE res_partner_address RENAME column name to lastname ;')
except Exception, e:
except Exception:
cr.rollback() cr.rollback()
logger.notifyChannel(_('LDAP address init'), logger.notifyChannel(_('LDAP address init'),
netsvc.LOG_INFO, netsvc.LOG_INFO,
@ -119,7 +121,7 @@ class LDAPAddress(osv.osv):
def _compute_name(self, firstname, lastname): def _compute_name(self, firstname, lastname):
firstname = firstname or u'' firstname = firstname or u''
lastname = lastname or u'' lastname = lastname or u''
firstname = (u' '+ firstname).rstrip()
firstname = (u' ' + firstname).rstrip()
return u"%s%s" % (lastname, firstname) return u"%s%s" % (lastname, firstname)
def _name_get_fnc(self, cursor, uid, ids, name, arg, context=None): def _name_get_fnc(self, cursor, uid, ids, name, arg, context=None):
@ -129,37 +131,36 @@ class LDAPAddress(osv.osv):
reads = self.read(cursor, uid, ids, ['lastname', 'firstname']) reads = self.read(cursor, uid, ids, ['lastname', 'firstname'])
res = [] res = []
for record in reads: for record in reads:
## We want to have " firstname" or ""
# We want to have " firstname" or ""
name = self._compute_name(record['firstname'], record['lastname']) name = self._compute_name(record['firstname'], record['lastname'])
res.append((record['id'], name)) res.append((record['id'], name))
return dict(res) return dict(res)
# TODO get the new version of name search not vulnerable to sql injections
# def name_search(self, cursor, user, name, args=None, operator='ilike', context=None, limit=100):
# if not context: context = {}
# prep_name = '.*%s.*' %(name)
# cursor.execute(("select id from res_partner_address where"
# " (to_ascii(convert( lastname, 'UTF8', 'LATIN1'),'LATIN-1') ~* '%s'"
# " or to_ascii(convert( firstname, 'UTF8', 'LATIN1'),'LATIN-1') ~* '%s')"
# " limit %s") % (prep_name, prep_name, limit))
# res = cursor.fetchall()
# if res:
# res = [x[0] for x in res]
# else:
# res = []
# # search in partner name to know if we are searching partner...
# partner_obj=self.pool.get('res.partner')
# part_len = len(res)-limit
# if part_len > 0:
# partner_res = partner_obj.search(cursor, user, [('name', 'ilike', name)],
# limit=part_len, context=context)
# for p in partner_res:
# addresses = partner_obj.browse(cursor, user, p).address
# # Take each contact and add it to
# for add in addresses:
# res.append(add.id)
# return self.name_get(cursor, user, res, context)
# TODO get the new version of name search not vulnerable to sql injections
# def name_search(self, cursor, user, name, args=None, operator='ilike', context=None, limit=100):
# if not context: context = {}
# prep_name = '.*%s.*' %(name)
# cursor.execute(("select id from res_partner_address where"
# " (to_ascii(convert( lastname, 'UTF8', 'LATIN1'),'LATIN-1') ~* '%s'"
# " or to_ascii(convert( firstname, 'UTF8', 'LATIN1'),'LATIN-1') ~* '%s')"
# " limit %s") % (prep_name, prep_name, limit))
# res = cursor.fetchall()
# if res:
# res = [x[0] for x in res]
# else:
# res = []
# # search in partner name to know if we are searching partner...
# partner_obj=self.pool.get('res.partner')
# part_len = len(res)-limit
# if part_len > 0:
# partner_res = partner_obj.search(cursor, user, [('name', 'ilike', name)],
# limit=part_len, context=context)
# for p in partner_res:
# addresses = partner_obj.browse(cursor, user, p).address
# # Take each contact and add it to
# for add in addresses:
# res.append(add.id)
# return self.name_get(cursor, user, res, context)
_columns = { _columns = {
'firstname': fields.char('First name', size=256), 'firstname': fields.char('First name', size=256),
@ -169,7 +170,7 @@ class LDAPAddress(osv.osv):
store=True, string='Contact Name', store=True, string='Contact Name',
help='Name generated from the first name and last name', help='Name generated from the first name and last name',
nodrop=True), nodrop=True),
'private_phone':fields.char('Private phone', size=128),
'private_phone': fields.char('Private phone', size=128),
} }
def create(self, cursor, uid, vals, context={}): def create(self, cursor, uid, vals, context={}):
@ -189,9 +190,9 @@ class LDAPAddress(osv.osv):
ids = [ids] ids = [ids]
if ids: if ids:
self.validate_entries(vals, cursor, uid, ids) self.validate_entries(vals, cursor, uid, ids)
if context.has_key('init_mode') and context['init_mode'] :
if context.get('init_mode'):
success = True success = True
else :
else:
success = super(LDAPAddress, self).write(cursor, uid, ids, success = super(LDAPAddress, self).write(cursor, uid, ids,
vals, context) vals, context)
if self.ldaplinkactive(cursor, uid, context): if self.ldaplinkactive(cursor, uid, context):
@ -200,7 +201,8 @@ class LDAPAddress(osv.osv):
return success return success
def unlink(self, cursor, uid, ids, context=None): def unlink(self, cursor, uid, ids, context=None):
if not context: context = {}
if context is None:
context = {}
if ids: if ids:
self.getconn(cursor, uid, {}) self.getconn(cursor, uid, {})
if not isinstance(ids, list): if not isinstance(ids, list):
@ -212,77 +214,76 @@ class LDAPAddress(osv.osv):
def validate_entries(self, vals, cursor, uid, ids): def validate_entries(self, vals, cursor, uid, ids):
"""Validate data of an address based on the inetOrgPerson schema""" """Validate data of an address based on the inetOrgPerson schema"""
for val in vals :
try :
for val in vals:
try:
if isinstance(vals[val], basestring): if isinstance(vals[val], basestring):
vals[val] = unicode(vals[val].decode('utf8')) vals[val] = unicode(vals[val].decode('utf8'))
except UnicodeError: except UnicodeError:
logger.notifyChannel('LDAP encode', netsvc.LOG_DEBUG, logger.notifyChannel('LDAP encode', netsvc.LOG_DEBUG,
'cannot unicode '+ vals[val])
'cannot unicode ' + vals[val])
pass pass
if ids is not None: if ids is not None:
if isinstance(ids, (int, long)): if isinstance(ids, (int, long)):
ids = [ids] ids = [ids]
if len(ids) == 1: if len(ids) == 1:
self.addNeededFields(ids[0],vals,cursor,uid)
self.addNeededFields(ids[0], vals, cursor, uid)
email = vals.get('email', False) email = vals.get('email', False)
phone = vals.get('phone', False) phone = vals.get('phone', False)
fax = vals.get('fax', False) fax = vals.get('fax', False)
mobile = vals.get('mobile', False) mobile = vals.get('mobile', False)
lastname = vals.get('lastname', False)
private_phone = vals.get('private_phone', False) private_phone = vals.get('private_phone', False)
if email :
if email:
if re.match("^.+\\@(\\[?)[a-zA-Z0-9\\-\\.]+\\.([a-zA-Z]{2,3}|[0-9]{1,3})(\\]?)$", if re.match("^.+\\@(\\[?)[a-zA-Z0-9\\-\\.]+\\.([a-zA-Z]{2,3}|[0-9]{1,3})(\\]?)$",
email) is None: email) is None:
raise osv.except_osv(_('Warning !'),
raise orm.except_orm(_('Warning !'),
_('Please enter a valid e-mail')) _('Please enter a valid e-mail'))
phones = (('phone', phone), ('fax', fax), ('mobile', mobile), phones = (('phone', phone), ('fax', fax), ('mobile', mobile),
('private_phone', private_phone)) ('private_phone', private_phone))
for phone_tuple in phones: for phone_tuple in phones:
phone_number = phone_tuple[1] phone_number = phone_tuple[1]
if phone_number :
if phone_number:
if not phone_number.startswith('+'): if not phone_number.startswith('+'):
raise osv.except_osv(_('Warning !'),
raise orm.except_orm(_('Warning !'),
_('Please enter a valid phone number in %s' _('Please enter a valid phone number in %s'
' international format (i.e. leading +)') % phone_tuple[0]) ' international format (i.e. leading +)') % phone_tuple[0])
def getVals(self, att_name, key, vals, dico, uid, ids, cursor, context=None): def getVals(self, att_name, key, vals, dico, uid, ids, cursor, context=None):
"""map to values to dict""" """map to values to dict"""
if not context: context = {}
## We explicitely test False value
if vals.get(key, False) != False:
if not context:
context = {}
# We explicitly test False value
if vals.get(key, False) is not False:
dico[att_name] = vals[key] dico[att_name] = vals[key]
else :
else:
if context.get('init_mode'): if context.get('init_mode'):
return False return False
tmp = self.read(cursor, uid, ids, [key], context={}) tmp = self.read(cursor, uid, ids, [key], context={})
if tmp.get(key, False) :
if tmp.get(key, False):
dico[att_name] = tmp[key] dico[att_name] = tmp[key]
def _un_unicodize_buf(self, in_buf): def _un_unicodize_buf(self, in_buf):
if isinstance(in_buf, unicode) :
if isinstance(in_buf, unicode):
try: try:
return in_buf.encode() return in_buf.encode()
except Exception, e:
return unicodedata.normalize("NFKD", in_buf).encode('ascii','ignore')
except Exception:
return unicodedata.normalize("NFKD", in_buf).encode('ascii', 'ignore')
return in_buf return in_buf
def unUnicodize(self, indict) :
def unUnicodize(self, indict):
"""remove unicode data of modlist as unicode is not supported """remove unicode data of modlist as unicode is not supported
by python-ldap librairy till version 2.7"""
for key in indict :
by python-ldap library till version 2.7
"""
for key in indict:
if not isinstance(indict[key], list): if not isinstance(indict[key], list):
indict[key] = self._un_unicodize_buf(indict[key]) indict[key] = self._un_unicodize_buf(indict[key])
else: else:
nonutfArray = [] nonutfArray = []
for val in indict[key] :
for val in indict[key]:
nonutfArray.append(self._un_unicodize_buf(val)) nonutfArray.append(self._un_unicodize_buf(val))
indict[key] = nonutfArray indict[key] = nonutfArray
def addNeededFields(self, id, vals, cursor, uid): def addNeededFields(self, id, vals, cursor, uid):
keys = vals.keys()
previousvalue = self.browse(cursor, uid, [id])[0] previousvalue = self.browse(cursor, uid, [id])[0]
if not vals.get('partner_id'): if not vals.get('partner_id'):
vals['partner_id'] = previousvalue.partner_id.id vals['partner_id'] = previousvalue.partner_id.id
@ -296,142 +297,141 @@ class LDAPAddress(osv.osv):
"""Mapp ResPArtner adress to moddlist""" """Mapp ResPArtner adress to moddlist"""
self.addNeededFields(id, vals, cursor, uid) self.addNeededFields(id, vals, cursor, uid)
conn = self.getconn(cursor, uid, {}) conn = self.getconn(cursor, uid, {})
keys = vals.keys()
partner_obj=self.pool.get('res.partner')
partner_obj = self.pool.get('res.partner')
part_name = partner_obj.browse(cursor, uid, vals['partner_id']).name part_name = partner_obj.browse(cursor, uid, vals['partner_id']).name
vals['partner'] = part_name vals['partner'] = part_name
name = self._compute_name(vals.get('firstname'), vals.get('lastname')) name = self._compute_name(vals.get('firstname'), vals.get('lastname'))
if name :
if name:
cn = name cn = name
else: else:
cn = part_name cn = part_name
if not vals.get('lastname') :
if not vals.get('lastname'):
vals['lastname'] = part_name vals['lastname'] = part_name
contact_obj = {'objectclass' : ['inetOrgPerson'],
'uid': ['terp_'+str(id)],
'ou':[conn.OU],
'cn':[cn],
'sn':[vals['lastname']]}
contact_obj = {'objectclass': ['inetOrgPerson'],
'uid': ['terp_' + str(id)],
'ou': [conn.OU],
'cn': [cn],
'sn': [vals['lastname']]}
if not vals.get('street'): if not vals.get('street'):
vals['street'] = u'' vals['street'] = u''
if not vals.get('street2'): if not vals.get('street2'):
vals['street2'] = u'' vals['street2'] = u''
street_key = 'street' street_key = 'street'
if self.getconn(cursor, uid, {}).ACTIVDIR :
if self.getconn(cursor, uid, {}).ACTIVDIR:
# ENTERING THE M$ Realm and it is weird # ENTERING THE M$ Realm and it is weird
# We manage the address # We manage the address
street_key = 'streetAddress' street_key = 'streetAddress'
contact_obj[street_key] = vals['street'] + "\r\n" + vals['street2'] contact_obj[street_key] = vals['street'] + "\r\n" + vals['street2']
#we modifiy the class
contact_obj['objectclass'] = ['top','person','organizationalPerson','inetOrgPerson','user']
#we handle the country
if vals.get('country_id') :
country = self.browse(cursor, uid, id).country_id
if country :
# we modifiy the class
contact_obj['objectclass'] = ['top', 'person', 'organizationalPerson', 'inetOrgPerson', 'user']
# we handle the country
if vals.get('country_id'):
country = self.browse(cursor, uid, id, context=context).country_id
if country:
vals['country_id'] = country.name vals['country_id'] = country.name
vals['c'] = country.code vals['c'] = country.code
else :
else:
vals['country_id'] = False vals['country_id'] = False
vals['c'] = False vals['c'] = False
if vals.get('country_id', False) :
if vals.get('country_id', False):
self.getVals('co', 'country_id', vals, contact_obj, uid, id, cursor, context) self.getVals('co', 'country_id', vals, contact_obj, uid, id, cursor, context)
self.getVals('c', 'c', vals, contact_obj, uid, id, cursor, context) self.getVals('c', 'c', vals, contact_obj, uid, id, cursor, context)
# we compute the display name # we compute the display name
vals['display'] = '%s %s'%(vals['partner'], contact_obj['cn'][0])
vals['display'] = '%s %s' % (vals['partner'], contact_obj['cn'][0])
# we get the title # we get the title
if self.browse(cursor, uid, id).function: if self.browse(cursor, uid, id).function:
contact_obj['description'] = self.browse(cursor, uid, id).function.name contact_obj['description'] = self.browse(cursor, uid, id).function.name
# we replace carriage return # we replace carriage return
if vals.get('comment', False): if vals.get('comment', False):
vals['comment'] = vals['comment'].replace("\n","\r\n")
vals['comment'] = vals['comment'].replace("\n", "\r\n")
# Active directory specific fields # Active directory specific fields
self.getVals('company', 'partner' ,vals, contact_obj, uid, id, cursor, context)
self.getVals('info', 'comment' ,vals, contact_obj, uid, id, cursor, context)
self.getVals('displayName', 'partner' ,vals, contact_obj, uid, id, cursor, context)
## Web site management
self.getVals('company', 'partner', vals, contact_obj, uid, id, cursor, context)
self.getVals('info', 'comment', vals, contact_obj, uid, id, cursor, context)
self.getVals('displayName', 'partner', vals, contact_obj, uid, id, cursor, context)
# Web site management
if self.browse(cursor, uid, id).partner_id.website: if self.browse(cursor, uid, id).partner_id.website:
vals['website'] = self.browse(cursor, uid, id).partner_id.website vals['website'] = self.browse(cursor, uid, id).partner_id.website
self.getVals('wWWHomePage', 'website', vals, contact_obj, uid, id, cursor, context) self.getVals('wWWHomePage', 'website', vals, contact_obj, uid, id, cursor, context)
del(vals['website']) del(vals['website'])
self.getVals('title', 'title', vals, contact_obj, uid, id, cursor, context) self.getVals('title', 'title', vals, contact_obj, uid, id, cursor, context)
else :
else:
contact_obj[street_key] = vals['street'] + u"\n" + vals['street2'] contact_obj[street_key] = vals['street'] + u"\n" + vals['street2']
self.getVals('o','partner' ,vals, contact_obj, uid, id, cursor, context)
#Common attributes
self.getVals('givenName', 'firstname',vals, contact_obj, uid, id, cursor, context)
self.getVals('mail', 'email',vals, contact_obj, uid, id, cursor, context)
self.getVals('telephoneNumber', 'phone',vals, contact_obj, uid, id, cursor, context)
self.getVals('l', 'city',vals, contact_obj, uid, id, cursor, context)
self.getVals('facsimileTelephoneNumber', 'fax',vals, contact_obj, uid, id, cursor, context)
self.getVals('mobile', 'mobile',vals, contact_obj, uid, id, cursor, context)
self.getVals('homePhone', 'private_phone',vals, contact_obj, uid, id, cursor, context)
self.getVals('postalCode', 'zip',vals, contact_obj, uid, id, cursor, context)
self.getVals('o', 'partner', vals, contact_obj, uid, id, cursor, context)
# Common attributes
self.getVals('givenName', 'firstname', vals, contact_obj, uid, id, cursor, context)
self.getVals('mail', 'email', vals, contact_obj, uid, id, cursor, context)
self.getVals('telephoneNumber', 'phone', vals, contact_obj, uid, id, cursor, context)
self.getVals('l', 'city', vals, contact_obj, uid, id, cursor, context)
self.getVals('facsimileTelephoneNumber', 'fax', vals, contact_obj, uid, id, cursor, context)
self.getVals('mobile', 'mobile', vals, contact_obj, uid, id, cursor, context)
self.getVals('homePhone', 'private_phone', vals, contact_obj, uid, id, cursor, context)
self.getVals('postalCode', 'zip', vals, contact_obj, uid, id, cursor, context)
self.unUnicodize(contact_obj) self.unUnicodize(contact_obj)
return contact_obj return contact_obj
def saveLdapContact(self, id, vals, cursor, uid, context=None): def saveLdapContact(self, id, vals, cursor, uid, context=None):
"""save openerp adress to ldap""" """save openerp adress to ldap"""
contact_obj = self.mappLdapObject(id,vals,cursor,uid,context)
contact_obj = self.mappLdapObject(id, vals, cursor, uid, context)
conn = self.connectToLdap(cursor, uid, context=context) conn = self.connectToLdap(cursor, uid, context=context)
try: try:
if self.getconn(cursor, uid, context).ACTIVDIR: if self.getconn(cursor, uid, context).ACTIVDIR:
conn.connexion.add_s("CN=%s,OU=%s,%s"%(contact_obj['cn'][0], conn.OU, conn.CONTACT_DN),
conn.connexion.add_s("CN=%s,OU=%s,%s" % (contact_obj['cn'][0], conn.OU, conn.CONTACT_DN),
ldap.modlist.addModlist(contact_obj)) ldap.modlist.addModlist(contact_obj))
else: else:
conn.connexion.add_s("uid=terp_%s,OU=%s,%s"%(str(id), conn.OU, conn.CONTACT_DN),
ldap.modlist.addModlist(contact_obj))
except Exception, e:
raise e
conn.connexion.add_s("uid=terp_%s,OU=%s,%s" % (str(id), conn.OU, conn.CONTACT_DN),
ldap.modlist.addModlist(contact_obj))
except Exception:
raise
conn.connexion.unbind_s() conn.connexion.unbind_s()
def updateLdapContact(self, id, vals, cursor, uid, context): def updateLdapContact(self, id, vals, cursor, uid, context):
"""update an existing contact with the data of OpenERP""" """update an existing contact with the data of OpenERP"""
conn = self.connectToLdap(cursor,uid,context={})
conn = self.connectToLdap(cursor, uid, context={})
try: try:
old_contatc_obj = self.getLdapContact(conn,id)
old_contatc_obj = self.getLdapContact(conn, id)
except ldap.NO_SUCH_OBJECT: except ldap.NO_SUCH_OBJECT:
self.saveLdapContact(id,vals,cursor,uid,context)
self.saveLdapContact(id, vals, cursor, uid, context)
return return
contact_obj = self.mappLdapObject(id,vals,cursor,uid,context)
contact_obj = self.mappLdapObject(id, vals, cursor, uid, context)
if conn.ACTIVDIR: if conn.ACTIVDIR:
modlist = [] modlist = []
for key, val in contact_obj.items() :
for key, val in contact_obj.items():
if key in ('cn', 'uid', 'objectclass'): if key in ('cn', 'uid', 'objectclass'):
continue continue
if isinstance(val, list): if isinstance(val, list):
val = val[0] val = val[0]
modlist.append((ldap.MOD_REPLACE, key, val)) modlist.append((ldap.MOD_REPLACE, key, val))
else :
else:
modlist = ldap.modlist.modifyModlist(old_contatc_obj[1], contact_obj) modlist = ldap.modlist.modifyModlist(old_contatc_obj[1], contact_obj)
try: try:
conn.connexion.modify_s(old_contatc_obj[0], modlist) conn.connexion.modify_s(old_contatc_obj[0], modlist)
conn.connexion.unbind_s() conn.connexion.unbind_s()
except Exception, e:
raise e
except Exception:
raise
def removeLdapContact(self, id, cursor, uid): def removeLdapContact(self, id, cursor, uid):
"""Remove a contact from ldap""" """Remove a contact from ldap"""
conn = self.connectToLdap(cursor,uid,context={})
conn = self.connectToLdap(cursor, uid, context={})
to_delete = None to_delete = None
try: try:
to_delete = self.getLdapContact(conn,id)
to_delete = self.getLdapContact(conn, id)
except ldap.NO_SUCH_OBJECT: except ldap.NO_SUCH_OBJECT:
logger.notifyChannel("Warning", netsvc.LOG_INFO, logger.notifyChannel("Warning", netsvc.LOG_INFO,
_("'no object to delete in ldap' %s") %(id))
except Exception, e :
raise e
_("'no object to delete in ldap' %s") % (id))
except Exception:
raise
try: try:
if to_delete :
if to_delete:
conn.connexion.delete_s(to_delete[0]) conn.connexion.delete_s(to_delete[0])
conn.connexion.unbind_s() conn.connexion.unbind_s()
except Exception, e:
raise e
except Exception:
raise
def getLdapContact(self, conn, id): def getLdapContact(self, conn, id):
result = conn.connexion.search_ext_s("ou=%s,%s"%(conn.OU,conn.CONTACT_DN),
result = conn.connexion.search_ext_s("ou=%s, %s" % (conn.OU, conn.CONTACT_DN),
ldap.SCOPE_SUBTREE, ldap.SCOPE_SUBTREE,
"(&(objectclass=*)(uid=terp_"+str(id)+"))")
"(&(objectclass=*)(uid=terp_" + str(id) + "))")
if not result: if not result:
raise ldap.NO_SUCH_OBJECT raise ldap.NO_SUCH_OBJECT
return result[0] return result[0]
@ -439,21 +439,19 @@ class LDAPAddress(osv.osv):
def ldaplinkactive(self, cursor, uid, context=None): def ldaplinkactive(self, cursor, uid, context=None):
"""Check if ldap is activated for this company""" """Check if ldap is activated for this company"""
user = self.pool.get('res.users').browse(cursor, uid, uid, context=context) user = self.pool.get('res.users').browse(cursor, uid, uid, context=context)
company = self.pool.get('res.company').browse(cursor, uid,user.company_id.id, context=context)
company = self.pool.get('res.company').browse(cursor, uid, user.company_id.id, context=context)
return company.ldap_active return company.ldap_active
def getconn(self, cursor, uid, context=None): def getconn(self, cursor, uid, context=None):
"""LdapConnMApper""" """LdapConnMApper"""
if not self.ldapMapper :
if not self.ldapMapper:
self.ldapMapper = LdapConnMApper(cursor, uid, self) self.ldapMapper = LdapConnMApper(cursor, uid, self)
return self.ldapMapper return self.ldapMapper
def connectToLdap(self, cursor, uid, context=None): def connectToLdap(self, cursor, uid, context=None):
"""Reinitialize ldap connection""" """Reinitialize ldap connection"""
#getting ldap pref
if not self.ldapMapper :
# getting ldap pref
if not self.ldapMapper:
self.getconn(cursor, uid, context) self.getconn(cursor, uid, context)
self.ldapMapper.get_connexion() self.ldapMapper.get_connexion()
return self.ldapMapper return self.ldapMapper
LDAPAddress()

25
partner_address_ldap/company.py

@ -31,35 +31,36 @@
# #
############################################################################## ##############################################################################
from osv import osv, fields
from openerp.osv import orm, fields
class Res_company(osv.osv):
"""Defin ldap connexion parameters"""
class Res_company(orm.Model):
"""Define ldap connexion parameters"""
_inherit = 'res.company' _inherit = 'res.company'
_columns = { _columns = {
'base_dn': fields.char( 'base_dn': fields.char(
'User dn', 'User dn',
size=128, size=128,
help="Exemple: cn=contacts_admin,dc=ldap,dc=dcc2c"
help="Example: cn=contacts_admin,dc=ldap,dc=dcc2c"
), ),
'contact_dn': fields.char( 'contact_dn': fields.char(
'Bind dn', 'Bind dn',
size=128, size=128,
help="Exemple: dc=ldap,dc=dcc2c -- watchout" +\
" the OU will be automatically included inside"
help="Example: dc=ldap,dc=dcc2c -- watch out "
"the OU will be automatically included inside"
), ),
'ounit': fields.char( 'ounit': fields.char(
'Contact Organizational unit of the contacts', 'Contact Organizational unit of the contacts',
size=128, size=128,
help="Exemple: Contacts"
help="Example: Contacts"
), ),
'ldap_server': fields.char( 'ldap_server': fields.char(
'Server address', 'Server address',
size=128, size=128,
help="Exemple: ldap.camptocamp.com"
help="Example: ldap.camptocamp.com"
), ),
'passwd': fields.char('ldap password', size=128, help="Exemple: Mypassword1234"),
'passwd': fields.char('ldap password', size=128, help="Example: Mypassword1234"),
'ldap_active': fields.boolean( 'ldap_active': fields.boolean(
'Activate ldap link for this company', 'Activate ldap link for this company',
help='If not check nothing will be reported into the ldap' help='If not check nothing will be reported into the ldap'
@ -69,8 +70,6 @@ class Res_company(osv.osv):
help='The ldap is part of an Active Directory' help='The ldap is part of an Active Directory'
), ),
'ldap_port': fields.integer('LDAP Port', 'ldap_port': fields.integer('LDAP Port',
help="If not specified, the default port"
"(389), will be used")
help="If not specified, the default port"
"(389), will be used")
} }
Res_company()

7
partner_address_ldap/partner.py

@ -31,9 +31,10 @@
# #
############################################################################## ##############################################################################
from osv import osv, fields
from openerp.osv import orm
class LdapPartner(osv.osv):
class LdapPartner(orm.Model):
"""Ensure that when deleting a partner unlink function is called on all """Ensure that when deleting a partner unlink function is called on all
related addresses""" related addresses"""
_inherit = 'res.partner' _inherit = 'res.partner'
@ -46,5 +47,3 @@ class LdapPartner(osv.osv):
addr_ids = addr_obj.search(cursor, uid, [('partner_id', 'in', ids)]) addr_ids = addr_obj.search(cursor, uid, [('partner_id', 'in', ids)])
addr_obj.unlink(cursor, uid, addr_ids, context=context) addr_obj.unlink(cursor, uid, addr_ids, context=context)
return super(LdapPartner, self).unlink(cursor, uid, ids, context=context) return super(LdapPartner, self).unlink(cursor, uid, ids, context=context)
LdapPartner()

120
partner_address_ldap/wizard/wiz_import_adresses.py

@ -35,14 +35,15 @@
# Copyright (c) 2009 CamptoCamp. All rights reserved. # Copyright (c) 2009 CamptoCamp. All rights reserved.
# #
import wizard
import pooler
import base64 import base64
import unicodedata import unicodedata
import netsvc
import re import re
import wizard
from openerp import pooler
from openerp import netsvc
_FORM = '''<?xml version="1.0"?> _FORM = '''<?xml version="1.0"?>
<form string="Export adresses to ldap">
<form string="Export addresses to ldap">
</form>''' </form>'''
@ -61,99 +62,100 @@ _FIELDS = {
}, },
} }
### As this is a bulck batch wizzard the performance process was not reallay taken in account ###
##The ideal way of doing would be to modify the connexion settings in order to have a connexion singelton
## in the file partner.py it will avoid connexion renegotiation for each partner.
def _action_import_adresses(self, cr, uid, data, context):
""" This function create or update each adresses present in the database.
It will also genreate an error report"""
# As this is a bulk batch wizard the performance process was not really taken in account ###
# The ideal way of doing would be to modify the connexion settings in order to have a connexion singleton
# in the file partner.py it will avoid connexion renegotiation for each partner.
def _action_import_addresses(self, cr, uid, data, context):
""" This function create or update each addresses present in the database.
It will also generate an error report"""
logger = netsvc.Logger() logger = netsvc.Logger()
error_report = [u'Error report'] error_report = [u'Error report']
add_obj = pooler.get_pool(cr.dbname).get('res.partner.address') add_obj = pooler.get_pool(cr.dbname).get('res.partner.address')
add_ids = add_obj.search(cr,uid,[])
add_ids = add_obj.search(cr, uid, [])
addresses = add_obj.browse(cr, uid, add_ids) addresses = add_obj.browse(cr, uid, add_ids)
phone_fields = ['phone','fax','mobile','private_phone']
for add in addresses :
vals = {}
vals['partner_id'] = add.partner_id.id
vals['email'] = add.email
vals['phone'] = add.phone
vals['fax'] = add.fax
vals['mobile'] = add.mobile
vals['firstname'] = add.firstname
vals['lastname'] = add.lastname
vals['private_phone'] = add.private_phone
vals['street'] = add.street
vals['street2'] = add.street2
vals['city'] = add.city
phone_fields = ['phone', 'fax', 'mobile', 'private_phone']
for add in addresses:
vals = {
'partner_id': add.partner_id.id,
'email': add.email,
'phone': add.phone,
'fax': add.fax,
'mobile': add.mobile,
'firstname': add.firstname,
'lastname': add.lastname,
'private_phone': add.private_phone,
'street': add.street,
'street2': add.street2,
'city': add.city,
}
# Validating the mail # Validating the mail
if add.email :
if add.email:
if re.match( if re.match(
"^.+\\@(\\[?)[a-zA-Z0-9\\-\\.]+\\.([a-zA-Z]{2,3}|[0-9]{1,3})(\\]?)$", add.email) is None or\ "^.+\\@(\\[?)[a-zA-Z0-9\\-\\.]+\\.([a-zA-Z]{2,3}|[0-9]{1,3})(\\]?)$", add.email) is None or\
re.search(u"[éèàêöüäï&]", add.email) is not None: re.search(u"[éèàêöüäï&]", add.email) is not None:
msg=u'Addresse %s for partner %s has email that is invalid %s'%(
unicode(vals['firstname']) +' '+ unicode(vals['lastname']),
msg = u'Address %s for partner %s has email that is invalid %s' % (
unicode(vals['firstname']) + ' ' + unicode(vals['lastname']),
add.partner_id.name, add.partner_id.name,
unicode(add.email) unicode(add.email)
)
)
logger.notifyChannel('ldap export', netsvc.LOG_INFO, msg) logger.notifyChannel('ldap export', netsvc.LOG_INFO, msg)
error_report.append(msg) error_report.append(msg)
vals['email'] = False vals['email'] = False
# Validating the Phone # Validating the Phone
for key in phone_fields :
if not unicode(vals[key]).startswith('+') or unicode(vals[key]).find("\n") != -1\
or re.search(u"[éèàêöüä#&]", unicode(vals[key])) is not None:
for key in phone_fields:
if (not unicode(vals[key]).startswith('+') or unicode(vals[key]).find("\n") != -1
or re.search(u"[éèàêöüä#&]", unicode(vals[key])) is not None):
vals[key] = False vals[key] = False
msg = u'Addresse %s for partner %s has %s that is invalid '%(
unicode(vals['firstname']) +' '+ unicode(vals['lastname']),
msg = u'Addresse %s for partner %s has %s that is invalid ' % (
unicode(vals['firstname']) + ' ' + unicode(vals['lastname']),
add.partner_id.name, add.partner_id.name,
key key
)
)
logger.notifyChannel('ldap export', netsvc.LOG_INFO, msg) logger.notifyChannel('ldap export', netsvc.LOG_INFO, msg)
error_report.append(msg) error_report.append(msg)
# Validating the CN # Validating the CN
if not add.lastname and add.firstname: if not add.lastname and add.firstname:
msg = u'!!! Addresse %s for partner %s has no last name and first name that is valid partner name was used'%(
unicode(add.id),
add.partner_id.name,
)
msg = (u'!!! Address %s for partner %s has no last name and first name that is valid partner name was used'
% (unicode(add.id), add.partner_id.name))
logger.notifyChannel('ldap export', netsvc.LOG_INFO, msg) logger.notifyChannel('ldap export', netsvc.LOG_INFO, msg)
error_report.append(msg) error_report.append(msg)
# We save to LDAP # We save to LDAP
add.write(vals, {'init_mode':True})
#we by pass the encoding errors
map(lambda x: unicodedata.normalize("NFKD",x).encode('ascii','ignore'), error_report)
add.write(vals, {'init_mode': True})
# we by pass the encoding errors
map(lambda x: unicodedata.normalize("NFKD", x).encode('ascii', 'ignore'), error_report)
error_report = "\n".join(error_report) error_report = "\n".join(error_report)
logger.notifyChannel("MY TOPIC", netsvc.LOG_ERROR, error_report) logger.notifyChannel("MY TOPIC", netsvc.LOG_ERROR, error_report)
try: try:
data= base64.encodestring(error_report.encode())
except Exception, e:
data= base64.encodestring("Could not generate report file. Please look in the log for details")
data = base64.encodestring(error_report.encode())
except:
data = base64.encodestring("Could not generate report file. Please look in the log for details")
return {'errors': data} return {'errors': data}
class Wiz_import_addresses(wizard.interface): class Wiz_import_addresses(wizard.interface):
states = { states = {
'init': { 'init': {
'actions': [], 'actions': [],
'result': { 'result': {
'type': 'form',
'arch':_FORM,
'fields':{},
'state':[
('end','Cancel'),
('importadd','Export adresses into company LDAP')
]
}
'type': 'form',
'arch': _FORM,
'fields': {},
'state': [
('end', 'Cancel'),
('importadd', 'Export adresses into company LDAP')
]
}
}, },
'importadd': { 'importadd': {
'actions': [_action_import_adresses],
'actions': [_action_import_addresses],
'result': { 'result': {
'state':[('end', 'OK', 'gtk-ok', True)],
'arch' : _FORM1,
'fields' : _FIELDS,
'type':'form'
}
'state': [('end', 'OK', 'gtk-ok', True)],
'arch': _FORM1,
'fields': _FIELDS,
'type': 'form'
}
} }
} }
Wiz_import_addresses('ldap.import_adresses') Wiz_import_addresses('ldap.import_adresses')

1
partner_contact_address_detailed/__init__.py

@ -17,4 +17,3 @@
# along with this program. If not, see <http://www.gnu.org/licenses/>. # along with this program. If not, see <http://www.gnu.org/licenses/>.
# #
############################################################################## ##############################################################################

29
partner_contact_address_detailed/__openerp__.py

@ -18,18 +18,19 @@
# #
############################################################################## ##############################################################################
{'name': 'All address data in summarized contact form',
'description': """Show all informations of partner address
{
'name': 'All address data in summarized contact form',
'description': """Show all informations of partner address
in summarized partner contact form when creating a new partner""", in summarized partner contact form when creating a new partner""",
'version': '1.0',
'author': 'Camptocamp',
'category': 'MISC',
'website': 'http://www.camptocamp.com',
'depends': ['base'],
'data': ['partner_view.xml'],
'demo': [],
'test': [],
'auto_install': False,
'installable': True,
'images': []
}
'version': '1.0',
'author': 'Camptocamp',
'category': 'MISC',
'website': 'http://www.camptocamp.com',
'depends': ['base'],
'data': ['partner_view.xml'],
'demo': [],
'test': [],
'auto_install': False,
'installable': True,
'images': [],
}

35
partner_firstname/__openerp__.py

@ -18,8 +18,9 @@
# #
############################################################################## ##############################################################################
{'name': 'Partner first name, last name',
'description': """
{
'name': 'Partner first name, last name',
'description': """
This module splits first name and last name for non company partners This module splits first name and last name for non company partners
==================================================================== ====================================================================
@ -32,19 +33,19 @@ Jonathan Nemry <jonathan.nemry@acsone.eu>
Olivier Laurent <olivier.laurent@acsone.eu> Olivier Laurent <olivier.laurent@acsone.eu>
""", """,
'version': '1.2',
'author': 'Camptocamp',
'maintainer': 'Camptocamp, Acsone',
'category': 'Extra Tools',
'website': 'http://www.camptocamp.com, http://www.acsone.eu',
'depends': ['base'],
'data': [
'partner_view.xml',
'res_user_view.xml',
],
'demo': [],
'test': [],
'auto_install': False,
'installable': True,
'images': []
'version': '1.2',
'author': 'Camptocamp',
'maintainer': 'Camptocamp, Acsone',
'category': 'Extra Tools',
'website': 'http://www.camptocamp.com, http://www.acsone.eu',
'depends': ['base'],
'data': [
'partner_view.xml',
'res_user_view.xml',
],
'demo': [],
'test': [],
'auto_install': False,
'installable': True,
'images': []
} }

4
partner_firstname/partner.py

@ -83,7 +83,9 @@ class ResPartner(Model):
default = default or {} default = default or {}
if not default.get('lastname'): if not default.get('lastname'):
default = default.copy() default = default.copy()
default['lastname'] = _('%s (copy)') % self.read(cr, uid, [_id], ['lastname'], context=context)[0]['lastname']
default['lastname'] = (
_('%s (copy)') % self.read(cr, uid, [_id], ['lastname'], context=context)[0]['lastname']
)
if default.get('name'): if default.get('name'):
del(default['name']) del(default['name'])
return super(ResPartner, self).copy_data(cr, uid, _id, default, context=context) return super(ResPartner, self).copy_data(cr, uid, _id, default, context=context)

4
partner_firstname/res_user.py

@ -32,7 +32,9 @@ class ResUsers(orm.Model):
default = default or {} default = default or {}
if not default.get('lastname'): if not default.get('lastname'):
default = default.copy() default = default.copy()
default['lastname'] = _('%s (copy)') % self.read(cr, uid, [_id], ['lastname'], context=context)[0]['lastname']
default['lastname'] = (
_('%s (copy)') % self.read(cr, uid, [_id], ['lastname'], context=context)[0]['lastname']
)
if default.get('name'): if default.get('name'):
del(default['name']) del(default['name'])
return super(ResUsers, self).copy_data(cr, uid, _id, default, context=context) return super(ResUsers, self).copy_data(cr, uid, _id, default, context=context)

4
partner_firstname/tests/__init__.py

@ -32,6 +32,4 @@ import test_partner_firstname
checks = [ checks = [
test_partner_firstname test_partner_firstname
]
# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
]

26
partner_firstname/tests/test_partner_firstname.py

@ -53,8 +53,16 @@ class test_partner_firstname(common.TransactionCase):
res_id = self.partner_model.copy(cr, uid, res_id, default={}, context=context) res_id = self.partner_model.copy(cr, uid, res_id, default={}, context=context)
vals = self.partner_model.read(cr, uid, [res_id], ['name', 'lastname', 'firstname'], context=context)[0] vals = self.partner_model.read(cr, uid, [res_id], ['name', 'lastname', 'firstname'], context=context)[0]
self.assertEqual(vals['name'], _('%s (copy)') % 'lastname' + " firstname", 'Copy of the partner failed with wrong name')
self.assertEqual(vals['lastname'], _('%s (copy)') % 'lastname', 'Copy of the partner failed with wrong lastname')
self.assertEqual(
vals['name'],
_('%s (copy)') % 'lastname' + " firstname",
'Copy of the partner failed with wrong name'
)
self.assertEqual(
vals['lastname'],
_('%s (copy)') % 'lastname',
'Copy of the partner failed with wrong lastname'
)
self.assertEqual(vals['firstname'], 'firstname', 'Copy of the partner failed with wrong firstname') self.assertEqual(vals['firstname'], 'firstname', 'Copy of the partner failed with wrong firstname')
def test_copy_user(self): def test_copy_user(self):
@ -63,12 +71,16 @@ class test_partner_firstname(common.TransactionCase):
res_id = self.user_model.create(cr, uid, self.fields_user, context=context) res_id = self.user_model.create(cr, uid, self.fields_user, context=context)
# get the related partner id and add it a firstname # get the related partner id and add it a firstname
flds = self.user_model.read(cr, uid, [res_id], ['partner_id'], context=context)[0] flds = self.user_model.read(cr, uid, [res_id], ['partner_id'], context=context)[0]
self.partner_model.write(cr, uid, flds['partner_id'][0], {'firstname':'firstname'}, context=context)
self.partner_model.write(cr, uid, flds['partner_id'][0], {'firstname': 'firstname'}, context=context)
# copy the user and compare result # copy the user and compare result
res_id = self.user_model.copy(cr, uid, res_id, default={}, context=context) res_id = self.user_model.copy(cr, uid, res_id, default={}, context=context)
vals = self.user_model.read(cr, uid, [res_id], ['name', 'lastname', 'firstname'], context=context)[0] vals = self.user_model.read(cr, uid, [res_id], ['name', 'lastname', 'firstname'], context=context)[0]
self.assertEqual(vals['name'], _('%s (copy)') % 'lastname' + ' firstname', 'Copy of the user failed with wrong name')
self.assertEqual(
vals['name'],
_('%s (copy)') % 'lastname' + ' firstname',
'Copy of the user failed with wrong name'
)
self.assertEqual(vals['lastname'], _('%s (copy)') % 'lastname', 'Copy of the user failed with wrong lastname') self.assertEqual(vals['lastname'], _('%s (copy)') % 'lastname', 'Copy of the user failed with wrong lastname')
self.assertEqual(vals['firstname'], 'firstname', 'Copy of the user failed with wrong firstname') self.assertEqual(vals['firstname'], 'firstname', 'Copy of the user failed with wrong firstname')
@ -78,7 +90,7 @@ class test_partner_firstname(common.TransactionCase):
res_id = self.user_model.create(cr, uid, self.fields_user, context=context) res_id = self.user_model.create(cr, uid, self.fields_user, context=context)
# get the related partner id and add it a firstname # get the related partner id and add it a firstname
flds = self.user_model.read(cr, uid, [res_id], ['partner_id'], context=context)[0] flds = self.user_model.read(cr, uid, [res_id], ['partner_id'], context=context)[0]
self.partner_model.write(cr, uid, flds['partner_id'][0], {'firstname':'firstname'}, context=context)
self.partner_model.write(cr, uid, flds['partner_id'][0], {'firstname': 'firstname'}, context=context)
self.user_model.write(cr, uid, res_id, {'name': 'change firstname'}, context=context) self.user_model.write(cr, uid, res_id, {'name': 'change firstname'}, context=context)
vals = self.user_model.read(cr, uid, [res_id], ['name', 'lastname', 'firstname'], context=context)[0] vals = self.user_model.read(cr, uid, [res_id], ['name', 'lastname', 'firstname'], context=context)[0]
@ -92,12 +104,10 @@ class test_partner_firstname(common.TransactionCase):
res_id = self.user_model.create(cr, uid, self.fields_user, context=context) res_id = self.user_model.create(cr, uid, self.fields_user, context=context)
# get the related partner id and add it a firstname # get the related partner id and add it a firstname
flds = self.user_model.read(cr, uid, [res_id], ['partner_id'], context=context)[0] flds = self.user_model.read(cr, uid, [res_id], ['partner_id'], context=context)[0]
self.partner_model.write(cr, uid, flds['partner_id'][0], {'firstname':'firstname'}, context=context)
self.partner_model.write(cr, uid, flds['partner_id'][0], {'firstname': 'firstname'}, context=context)
self.user_model.write(cr, uid, res_id, {'name': 'lastname other'}, context=context) self.user_model.write(cr, uid, res_id, {'name': 'lastname other'}, context=context)
vals = self.user_model.read(cr, uid, [res_id], ['name', 'lastname', 'firstname'], context=context)[0] vals = self.user_model.read(cr, uid, [res_id], ['name', 'lastname', 'firstname'], context=context)[0]
self.assertEqual(vals['name'], 'lastname other', 'Update of the user firstname failed with wrong name') self.assertEqual(vals['name'], 'lastname other', 'Update of the user firstname failed with wrong name')
self.assertEqual(vals['lastname'], 'lastname other', 'Update of the user firstname failed with wrong lastname') self.assertEqual(vals['lastname'], 'lastname other', 'Update of the user firstname failed with wrong lastname')
self.assertFalse(vals['firstname'], 'Update of the user firstname failed with wrong firstname') self.assertFalse(vals['firstname'], 'Update of the user firstname failed with wrong firstname')
# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

8
portal_partner_merge/__openerp__.py

@ -18,16 +18,16 @@
# along with this program. If not, see <http://www.gnu.org/licenses/>. # along with this program. If not, see <http://www.gnu.org/licenses/>.
# #
############################################################################## ##############################################################################
{'name' : 'Portal Partner Merge',
'version' : '1.0',
{'name': 'Portal Partner Merge',
'version': '1.0',
'category': 'Hidden', 'category': 'Hidden',
'description': """ 'description': """
Link module for base_partner_merge which extract portal dependency Link module for base_partner_merge which extract portal dependency
""", """,
'author' : 'Camptocamp',
'author': 'Camptocamp',
'maintainer': 'Camptocamp', 'maintainer': 'Camptocamp',
'website': 'http://www.camptocamp.com/', 'website': 'http://www.camptocamp.com/',
'depends' : ['portal', 'base_partner_merge'],
'depends': ['portal', 'base_partner_merge'],
'data': [], 'data': [],
'test': [], 'test': [],
'installable': True, 'installable': True,

7
portal_partner_merge/wizard/portal_wizard.py

@ -19,10 +19,11 @@
# #
############################################################################## ##############################################################################
from openerp.osv import osv
from openerp.osv import orm
from openerp.tools.translate import _ from openerp.tools.translate import _
class wizard_user(osv.TransientModel):
class wizard_user(orm.TransientModel):
_inherit = 'portal.wizard.user' _inherit = 'portal.wizard.user'
def get_error_messages(self, cr, uid, ids, context=None): def get_error_messages(self, cr, uid, ids, context=None):
@ -35,5 +36,3 @@ class wizard_user(osv.TransientModel):
"Merge wizard, available in the More menu after selecting " "Merge wizard, available in the More menu after selecting "
"several contacts in the Customers list")) "several contacts in the Customers list"))
return error_msg return error_msg
# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

41
res_partner_affiliate/__openerp__.py

@ -18,25 +18,22 @@
# along with this program. If not, see <http://www.gnu.org/licenses/>. # along with this program. If not, see <http://www.gnu.org/licenses/>.
# #
############################################################################## ##############################################################################
{'name': 'Partner Affilates',
'version': '1.0',
'author': 'Camptocamp',
'maintainer': 'Camptocamp',
'category': 'CRM',
'complexity': 'normal', # easy, normal, expert
'depends': ['base'],
'description': """Allows to use parent_id in company partner to refer to a parent company
Plus will show a tab in parent company of affiliated companies""",
'website': 'http://www.camptocamp.com',
'init_xml': [],
'update_xml': [
'res_partner_view.xml',
],
'demo_xml': [],
'tests': [],
'installable': True,
'images': [],
'auto_install': False,
'license': 'AGPL-3',
'application': True}
{
'name': 'Partner Affiliates',
'version': '1.0',
'author': 'Camptocamp',
'maintainer': 'Camptocamp',
'category': 'CRM',
'license': 'AGPL-3',
'depends': ['base'],
'description': """Allows to use parent_id in company partner to refer to a parent company
Plus will show a tab in parent company of affiliated companies""",
'website': 'http://www.camptocamp.com',
'data': [
'res_partner_view.xml',
],
'demo': [],
'installable': True,
'auto_install': False,
'application': True
}

4
res_partner_affiliate/res_partner.py

@ -18,8 +18,10 @@
# along with this program. If not, see <http://www.gnu.org/licenses/>. # along with this program. If not, see <http://www.gnu.org/licenses/>.
# #
############################################################################## ##############################################################################
from openerp.osv import orm, fields from openerp.osv import orm, fields
class ResPartner(orm.Model): class ResPartner(orm.Model):
""" """
Add relation affiliate_ids Add relation affiliate_ids
@ -34,4 +36,4 @@ class ResPartner(orm.Model):
'affiliate_ids': fields.one2many( 'affiliate_ids': fields.one2many(
'res.partner', 'parent_id', 'res.partner', 'parent_id',
'Affiliates', domain=[('is_company', '=', True)]), 'Affiliates', domain=[('is_company', '=', True)]),
}
}
Loading…
Cancel
Save