From ca0f8598f614f79db8d4caeeb62424dcfa856f98 Mon Sep 17 00:00:00 2001
From: Jigar Amin - OpenERP <jam@tinyerp.com>
Date: Tue, 6 Mar 2012 17:38:33 +0530
Subject: [PATCH 1/7] [IMP] crm claim : removed crm clamin partner address

bzr revid: jam@tinyerp.com-20120306120833-90x9fl5dvpsg8vvp
---
 addons/crm_claim/crm_claim.py       | 26 +++++---------------------
 addons/crm_claim/crm_claim_demo.xml |  5 -----
 addons/crm_claim/crm_claim_view.xml | 12 ------------
 3 files changed, 5 insertions(+), 38 deletions(-)

diff --git a/addons/crm_claim/crm_claim.py b/addons/crm_claim/crm_claim.py
index fb6b8f90bda..ef56c97deea 100644
--- a/addons/crm_claim/crm_claim.py
+++ b/addons/crm_claim/crm_claim.py
@@ -70,9 +70,6 @@ class crm_claim(crm.crm_case, osv.osv):
                                 " mail gateway."),
         'company_id': fields.many2one('res.company', 'Company'),
         'partner_id': fields.many2one('res.partner', 'Partner'),
-        'partner_address_id': fields.many2one('res.partner.address', 'Partner Contact', \
-                                # domain="[('partner_id','=',partner_id)]"
-                                 ),
         'email_cc': fields.text('Watchers Emails', size=252, help="These email addresses will be added to the CC field of all inbound and outbound emails for this record before being sent. Separate multiple email addresses with a comma"),
         'email_from': fields.char('Email', size=128, help="These people will receive email."),
         'partner_phone': fields.char('Phone', size=32),
