credativ team mailing list archive

["Sandy Carter \(http://www.savoirfairelinux.com\)" <sandy.carter@xxxxxxxxxxxxxxxxxxxx>]

Sandy Carter (http://www.savoirfairelinux.com) has proposed merging lp:~savoirfairelinux-openerp/openupgrade-server/NoDemo into lp:openupgrade-server.

Requested reviews:
  OpenUpgrade Committers (openupgrade-committers)

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/openupgrade-server/NoDemo/+merge/193145

Forces Openupgrade to skip inserting demo data itself.

In the instance of 6.1 -> 7.0:

This allows the possibility to run OpenERP 6.1 with demo data on, followed by OpenUpgrade followed by OpenERP 7.0 which will have the demo data of 6.1

Prior to this, having a database populated by demo data, OpenUpgrade will try to insert its own demo data resulting in all sorts of errors.

This saves a lot of time creating migration scripts as one does not need to populate the tables manually. It also helps in catching incompatibilities one might have missed.
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openupgrade-server/NoDemo/+merge/193145
Your team OpenUpgrade Committers is requested to review the proposed merge of lp:~savoirfairelinux-openerp/openupgrade-server/NoDemo into lp:openupgrade-server.

=== modified file 'openerp/modules/loading.py'
--- openerp/modules/loading.py	2013-10-25 19:31:47 +0000
+++ openerp/modules/loading.py	2013-10-29 21:30:22 +0000
@@ -114,6 +114,8 @@
             noupdate = False
             if kind in ('demo', 'demo_xml'):
                 noupdate = True
+                # OpenUpgrade: skip loading any demo data
+                continue
             try:
                 ext = ext.lower()
                 if ext == '.csv':