From b741852c3d170b0cf2ed733a40a3d80ed5abbee0 Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@openerp.com>
Date: Wed, 18 Dec 2013 14:49:32 +0100
Subject: [PATCH] [IMP] correctly handle writing to a view without a model_data
 (e.g. test views)

bzr revid: xmo@openerp.com-20131218134932-382cvndk8asr82ip
---
 openerp/addons/base/ir/ir_ui_view.py | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/openerp/addons/base/ir/ir_ui_view.py b/openerp/addons/base/ir/ir_ui_view.py
index 91785af7664..fddf77d2479 100644
--- a/openerp/addons/base/ir/ir_ui_view.py
+++ b/openerp/addons/base/ir/ir_ui_view.py
@@ -180,11 +180,10 @@ class view(osv.osv):
 
         if not context.get('install_mode', False):
             # touched views become noupdatable
-            for id in ids:
-                _view = self.browse(cr, uid, id, context=context)
-                _view.model_data_id.write({
-                    'noupdate': True
-                })
+            # TODO: should be doable in a read and a write
+            for view_ in self.browse(cr, uid, ids, context=context):
+                if view_.model_data_id:
+                    view_.model_data_id.write({'noupdate': True})
         return ret
 
     # default view selection