@@ -89,7 +86,6 @@ class crm_claim(crm.crm_case, osv.osv):
     _defaults = {
         'user_id': crm.crm_case._get_default_user,
         'partner_id': crm.crm_case._get_default_partner,
-        'partner_address_id': crm.crm_case._get_default_partner_address,
         'email_from':crm.crm_case. _get_default_email,
         'state': lambda *a: 'draft',
         'section_id':crm.crm_case. _get_section,
@@ -105,23 +101,11 @@ class crm_claim(crm.crm_case, osv.osv):
            :param email: ignored
         """
         if not part:
-            return {'value': {'partner_address_id': False,
-                            'email_from': False,
-                            'partner_phone': False
-                            }}
-        addr = self.pool.get('res.partner').address_get(cr, uid, [part], ['contact'])
-        data = {'partner_address_id': addr['contact']}
-        data.update(self.onchange_partner_address_id(cr, uid, ids, addr['contact'])['value'])
-        return {'value': data}
-
-    def onchange_partner_address_id(self, cr, uid, ids, add, email=False):
-        """This function returns value of partner email based on Partner Address
-           :param part: Partner's id
-           :param email: ignored
-        """
-        if not add:
-            return {'value': {'email_from': False}}
-        address = self.pool.get('res.partner.address').browse(cr, uid, add)
+            return {'value': {'email_from': False,
+                              'partner_phone': False
+                            }
+                   }
+        address = self.pool.get('res.partner.address').browse(cr, uid, part)
         return {'value': {'email_from': address.email, 'partner_phone': address.phone}}
 
     def case_open(self, cr, uid, ids, *args):
diff --git a/addons/crm_claim/crm_claim_demo.xml b/addons/crm_claim/crm_claim_demo.xml
index 8279e04eb6f..592e655b489 100644
--- a/addons/crm_claim/crm_claim_demo.xml
+++ b/addons/crm_claim/crm_claim_demo.xml
@@ -7,7 +7,6 @@
     -->
     
         <record id="crm_case_claim01" model="crm.claim">
-            <field name="partner_address_id" ref="base.res_partner_address_15"/>
             <field eval="time.strftime('%Y-%m-04 10:45:36')" name="date"/>
             <field name="partner_id" ref="base.res_partner_11"/>
             <field eval="&quot;3&quot;" name="priority"/>
@@ -21,7 +20,6 @@
         </record>
         
         <record id="crm_case_claim02" model="crm.claim">
-            <field name="partner_address_id" ref="base.res_partner_address_6"/>
             <field eval="time.strftime('%Y-%m-11 11:19:25')" name="date"/>
             <field name="partner_id" ref="base.res_partner_6"/>
             <field eval="&quot;4&quot;" name="priority"/>
@@ -35,7 +33,6 @@
         </record>
         
         <record id="crm_case_claim03" model="crm.claim">
-            <field name="partner_address_id" ref="base.res_partner_address_2"/>
             <field eval="time.strftime('%Y-%m-15 17:44:12')" name="date"/>
             <field name="partner_id" ref="base.res_partner_10"/>
             <field eval="&quot;2&quot;" name="priority"/>
@@ -63,7 +60,6 @@
         </record>
         
         <record id="crm_case_claim05" model="crm.claim">
-            <field name="partner_address_id" ref="base.res_partner_address_10"/>
             <field eval="time.strftime('%Y-%m-28 16:20:43')" name="date"/>
             <field name="partner_id" ref="base.res_partner_5"/>
             <field eval="&quot;3&quot;" name="priority"/>
@@ -77,7 +73,6 @@
         </record>
         
         <record id="crm_case_claim06" model="crm.claim">
-            <field name="partner_address_id" ref="base.res_partner_address_1"/>
             <field name="partner_id" ref="base.res_partner_9"/>
             <field eval="&quot;3&quot;" name="priority"/>
             <field name="user_id" ref="base.user_root"/>
diff --git a/addons/crm_claim/crm_claim_view.xml b/addons/crm_claim/crm_claim_view.xml
index 6bb7e089baa..edeb43a8801 100644
--- a/addons/crm_claim/crm_claim_view.xml
+++ b/addons/crm_claim/crm_claim_view.xml
@@ -90,8 +90,6 @@
                                     <separator colspan="2" string="Claim Reporter"/>
                                     <field name="partner_id" string="Partner"
                                         on_change="onchange_partner_id(partner_id)" />
-                                    <field name="partner_address_id" string="Contact"
-                                        on_change="onchange_partner_address_id(partner_address_id, email_from)" />
                                     <field name="partner_phone"/>
                                     <field name="email_from" widget="email"/>
                                 </group>
@@ -289,15 +287,5 @@
             res_model="crm.claim"
             src_model="res.partner"/>
 
-
-        <act_window
-            domain="[('partner_address_id', '=', active_id)]"
-            context="{'default_partner_address_id': active_id}"
-            id="act_claim_partner_address"
-            name="Claims"
-            view_mode="tree,form"
-            res_model="crm.claim"
-            src_model="res.partner.address"/>
-
      </data>
 </openerp>

From 1e966e5409d6407570d73017f5fe6e2394b3732b Mon Sep 17 00:00:00 2001
From: Jigar Amin - OpenERP <jam@tinyerp.com>
Date: Tue, 6 Mar 2012 18:04:12 +0530
Subject: [PATCH 2/7] [IMP] crm claim : removed crm fundraising partner address

bzr revid: jam@tinyerp.com-20120306123412-olvufv1c25rcoo08
---
 addons/crm_fundraising/crm_fundraising.py       | 3 ---
 addons/crm_fundraising/crm_fundraising_demo.xml | 8 --------
 addons/crm_fundraising/crm_fundraising_view.xml | 4 ----
 3 files changed, 15 deletions(-)

diff --git a/addons/crm_fundraising/crm_fundraising.py b/addons/crm_fundraising/crm_fundraising.py
index 37c5bb8ccee..76d0ca8fdd5 100644
--- a/addons/crm_fundraising/crm_fundraising.py
+++ b/addons/crm_fundraising/crm_fundraising.py
@@ -47,8 +47,6 @@ class crm_fundraising(crm.crm_case, osv.osv):
                             select=True, help='Sales team to which Case belongs to. Define Responsible user and Email account for mail gateway.'),
             'company_id': fields.many2one('res.company', 'Company'),
             'partner_id': fields.many2one('res.partner', 'Partner'),
-            'partner_address_id': fields.many2one('res.partner.address', 'Partner Contact', \
-                                 domain="[('partner_id','=',partner_id)]"),
             'email_cc': fields.text('Watchers Emails', size=252 , help="These email addresses will be added to the CC field of all inbound and outbound emails for this record before being sent. Separate multiple email addresses with a comma"),
             'email_from': fields.char('Email', size=128, help="These people will receive email."),
             'date_closed': fields.datetime('Closed', readonly=True),
@@ -100,7 +98,6 @@ class crm_fundraising(crm.crm_case, osv.osv):
             'active': 1,
             'user_id': crm.crm_case._get_default_user,
             'partner_id': crm.crm_case._get_default_partner,
-            'partner_address_id': crm.crm_case._get_default_partner_address,
             'email_from': crm.crm_case. _get_default_email,
             'state': 'draft',
             'section_id': crm.crm_case. _get_section,
diff --git a/addons/crm_fundraising/crm_fundraising_demo.xml b/addons/crm_fundraising/crm_fundraising_demo.xml
index c7588701af0..52b4094d126 100644
--- a/addons/crm_fundraising/crm_fundraising_demo.xml
+++ b/addons/crm_fundraising/crm_fundraising_demo.xml
@@ -3,7 +3,6 @@
     <data noupdate="1">
         <record id="crm_case_helpingstreetchildren0" model="crm.fundraising">
             <field eval="50" name="probability"/>
-            <field name="partner_address_id" ref="base.res_partner_address_1"/>
             <field eval="1" name="active"/>
             <field name="type_id" ref="type_fund3"/>
             <field eval="3.0" name="duration"/>
@@ -22,7 +21,6 @@
     <data noupdate="1">
         <record id="crm_case_helpingearthquakevictims0" model="crm.fundraising">
             <field eval="80" name="probability"/>
-            <field name="partner_address_id" ref="base.main_address"/>
             <field eval="1" name="active"/>
             <field name="type_id" ref="type_fund4"/>
             <field name="partner_id" ref="base.main_partner"/>
@@ -39,7 +37,6 @@
     </data>
     <data noupdate="1">
         <record id="crm_case_donatingbookstoschoollibraries0" model="crm.fundraising">
-            <field name="partner_address_id" ref="base.res_partner_address_zen"/>
             <field eval="1" name="active"/>
             <field name="type_id" ref="type_fund1"/>
             <field eval="5.0" name="duration"/>
@@ -56,7 +53,6 @@
     </data>
     <data noupdate="1">
         <record id="crm_case_renovatinggovernmentschools0" model="crm.fundraising">
-            <field name="partner_address_id" ref="base.res_partner_address_7"/>
             <field eval="1" name="active"/>
             <field name="type_id" ref="type_fund2"/>
             <field eval="3.0" name="duration"/>
@@ -74,7 +70,6 @@
     </data>
     <data noupdate="1">
         <record id="crm_case_donatingambulancestohospitals0" model="crm.fundraising">
-            <field name="partner_address_id" ref="base.res_partner_address_13"/>
             <field eval="1" name="active"/>
             <field name="type_id" ref="type_fund4"/>
             <field name="partner_id" ref="base.res_partner_14"/>
@@ -91,7 +86,6 @@
     </data>
     <data noupdate="1">
         <record id="crm_case_donatinghospitalequipments0" model="crm.fundraising">
-            <field name="partner_address_id" ref="base.res_partner_address_2"/>
             <field eval="1" name="active"/>
             <field name="type_id" ref="type_fund3"/>
             <field name="partner_id" ref="base.res_partner_10"/>
@@ -109,7 +103,6 @@
     </data>
     <data noupdate="1">
         <record id="crm_case_encouragingarts0" model="crm.fundraising">
-            <field name="partner_address_id" ref="base.res_partner_address_14"/>
             <field eval="1" name="active"/>
             <field name="type_id" ref="type_fund2"/>
             <field eval="7.0" name="duration"/>
@@ -127,7 +120,6 @@
     <data noupdate="1">
         <record id="crm_case_promotingculturalprogramsandpreservingdyingartforms0" model="crm.fundraising">
             <field eval="10" name="probability"/>
-            <field name="partner_address_id" ref="base.res_partner_address_1"/>
             <field eval="1" name="active"/>
             <field name="type_id" ref="type_fund1"/>
             <field eval="6.0" name="duration"/>
diff --git a/addons/crm_fundraising/crm_fundraising_view.xml b/addons/crm_fundraising/crm_fundraising_view.xml
index cc58b793c17..f8c7ae6cf80 100644
--- a/addons/crm_fundraising/crm_fundraising_view.xml
+++ b/addons/crm_fundraising/crm_fundraising_view.xml
@@ -94,10 +94,6 @@
                                 <field name="partner_id" select="1"
                                     on_change="onchange_partner_id(partner_id, email_from)"
                                     colspan="2" />
-                                <field name="partner_address_id"
-                                    string="Contact"
-                                    on_change="onchange_partner_address_id(partner_address_id, email_from)"
-                                    colspan="1" />
                                 <field name="email_from" colspan="2"/>
                             </group>
                             <group colspan="2" col="2">

From 9b54f8dcce4cbddea8aa043188d391e9322cfad8 Mon Sep 17 00:00:00 2001
From: Jigar Amin - OpenERP <jam@tinyerp.com>
Date: Tue, 6 Mar 2012 18:08:08 +0530
Subject: [PATCH 3/7] [IMP] crm helpdesk: removed crm helpdesk partner address

bzr revid: jam@tinyerp.com-20120306123808-qqd33knykkrgvjzr
---
 addons/crm_helpdesk/crm_helpdesk.py       | 3 ---
 addons/crm_helpdesk/crm_helpdesk_demo.xml | 1 -
 addons/crm_helpdesk/crm_helpdesk_view.xml | 3 ---
 3 files changed, 7 deletions(-)

diff --git a/addons/crm_helpdesk/crm_helpdesk.py b/addons/crm_helpdesk/crm_helpdesk.py
index 97ed007f456..7717ae2ce21 100644
--- a/addons/crm_helpdesk/crm_helpdesk.py
+++ b/addons/crm_helpdesk/crm_helpdesk.py
@@ -58,8 +58,6 @@ class crm_helpdesk(crm.crm_case, osv.osv):
             'company_id': fields.many2one('res.company', 'Company'),
             'date_closed': fields.datetime('Closed', readonly=True),
             'partner_id': fields.many2one('res.partner', 'Partner'),
-            'partner_address_id': fields.many2one('res.partner.address', 'Partner Contact', \
-                                 domain="[('partner_id','=',partner_id)]"),
             'email_cc': fields.text('Watchers Emails', size=252 , help="These email addresses will be added to the CC field of all inbound and outbound emails for this record before being sent. Separate multiple email addresses with a comma"),
             'email_from': fields.char('Email', size=128, help="These people will receive email."),
             'date': fields.datetime('Date'),
@@ -86,7 +84,6 @@ class crm_helpdesk(crm.crm_case, osv.osv):
         'active': lambda *a: 1,
         'user_id': crm.crm_case._get_default_user,
         'partner_id': crm.crm_case._get_default_partner,
-        'partner_address_id': crm.crm_case._get_default_partner_address,
         'email_from': crm.crm_case. _get_default_email,
         'state': lambda *a: 'draft',
         'date': lambda *a: time.strftime('%Y-%m-%d %H:%M:%S'),
diff --git a/addons/crm_helpdesk/crm_helpdesk_demo.xml b/addons/crm_helpdesk/crm_helpdesk_demo.xml
index a7f36c931c9..c91205fa7e6 100644
--- a/addons/crm_helpdesk/crm_helpdesk_demo.xml
+++ b/addons/crm_helpdesk/crm_helpdesk_demo.xml
@@ -38,7 +38,6 @@
         </record>
 
         <record id="crm_helpdesk_howtocreateanewmodule0" model="crm.helpdesk">
-            <field name="partner_address_id" ref="base.res_partner_address_9"/>
             <field eval="1" name="active"/>
             <field name="partner_id" ref="base.res_partner_2"/>
             <field name="user_id" ref="base.user_root"/>
diff --git a/addons/crm_helpdesk/crm_helpdesk_view.xml b/addons/crm_helpdesk/crm_helpdesk_view.xml
index 3d48a3b88ac..5c9e87c3dc8 100644
--- a/addons/crm_helpdesk/crm_helpdesk_view.xml
+++ b/addons/crm_helpdesk/crm_helpdesk_view.xml
@@ -47,9 +47,6 @@
                                 <field name="partner_id" colspan="2"
                                     on_change="onchange_partner_id(partner_id, email_from)"
                                     select="1" />
-                                <field name="partner_address_id" colspan="2"
-                                    on_change="onchange_partner_address_id(partner_address_id, email_from)"
-                                />
                                 <newline/>
                                 <field name="email_from" colspan="2"/>
                                 <button name="remind_partner"

From ca5a69de4ffc4da0f6c576147573a496b2382d9b Mon Sep 17 00:00:00 2001
From: Jigar Amin - OpenERP <jam@tinyerp.com>
Date: Tue, 6 Mar 2012 19:18:46 +0530
Subject: [PATCH 4/7] [IMP] crm : crm_partner_assign cleanup address dependancy

bzr revid: jam@tinyerp.com-20120306134846-yp4llvi8kbg14n0w
---
 addons/crm_partner_assign/__openerp__.py      |  2 +-
 .../crm_partner_assign/partner_geo_assign.py  | 17 ++++++-------
 .../report/crm_partner_report.py              |  2 +-
 .../wizard/crm_forward_to_partner.py          | 25 +++++--------------
 .../wizard/crm_forward_to_partner_view.xml    |  1 -
 5 files changed, 15 insertions(+), 32 deletions(-)

diff --git a/addons/crm_partner_assign/__openerp__.py b/addons/crm_partner_assign/__openerp__.py
index 6f5d0d98dd9..8213e8fa8ef 100644
--- a/addons/crm_partner_assign/__openerp__.py
+++ b/addons/crm_partner_assign/__openerp__.py
@@ -37,7 +37,7 @@ The most appropriate partner can be assigned.
 You can also use the geolocalization without using the GPS coordinates.
     """,
     'author': 'OpenERP SA',
-    'depends': ['crm'],
+    'depends': ['crm', 'account'],
     'demo_xml': [
         'res_partner_demo.xml',
     ],
diff --git a/addons/crm_partner_assign/partner_geo_assign.py b/addons/crm_partner_assign/partner_geo_assign.py
index f522cc63d7a..e225f5dc0d1 100644
--- a/addons/crm_partner_assign/partner_geo_assign.py
+++ b/addons/crm_partner_assign/partner_geo_assign.py
@@ -89,14 +89,11 @@ class res_partner(osv.osv):
     }
     def geo_localize(self, cr, uid, ids, context=None):
         for partner in self.browse(cr, uid, ids, context=context):
