Merge lp:~openerp-dev/openobject-addons/trunk-remove-nodestroy-rga into lp:openobject-addons

[Ravi Gadhia (OpenERP)]

Remove GTK client specific attribute 'nodestroy'

'nodestroy' was introduce for stop crashing GTK client when form view button(type object) return action. if we not return 'nodestroy':True in action it will crash client for osv object but web client gracefully handle it

Unmerged revisions

8757. By RGA(OpenERP) on 2013-06-12

Remove GTK client specific attribute 'nodestroy'

=== modified file 'account/account_invoice.py'
--- account/account_invoice.py	2013-06-07 10:23:59 +0000
+++ account/account_invoice.py	2013-06-12 14:00:39 +0000
@@ -415,7 +415,6 @@
             'type': 'ir.actions.report.xml',
             'report_name': 'account.invoice',
             'datas': datas,
-            'nodestroy' : True
         }
 
     def action_invoice_sent(self, cr, uid, ids, context=None):

=== modified file 'account_analytic_analysis/account_analytic_analysis.py'
--- account_analytic_analysis/account_analytic_analysis.py	2013-06-07 10:23:59 +0000
+++ account_analytic_analysis/account_analytic_analysis.py	2013-06-12 14:00:39 +0000
@@ -539,7 +539,6 @@
             'context': context,
             'domain' : [('order_id','in',sale_ids)],
             'res_model': 'sale.order.line',
-            'nodestroy': True,
         }
 
     def on_change_template(self, cr, uid, ids, template_id, context=None):
@@ -642,7 +641,6 @@
             'view_mode': 'tree,form',
             'domain' : domain,
             'res_model': 'account.analytic.line',
-            'nodestroy': True,
         }
 
     def _prepare_invoice(self, cr, uid, contract, context=None):

=== modified file 'account_check_writing/account_voucher.py'
--- account_check_writing/account_voucher.py	2013-04-15 10:23:49 +0000
+++ account_check_writing/account_voucher.py	2013-06-12 14:00:39 +0000
@@ -78,7 +78,6 @@
                     'ids': ids and ids or [],
                     'report_type': 'pdf'
                 },
-            'nodestroy': True
             }
 
     def fields_view_get(self, cr, uid, view_id=None, view_type=False, context=None, toolbar=False, submenu=False):

=== modified file 'account_check_writing/wizard/account_check_batch_printing.py'
--- account_check_writing/wizard/account_check_batch_printing.py	2013-04-15 10:23:49 +0000
+++ account_check_writing/wizard/account_check_batch_printing.py	2013-06-12 14:00:39 +0000
@@ -80,7 +80,6 @@
                 'ids': voucher_ids,
                 'report_type': 'pdf'
                 },
-            'nodestroy': True
             }
 
 

=== modified file 'account_voucher/invoice.py'
--- account_voucher/invoice.py	2013-05-21 12:23:59 +0000
+++ account_voucher/invoice.py	2013-06-12 14:00:39 +0000
@@ -37,7 +37,6 @@
             'view_type': 'form',
             'res_model': 'account.voucher',
             'type': 'ir.actions.act_window',
-            'nodestroy': True,
             'target': 'new',
             'domain': '[]',
             'context': {

=== modified file 'analytic_contract_hr_expense/analytic_contract_hr_expense.py'
--- analytic_contract_hr_expense/analytic_contract_hr_expense.py	2013-04-29 15:14:10 +0000
+++ analytic_contract_hr_expense/analytic_contract_hr_expense.py	2013-06-12 14:00:39 +0000
@@ -149,7 +149,6 @@
             'view_mode': 'tree,form',
             'domain' : domain,
             'res_model': 'account.analytic.line',
-            'nodestroy': True,
         }
 
 

=== modified file 'fleet/fleet.py'
--- fleet/fleet.py	2013-01-18 14:39:02 +0000
+++ fleet/fleet.py	2013-06-12 14:00:39 +0000
@@ -745,7 +745,6 @@
             'view_type': 'tree,form',
             'res_model': 'fleet.vehicle.log.contract',
             'type': 'ir.actions.act_window',
