From 70a2e53926dabdebe54fafb77f8861dd8537f68a Mon Sep 17 00:00:00 2001
From: "Kuldeep Joshi (OpenERP)" <kjo@tinyerp.com>
Date: Thu, 15 Dec 2011 16:59:12 +0530
Subject: [PATCH 1/2] [FIX]portal: create record of ir_model_data and ir_value
 for portal

lp bug: https://launchpad.net/bugs/903474 fixed

bzr revid: kjo@tinyerp.com-20111215112912-o1y70syaidufq1ru
---
 addons/portal/portal.py       | 10 ++++++++--
 addons/portal/portal_view.xml |  2 +-
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/addons/portal/portal.py b/addons/portal/portal.py
index de39944e69f..39d63a3b662 100644
--- a/addons/portal/portal.py
+++ b/addons/portal/portal.py
@@ -98,8 +98,9 @@ class portal(osv.osv):
     def do_create_menu(self, cr, uid, ids, context=None):
         """ create a parent menu for the given portals """
         menu_obj = self.pool.get('ir.ui.menu')
+        ir_data = self.pool.get('ir.model.data')
+        ir_value = self.pool.get('ir.values')
         menu_root = self._res_xml_id(cr, uid, 'portal', 'portal_menu')
-        
         for p in self.browse(cr, uid, ids, context):
             # create a menuitem under 'portal.portal_menu'
             menu_values = {
@@ -110,7 +111,12 @@ class portal(osv.osv):
             menu_id = menu_obj.create(cr, uid, menu_values, context)
             # set the parent_menu_id to item_id
             self.write(cr, uid, [p.id], {'parent_menu_id': menu_id}, context)
-        
+            menu_values.update({'model': 'ir.ui.menu',
+                         'module': 'portal',
+                         'res_id': menu_id,
+                         'noupdate': 'True'})
+            data_id = ir_data.create(cr, uid, menu_values, context)
+            value_id = ir_value.create(cr, uid, menu_values, context)
         return True
 
     def _assign_menu(self, cr, uid, ids, context=None):
diff --git a/addons/portal/portal_view.xml b/addons/portal/portal_view.xml
index a6a780fdf16..bda74d501de 100644
--- a/addons/portal/portal_view.xml
+++ b/addons/portal/portal_view.xml
@@ -21,7 +21,7 @@ the portal's users.
         </record>
         <menuitem name="Portals" id="portal_list_menu"
             parent="portal_menu" sequence="1" action="portal_list_action"/>
-
+            
         <!-- portal tree view -->
         <record id="portal_list_view" model="ir.ui.view">
             <field name="name">Portal List</field>

From 22781f955685ccbed652dae5d9226676ac3c390c Mon Sep 17 00:00:00 2001
From: "Kuldeep Joshi (OpenERP)" <kjo@tinyerp.com>
Date: Tue, 20 Dec 2011 15:40:36 +0530
Subject: [PATCH 2/2] [IMP] portal: remove unused field

bzr revid: kjo@tinyerp.com-20111220101036-mbif1y0xa0d29o9u
---
 addons/portal/portal.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/addons/portal/portal.py b/addons/portal/portal.py
index 39d63a3b662..071b5f0a4b6 100644
--- a/addons/portal/portal.py
+++ b/addons/portal/portal.py
@@ -99,7 +99,6 @@ class portal(osv.osv):
         """ create a parent menu for the given portals """
         menu_obj = self.pool.get('ir.ui.menu')
         ir_data = self.pool.get('ir.model.data')
-        ir_value = self.pool.get('ir.values')
         menu_root = self._res_xml_id(cr, uid, 'portal', 'portal_menu')
         for p in self.browse(cr, uid, ids, context):
             # create a menuitem under 'portal.portal_menu'
@@ -111,12 +110,13 @@ class portal(osv.osv):
             menu_id = menu_obj.create(cr, uid, menu_values, context)
             # set the parent_menu_id to item_id
             self.write(cr, uid, [p.id], {'parent_menu_id': menu_id}, context)
+            menu_values.pop('parent_id')
+            menu_values.pop('groups_id')
             menu_values.update({'model': 'ir.ui.menu',
                          'module': 'portal',
                          'res_id': menu_id,
                          'noupdate': 'True'})
             data_id = ir_data.create(cr, uid, menu_values, context)
-            value_id = ir_value.create(cr, uid, menu_values, context)
         return True
 
     def _assign_menu(self, cr, uid, ids, context=None):