-            if not partner.address:
-                continue
-            contact = partner.address[0] #TOFIX: should be get latitude and longitude for default contact?
             addr = ', '.join(filter(None, [
-                    contact.street, 
-                    "%s %s" % (contact.zip , contact.city), 
-                    contact.state_id and contact.state_id.name, 
-                    contact.country_id and contact.country_id.name]))
+                    partner.street, 
+                    "%s %s" % (partner.zip , partner.city), 
+                    partner.state_id and partner.state_id.name, 
+                    partner.country_id and partner.country_id.name]))
             result = geo_find(tools.ustr(addr))
             if result:
                 self.write(cr, uid, [partner.id], {
@@ -192,7 +189,7 @@ class crm_lead(osv.osv):
                     ('partner_weight', '>', 0),
                     ('partner_latitude', '>', latitude - 2), ('partner_latitude', '<', latitude + 2),
                     ('partner_longitude', '>', longitude - 1.5), ('partner_longitude', '<', longitude + 1.5),
-                    ('country', '=', lead.country_id.id),
+                    ('country_id', '=', lead.country_id.id),
                 ], context=context)
 
                 # 2. second way: in the same country, big area
@@ -201,7 +198,7 @@ class crm_lead(osv.osv):
                         ('partner_weight', '>', 0),
                         ('partner_latitude', '>', latitude - 4), ('partner_latitude', '<', latitude + 4),
                         ('partner_longitude', '>', longitude - 3), ('partner_longitude', '<' , longitude + 3),