-            'nodestroy': True,
             'domain': '[]',
             'res_id': newid,
             'context': {'active_id':newid}, 

=== modified file 'hr_expense/hr_expense.py'
--- hr_expense/hr_expense.py	2013-05-28 10:45:00 +0000
+++ hr_expense/hr_expense.py	2013-06-12 14:00:39 +0000
@@ -384,7 +384,6 @@
             'view_id': view_id,
             'res_model': 'account.move',
             'type': 'ir.actions.act_window',
-            'nodestroy': True,
             'target': 'current',
             'res_id': expense.account_move_id.id,
         }

=== modified file 'hr_payroll/hr_payroll.py'
--- hr_payroll/hr_payroll.py	2013-04-29 15:14:10 +0000
+++ hr_payroll/hr_payroll.py	2013-06-12 14:00:39 +0000
@@ -347,7 +347,6 @@
             'view_type': 'form',
             'res_model': 'hr.payslip',
             'type': 'ir.actions.act_window',
-            'nodestroy': True,
             'target': 'current',
             'domain': "[('id', 'in', %s)]" % [id_copy],
             'views': [(tree_res, 'tree'), (form_res, 'form')],

=== modified file 'hr_recruitment/hr_recruitment.py'
--- hr_recruitment/hr_recruitment.py	2013-05-30 10:08:52 +0000
+++ hr_recruitment/hr_recruitment.py	2013-06-12 14:00:39 +0000
@@ -556,7 +556,6 @@
             'report_name': 'survey.form',
             'datas': datas,
             'context' : context,
-            'nodestroy':True,
         }
 
 class applicant_category(osv.osv):

=== modified file 'marketing_campaign/marketing_campaign.py'
--- marketing_campaign/marketing_campaign.py	2013-04-15 10:23:49 +0000
+++ marketing_campaign/marketing_campaign.py	2013-06-12 14:00:39 +0000
@@ -794,7 +794,6 @@
                 'views': [(view_id and view_id[1] or 0, 'form')],
                 'type': 'ir.actions.act_window',
                 'target': 'new',
-                'nodestroy':True,
                 'context': "{'template_id':%d,'default_res_id':%d}"%
                                 (wi_obj.activity_id.email_template_id.id,
                                  wi_obj.res_id)

=== modified file 'point_of_sale/point_of_sale.py'
--- point_of_sale/point_of_sale.py	2013-04-29 07:15:57 +0000
+++ point_of_sale/point_of_sale.py	2013-06-12 14:00:39 +0000
@@ -813,7 +813,6 @@
             'view_id': False,
             'context':context,
             'type': 'ir.actions.act_window',
-            'nodestroy': True,
             'target': 'current',
         }
         return abs
@@ -889,7 +888,6 @@
             'res_model': 'account.invoice',
             'context': "{'type':'out_invoice'}",
             'type': 'ir.actions.act_window',
-            'nodestroy': True,
             'target': 'current',
             'res_id': inv_ids and inv_ids[0] or False,
         }

=== modified file 'project/project.py'
--- project/project.py	2013-06-07 10:23:59 +0000
+++ project/project.py	2013-06-12 14:00:39 +0000
@@ -425,7 +425,6 @@
                 'views': [(form_view['res_id'],'form'),(tree_view['res_id'],'tree')],
                 'type': 'ir.actions.act_window',
                 'search_view_id': search_view['res_id'],
-                'nodestroy': True
             }
 
     # set active value for a project, its sub projects and its tasks

=== modified file 'project_issue/project_issue.py'
--- project_issue/project_issue.py	2013-06-05 09:04:53 +0000
+++ project_issue/project_issue.py	2013-06-12 14:00:39 +0000
@@ -377,7 +377,6 @@
             'views': [(id2,'form'),(id3,'tree'),(False,'calendar'),(False,'graph')],
             'type': 'ir.actions.act_window',
             'search_view_id': res['res_id'],
-            'nodestroy': True
         }
 
     def copy(self, cr, uid, id, default=None, context=None):

