Merge lp:~invitu/openerp-mgmtsystem/7-0_noreload_data into lp:openerp-mgmtsystem

[invitu]

Some sample data should not be recreated if deleted

[lp:~invitu/openerp-mgmtsystem/7-0_noreload_data]

93. By invitu on 2014-02-25

[IMP] some demo data should not be recreated if we delete them

[Pedro Manuel Baeza (pedro.baeza)]

Hi, thanks for the change, but if you set noupdate flag, I think forcecreate="0" is redundant, doesn't it?

Regards.

review: Needs Information

[invitu (invitu)]

Hello, thanks for your comment
noupdate="1" flag means "data are not updated but they might be recreated if they are deleted"
forcecreate="0" flag means "date are not recreated if they are deleted"

(cf Olivier Dony : http://help.openerp.com/question/13546/data-noupdate0-1-in-securityxml/)

[Pedro Manuel Baeza (pedro.baeza)]

Thank you very much for the clarification. One thing more that I learn on the reviews!

About the MP itself, I consider that the approach of no recreating deleted samples are OK to clean data, so I approve it.

Regards.

review: Approve (code review)

[Maxime Chambreuil (http://www.savoirfairelinux.com) (max3903)]

review: Approve

[Daniel Reis (dreis-pt)]

Just learned something new today.

review: Approve

=== modified file 'mgmtsystem_hazard/mgmtsystem_hazard_data.xml'
--- mgmtsystem_hazard/mgmtsystem_hazard_data.xml	2012-03-09 23:47:59 +0000
+++ mgmtsystem_hazard/mgmtsystem_hazard_data.xml	2014-02-25 21:54:44 +0000
@@ -1,52 +1,52 @@
 <?xml version="1.0" encoding="utf-8"?>
 <openerp>
-    <data>
+    <data noupdate="1">
 
         <!-- hazard hazard -->
 
-        <record id="hazard_spilling" model="mgmtsystem.hazard.hazard">
+        <record id="hazard_spilling" model="mgmtsystem.hazard.hazard" forcecreate="0">
             <field name="name">Spilling</field>
         </record>
 
-        <record id="hazard_fire_explosion" model="mgmtsystem.hazard.hazard">
+        <record id="hazard_fire_explosion" model="mgmtsystem.hazard.hazard" forcecreate="0">
             <field name="name">Fire / Explosion</field>
         </record>
 
-        <record id="hazard_intoxication" model="mgmtsystem.hazard.hazard">
+        <record id="hazard_intoxication" model="mgmtsystem.hazard.hazard" forcecreate="0">
             <field name="name">Intoxication</field>
         </record>
 
-        <record id="hazard_breakage" model="mgmtsystem.hazard.hazard">
+        <record id="hazard_breakage" model="mgmtsystem.hazard.hazard" forcecreate="0">
             <field name="name">Breakage</field>
         </record>
 
-        <record id="hazard_electric_bilge" model="mgmtsystem.hazard.hazard">
+        <record id="hazard_electric_bilge" model="mgmtsystem.hazard.hazard" forcecreate="0">
             <field name="name">Electric Bilge</field>
         </record>
 
         <!-- origin -->
 
-        <record id="origin_ignition_gas" model="mgmtsystem.hazard.origin">
+        <record id="origin_ignition_gas" model="mgmtsystem.hazard.origin" forcecreate="0">
             <field name="name">Ignition of Flammable Gas</field>
         </record>
 
-        <record id="origin_panic" model="mgmtsystem.hazard.origin">
+        <record id="origin_panic" model="mgmtsystem.hazard.origin" forcecreate="0">
             <field name="name">Panic</field>
         </record>
 
-        <record id="origin_exposure" model="mgmtsystem.hazard.origin">
+        <record id="origin_exposure" model="mgmtsystem.hazard.origin" forcecreate="0">
             <field name="name">Exposure to Hazardous Materials</field>
         </record>
 
-        <record id="origin_containment" model="mgmtsystem.hazard.origin">
+        <record id="origin_containment" model="mgmtsystem.hazard.origin" forcecreate="0">
             <field name="name">Containment Failure</field>
         </record>
 
-        <record id="origin_leak" model="mgmtsystem.hazard.origin">
+        <record id="origin_leak" model="mgmtsystem.hazard.origin" forcecreate="0">
             <field name="name">Gas Leak</field>
         </record>
 
-        <record id="origin_aging" model="mgmtsystem.hazard.origin">
+        <record id="origin_aging" model="mgmtsystem.hazard.origin" forcecreate="0">
             <field name="name">Materials Aging</field>
         </record>
 
@@ -102,19 +102,19 @@
 
         <!-- risk_type -->
 
-        <record id="risk_type_physical" model="mgmtsystem.hazard.risk.type">
+        <record id="risk_type_physical" model="mgmtsystem.hazard.risk.type" forcecreate="0">
             <field name="name">Physical</field>
         </record>
 
-        <record id="risk_type_chemical" model="mgmtsystem.hazard.risk.type">
+        <record id="risk_type_chemical" model="mgmtsystem.hazard.risk.type" forcecreate="0">
             <field name="name">Chemical</field>
         </record>
 
-        <record id="risk_type_fire" model="mgmtsystem.hazard.risk.type">
+        <record id="risk_type_fire" model="mgmtsystem.hazard.risk.type" forcecreate="0">
             <field name="name">Fire</field>
         </record>
 
-        <record id="risk_type_environment" model="mgmtsystem.hazard.risk.type">
+        <record id="risk_type_environment" model="mgmtsystem.hazard.risk.type" forcecreate="0">
             <field name="name">Environment</field>
         </record>