-                        ('country', '=', lead.country_id.id),
+                        ('country_id', '=', lead.country_id.id),
                     ], context=context)
 
 
@@ -210,7 +207,7 @@ class crm_lead(osv.osv):
                     # still haven't found any, let's take all partners in the country!
                     partner_ids = res_partner.search(cr, uid, [
                         ('partner_weight', '>', 0),
-                        ('country', '=', lead.country_id.id),
+                        ('country_id', '=', lead.country_id.id),
                     ], context=context)
 
                 # 6. sixth way: closest partner whatsoever, just to have at least one result
diff --git a/addons/crm_partner_assign/report/crm_partner_report.py b/addons/crm_partner_assign/report/crm_partner_report.py
index c225e35f581..764a09c52b9 100644
--- a/addons/crm_partner_assign/report/crm_partner_report.py
+++ b/addons/crm_partner_assign/report/crm_partner_report.py
@@ -51,7 +51,7 @@ class crm_partner_report_assign(osv.osv):
                 SELECT
                     coalesce(i.id, p.id - 1000000000) as id,
                     p.id as partner_id,
-                    (SELECT country_id FROM res_partner_address a WHERE a.partner_id=p.id AND country_id is not null limit 1) as country_id,
+                    (SELECT country_id FROM res_partner a WHERE a.partner_id=p.id AND country_id is not null limit 1) as country_id,
                     p.grade_id,
                     p.activation,
                     p.date_review,
