From 72714ec312de69d62d5e0d34e44d9a009d00d68d Mon Sep 17 00:00:00 2001
From: Raphael Collet <rco@openerp.com>
Date: Wed, 27 Apr 2011 16:48:23 +0200
Subject: [PATCH] use _defaults instead of default_get for initializing wizard
 users

bzr revid: rco@openerp.com-20110427144823-5ehmjkq8e712zbqc
---
 addons/portal/wizard.py       | 30 +++++++++++++-----------------
 addons/portal/wizard_view.xml |  3 ++-
 2 files changed, 15 insertions(+), 18 deletions(-)

diff --git a/addons/portal/wizard.py b/addons/portal/wizard.py
index 794882730c1..3856821df4c 100644
--- a/addons/portal/wizard.py
+++ b/addons/portal/wizard.py
@@ -61,12 +61,12 @@ class wizard(osv.osv_memory):
         'user_ids': fields.one2many('res.portal.wizard.user', 'wizard_id',
             string='Users'),
     }
+    _defaults = {
+        'user_ids': (lambda self, *args: self._default_user_ids(*args))
+    }
 
-    def default_get(self, cr, uid, fields, context=None):
-        """ determine user configs from the active records """
-        # get existing defaults
-        defs = super(wizard, self).default_get(cr, uid, fields, context)
-        
+    def _default_user_ids(self, cr, uid, context):
+        """ determine default user_ids from the active records """
         # determine relevant res.partner.address(es) depending on context
         addresses = []
         if context.get('active_model') == 'res.partner.address':
@@ -84,19 +84,15 @@ class wizard(osv.osv_memory):
                     addresses.append(good[0] if good else p.address[0])
         
         # create user configs based on these addresses
-        defs['user_ids'] = []
-        for address in addresses:
-            user_data = {
-                'name': address.name,
-                'email': address.email,
-                'lang': address.partner_id and address.partner_id.lang,
-                'address_id': address.id,
-            }
-            defs['user_ids'].append(user_data)
-        
-        return defs
+        user_data = lambda address: {
+                        'name': address.name,
+                        'email': address.email,
+                        'lang': address.partner_id and address.partner_id.lang,
+                        'address_id': address.id,
+                    }
+        return map(user_data, addresses)
 
-    def do_create(self, cr, uid, ids, context=None):
+    def action_create(self, cr, uid, ids, context=None):
         """ create new users in portal(s), and notify them by email """
         # we copy the context to change the language for translating emails
         context0 = context or {}
diff --git a/addons/portal/wizard_view.xml b/addons/portal/wizard_view.xml
index dbc48a6979e..5c3b095aa1e 100644
--- a/addons/portal/wizard_view.xml
+++ b/addons/portal/wizard_view.xml
@@ -28,7 +28,8 @@
                     <field name="user_ids" nolabel="1" colspan="4"/>
                     <label colspan="2"/>
                     <button string="Cancel" special="cancel" icon="gtk-cancel"/>
-                    <button string="Create Users" name="do_create" type="object" icon="gtk-ok"/>
+                    <button string="Create Users"
+                        name="action_create" type="object" icon="gtk-ok"/>
                 </form>
             </field>
         </record>