=== modified file 'purchase/purchase.py'
--- purchase/purchase.py	2013-05-21 12:23:59 +0000
+++ purchase/purchase.py	2013-06-12 14:00:39 +0000
@@ -358,7 +358,6 @@
             'res_model': 'account.invoice',
             'context': "{'type':'in_invoice', 'journal_type': 'purchase'}",
             'type': 'ir.actions.act_window',
-            'nodestroy': True,
             'target': 'current',
             'res_id': inv_ids and inv_ids[0] or False,
         }
@@ -450,7 +449,7 @@
                  'ids': ids,
                  'form': self.read(cr, uid, ids[0], context=context),
         }
-        return {'type': 'ir.actions.report.xml', 'report_name': 'purchase.quotation', 'datas': datas, 'nodestroy': True}
+        return {'type': 'ir.actions.report.xml', 'report_name': 'purchase.quotation', 'datas': datas}
 
     #TODO: implement messages system
     def wkf_confirm_order(self, cr, uid, ids, context=None):

=== modified file 'sale/sale.py'
--- sale/sale.py	2013-06-07 10:23:59 +0000
+++ sale/sale.py	2013-06-12 14:00:39 +0000
@@ -290,7 +290,6 @@
             'view_mode': 'form',
             'view_id': view_id,
             'target': 'current',
-            'nodestroy': True,
         }
 
     def onchange_pricelist_id(self, cr, uid, ids, pricelist_id, order_lines, context=None):
@@ -432,7 +431,7 @@
                  'ids': ids,
                  'form': self.read(cr, uid, ids[0], context=context),
         }
-        return {'type': 'ir.actions.report.xml', 'report_name': 'sale.order', 'datas': datas, 'nodestroy': True}
+        return {'type': 'ir.actions.report.xml', 'report_name': 'sale.order', 'datas': datas }
 
     def manual_invoice(self, cr, uid, ids, context=None):
         """ create invoices for the given sales orders (ids), and open the form
@@ -458,7 +457,6 @@
             'res_model': 'account.invoice',
             'context': "{'type':'out_invoice'}",
             'type': 'ir.actions.act_window',
-            'nodestroy': True,
             'target': 'current',
             'res_id': new_inv_ids and new_inv_ids[0] or False,
         }
@@ -596,7 +594,6 @@
             'view_mode': 'form',
             'view_id': view_id,
             'target': 'current',
-            'nodestroy': True,
         }
 
     def action_wait(self, cr, uid, ids, context=None):

=== modified file 'stock/stock.py'
--- stock/stock.py	2013-06-07 10:23:59 +0000
+++ stock/stock.py	2013-06-12 14:00:39 +0000
@@ -692,7 +692,6 @@
             'type': 'ir.actions.act_window',
             'target': 'new',
             'context': context,
-            'nodestroy': True,
         }
 
     def copy(self, cr, uid, id, default=None, context=None):
@@ -1543,7 +1542,6 @@
             'res_model': 'stock.partial.move',
             'res_id': partial_id,
             'type': 'ir.actions.act_window',
-            'nodestroy': True,
             'target': 'new',
             'domain': '[]',
             'context': context

=== modified file 'stock/wizard/stock_traceability.py'
--- stock/wizard/stock_traceability.py	2013-04-15 10:23:49 +0000
+++ stock/wizard/stock_traceability.py	2013-06-12 14:00:39 +0000
@@ -58,7 +58,6 @@
             'field_parent': type1,
             'view_id': (view_id,'View'),
             'type': 'ir.actions.act_window',
-            'nodestroy':True,
         }
         return value
 

=== modified file 'survey/survey.py'
--- survey/survey.py	2013-04-15 10:23:49 +0000
+++ survey/survey.py	2013-06-12 14:00:39 +0000
@@ -127,7 +127,6 @@
                 'report_name': 'survey.browse.response',
                 'datas': datas,
                 'context' : context,
-                'nodestroy':True,
             }
         else:
 
@@ -138,7 +137,6 @@
                 'report_name': 'survey.form',
                 'datas': datas,
                 'context' : context,
-                'nodestroy':True,
             }
         return report