diff --git a/addons/crm_partner_assign/wizard/crm_forward_to_partner.py b/addons/crm_partner_assign/wizard/crm_forward_to_partner.py
index 565f99b9db1..e9edca8166d 100644
--- a/addons/crm_partner_assign/wizard/crm_forward_to_partner.py
+++ b/addons/crm_partner_assign/wizard/crm_forward_to_partner.py
@@ -37,7 +37,6 @@ class crm_lead_forward_to_partner(osv.osv_memory):
         'user_id': fields.many2one('res.users', "User"),
         'attachment_ids': fields.many2many('ir.attachment','lead_forward_to_partner_attachment_rel', 'wizard_id', 'attachment_id', 'Attachments'),
         'partner_id' : fields.many2one('res.partner', 'Partner'),
-        'address_id' : fields.many2one('res.partner.address', 'Address'),
         'history': fields.selection([('info', 'Case Information'), ('latest', 'Latest email'), ('whole', 'Whole Story')], 'Send history', required=True),
     }
 
@@ -75,27 +74,15 @@ class crm_lead_forward_to_partner(osv.osv_memory):
         """This function fills address information based on partner/user selected
         """
         if not partner_id:
-            return {'value' : {'email_to' : False, 'address_id': False}}
-
+            return {'value' : {'email_to' : False}}
         partner_obj = self.pool.get('res.partner')
-        addr = partner_obj.address_get(cr, uid, [partner_id], ['contact'])
-        data = {'address_id': addr['contact']}
-        data.update(self.on_change_address(cr, uid, ids, addr['contact'])['value'])
-
+        data = {}
         partner = partner_obj.browse(cr, uid, [partner_id])
         user_id = partner and partner[0].user_id or False
-        email = user_id and user_id.user_email or ''
-        data.update({'email_cc' : email, 'user_id': user_id and user_id.id or False})
-        return {
-            'value' : data,
-            'domain' : {'address_id' : partner_id and "[('partner_id', '=', partner_id)]" or "[]"}
-        }
-
-    def on_change_address(self, cr, uid, ids, address_id):
-        email = ''
-        if address_id:
-            email = self.pool.get('res.partner.address').browse(cr, uid, address_id).email
-        return {'value': {'email_to' : email}}
+        data.update({'email_from': partner and partner[0].email or "", 
+                     'email_cc' : user_id and user_id.user_email or '', 
+                     'user_id': user_id and user_id.id or False})
+        return {'value' : data}
 
     def action_forward(self, cr, uid, ids, context=None):
         """
diff --git a/addons/crm_partner_assign/wizard/crm_forward_to_partner_view.xml b/addons/crm_partner_assign/wizard/crm_forward_to_partner_view.xml
index 645c729b164..7f20d5b4b15 100644
--- a/addons/crm_partner_assign/wizard/crm_forward_to_partner_view.xml
+++ b/addons/crm_partner_assign/wizard/crm_forward_to_partner_view.xml
@@ -20,7 +20,6 @@
                         </group>
                         <group col="4" colspan="4" attrs="{'invisible' : [('send_to','!=','partner')]}">
                             <field name="partner_id" attrs="{'required' : [('send_to','=','partner')]}" on_change="on_change_partner(partner_id)" colspan="2" />
-                            <field name="address_id" string="Contact" on_change="on_change_address(address_id)" colspan="2" />
                         </group>
                     </group>
                     <separator string="" colspan="4" />

From 545dc2f645d6f92d8f6f47cada31037f17ceeb8e Mon Sep 17 00:00:00 2001
From: Jigar Amin - OpenERP <jam@tinyerp.com>
Date: Wed, 7 Mar 2012 10:35:35 +0530
Subject: [PATCH 5/7] [IMP] crm_profiling cleaned partneraddress

bzr revid: jam@tinyerp.com-20120307050535-ek5esphxn2uujefy
---
 addons/crm_profiling/test/process/profiling.yml | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/addons/crm_profiling/test/process/profiling.yml b/addons/crm_profiling/test/process/profiling.yml
index 99fe2da1733..aa3060add3c 100644
--- a/addons/crm_profiling/test/process/profiling.yml
+++ b/addons/crm_profiling/test/process/profiling.yml
@@ -22,13 +22,12 @@
   I'm creating new partner "John" with his email "info@mycustomer.com".
 -
   !record {model: res.partner, id: res_partner_john0}:
-    address:
-      - city: Bruxelles
-        country_id: base.be
-        street: Rue des Palais 51, bte 33
-        type: default
-        zip: '1000'
-        email: 'info@mycustomer.com'
+    city: Bruxelles
+    country_id: base.be
+    street: Rue des Palais 51, bte 33
+    type: default
+    zip: '1000'
+    email: 'info@mycustomer.com'
     name: John
     category_id:
       - res_partner_category_customers0

From 108641b123693e189d68c9648240a0e1b47ec008 Mon Sep 17 00:00:00 2001
From: Jigar Amin - OpenERP <jam@tinyerp.com>
Date: Wed, 7 Mar 2012 11:17:51 +0530
Subject: [PATCH 6/7] [FIX] crm annonymous address cleanup

bzr revid: jam@tinyerp.com-20120307054751-rtnddry36p7zhxf8
---
 addons/crm/crm_lead.py                  | 2 --
 addons/crm/security/ir.model.access.csv | 2 --
 addons/crm_claim/crm_claim.py           | 2 +-
 3 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/addons/crm/crm_lead.py b/addons/crm/crm_lead.py
index f391797ed9e..34d38668d4f 100644
--- a/addons/crm/crm_lead.py
+++ b/addons/crm/crm_lead.py
@@ -60,7 +60,6 @@ class crm_lead(crm_case, osv.osv):
         'stage_id': _read_group_stage_ids
     }
 
-    # overridden because res.partner.address has an inconvenient name_get,
     # especially if base_contact is installed.
     def name_get(self, cr, user, ids, context=None):
         if isinstance(ids, (int, long)):
@@ -147,7 +146,6 @@ class crm_lead(crm_case, osv.osv):
         return res
 
     _columns = {
-        # Overridden from res.partner.address:
         'partner_id': fields.many2one('res.partner', 'Partner', ondelete='set null',
             select=True, help="Optional linked partner, usually after conversion of the lead"),
 
diff --git a/addons/crm/security/ir.model.access.csv b/addons/crm/security/ir.model.access.csv
index 47a0f8fa13a..7707118324f 100644
--- a/addons/crm/security/ir.model.access.csv
+++ b/addons/crm/security/ir.model.access.csv
@@ -26,13 +26,11 @@ access_crm_case_resource_type_manager,crm_case_resource_type manager,model_crm_c
 access_crm_phonecall_report_user,crm.phonecall.report.user,model_crm_phonecall_report,base.group_sale_salesman,1,0,0,0
 access_crm_phonecall_report_manager,crm.phonecall.report,model_crm_phonecall_report,base.group_sale_manager,1,1,1,1
 access_res_partner_manager,res.partner.crm.manager,base.model_res_partner,base.group_sale_manager,1,0,0,0
-access_res_partner_address_manager,res.partner.address.crm.user.manager,base.model_res_partner_address,base.group_sale_manager,1,0,0,0
 access_res_partner_category_manager,res.partner.category.crm.manager,base.model_res_partner_category,base.group_sale_manager,1,0,0,0
 mail_mail_message_manager,mail.message.manager,mail.model_mail_message,base.group_sale_manager,1,0,0,0
 access_calendar_attendee_crm_user,calendar.attendee.crm.user,model_calendar_attendee,base.group_sale_salesman,1,1,1,0
 access_calendar_attendee_crm_manager,calendar.attendee.crm.manager,model_calendar_attendee,base.group_sale_manager,1,1,1,1
 access_res_partner,res.partner.crm.user,base.model_res_partner,base.group_sale_salesman,1,1,1,0
-access_res_partner_address,res.partner.address.crm.user,base.model_res_partner_address,base.group_sale_salesman,1,1,1,0
 access_res_partner_category,res.partner.category.crm.user,base.model_res_partner_category,base.group_sale_salesman,1,1,1,0
 mail_mailgate_thread,mail.thread,mail.model_mail_thread,base.group_sale_salesman,1,1,1,1
 mail_gateway_mail_message_user,mail.message.user,mail.model_mail_message,base.group_sale_salesman,1,1,1,1
diff --git a/addons/crm_claim/crm_claim.py b/addons/crm_claim/crm_claim.py
index ef56c97deea..32562501901 100644
--- a/addons/crm_claim/crm_claim.py
+++ b/addons/crm_claim/crm_claim.py
@@ -105,7 +105,7 @@ class crm_claim(crm.crm_case, osv.osv):
                               'partner_phone': False
                             }
                    }
-        address = self.pool.get('res.partner.address').browse(cr, uid, part)
+        address = self.pool.get('res.partner').browse(cr, uid, part)
         return {'value': {'email_from': address.email, 'partner_phone': address.phone}}
 
     def case_open(self, cr, uid, ids, *args):

From 6c291f1aa0812be4f65e42e3c57803e411c8f564 Mon Sep 17 00:00:00 2001
From: Jigar Amin - OpenERP <jam@tinyerp.com>
Date: Wed, 7 Mar 2012 14:27:01 +0530
Subject: [PATCH 7/7] [IMP] crm lead filed optput to opt_out to support
 res_partner mailtemplare opt_out field consiatancy

bzr revid: jam@tinyerp.com-20120307085701-7j6usg7xszop0n61
---
 addons/crm/crm_lead.py          | 6 +++---
 addons/crm/crm_lead_view.xml    | 4 ++--
 addons/crm/test/ui/crm_demo.yml | 2 +-
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/addons/crm/crm_lead.py b/addons/crm/crm_lead.py
index 34d38668d4f..a6fe706f76a 100644
--- a/addons/crm/crm_lead.py
+++ b/addons/crm/crm_lead.py
@@ -170,7 +170,7 @@ class crm_lead(crm_case, osv.osv):
         'contact_name': fields.char('Contact Name', size=64),
         'partner_name': fields.char("Customer Name", size=64,help='The name of the future partner company that will be created while converting the lead into opportunity', select=1),
         'optin': fields.boolean('Opt-In', help="If opt-in is checked, this contact has accepted to receive emails."),
-        'optout': fields.boolean('Opt-Out', help="If opt-out is checked, this contact has refused to receive emails or unsubscribed to a campaign."),
+        'opt_out': fields.boolean('Opt-Out', help="If opt-out is checked, this contact has refused to receive emails or unsubscribed to a campaign."),
         'type':fields.selection([ ('lead','Lead'), ('opportunity','Opportunity'), ],'Type', help="Type is used to separate Leads and Opportunities"),
         'priority': fields.selection(crm.AVAILABLE_PRIORITIES, 'Priority', select=True),
         'date_closed': fields.datetime('Closed', readonly=True),
@@ -231,10 +231,10 @@ class crm_lead(crm_case, osv.osv):
         return {'value': {'email_from': address.email, 'phone': address.phone, 'country_id': address.country_id.id}}
 
     def on_change_optin(self, cr, uid, ids, optin):
-        return {'value':{'optin':optin,'optout':False}}
+        return {'value':{'optin':optin,'opt_out':False}}
 
     def on_change_optout(self, cr, uid, ids, optout):
-        return {'value':{'optout':optout,'optin':False}}
+        return {'value':{'opt_out':optout,'optin':False}}
 
     def onchange_stage_id(self, cr, uid, ids, stage_id, context={}):
         if not stage_id:
diff --git a/addons/crm/crm_lead_view.xml b/addons/crm/crm_lead_view.xml
index 6e2540f7954..09a214a7d53 100644
--- a/addons/crm/crm_lead_view.xml
+++ b/addons/crm/crm_lead_view.xml
@@ -186,7 +186,7 @@
                     <group colspan="2" col="2">
                         <separator string="Mailings" colspan="2" col="2"/>
                         <field name="optin" on_change="on_change_optin(optin)"/>
-                        <field name="optout" on_change="on_change_optout(optout)"/>
+                        <field name="opt_out" on_change="on_change_optout(opt_out)"/>
                     </group>
                     <group colspan="2" col="2">
                         <separator string="Statistics" colspan="2" col="2"/>
@@ -539,7 +539,7 @@
                     <group colspan="2" col="2">
                         <separator string="Mailings" colspan="2"/>
                         <field name="optin" on_change="on_change_optin(optin)"/>
-                        <field name="optout" on_change="on_change_optout(optout)"/>
+                        <field name="opt_out" on_change="on_change_optout(opt_out)"/>
                     </group>
                 </page>
                 <page string="Communication &amp; History" groups="base.group_extended">
diff --git a/addons/crm/test/ui/crm_demo.yml b/addons/crm/test/ui/crm_demo.yml
index 5ef40838466..a949fecae03 100644
--- a/addons/crm/test/ui/crm_demo.yml
+++ b/addons/crm/test/ui/crm_demo.yml
@@ -15,7 +15,7 @@
     name: 'Need 20 Days of Consultancy'
     type: opportunity
     state: draft
-    optout: True
+    opt_out: True
 -
   I create phonecall record to call partner onchange method.
 -