diff --git a/.travis.yml b/.travis.yml index d3e355a6e..50f6081b7 100644 --- a/.travis.yml +++ b/.travis.yml @@ -33,6 +33,7 @@ install: - export PATH=${HOME}/maintainer-quality-tools/travis:${PATH} - travis_install_nightly - printf '[options]\n\nrunning_env = dev\n' > ${HOME}/.openerp_serverrc + - pip install erppeek - ln -s ${TRAVIS_BUILD_DIR}/server_environment_files_sample ${TRAVIS_BUILD_DIR}/server_environment_files script: - travis_run_tests diff --git a/base_import_odoo/README.rst b/base_import_odoo/README.rst new file mode 100644 index 000000000..73c0fee64 --- /dev/null +++ b/base_import_odoo/README.rst @@ -0,0 +1,94 @@ +.. image:: https://img.shields.io/badge/licence-AGPL--3-blue.svg + :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html + :alt: License: AGPL-3 + +================================ +Import from remote Odoo database +================================ + +This module was written to import data from another Odoo database. The idea is that you define which models to import from the other database, and add eventual mappings for records you don't want to import. + +Use cases +========= + +- merging databases +- one way sync (needs a bit polishing) +- aggregating management data from distributed systems + + +Configuration +============= + +Go to Settings / Remote Odoo import / Import configurations and create a configuration. + +After filling in your credentials, select models you want to import from the remote database. If you only want to import a subset of the records, add an appropriate domain. + +The import will copy records of all models listed, and handle links to records of models which are not imported depending on the existing field mappings. Field mappings to local records also are a stopping condition. Without those, the import will have to create some record for all required x2x fields, which you probably don't want. + +Probably you'll want to map records of model `res.company`, and at least the admin user. + +The module doesn't import one2many fields, if you want to have those, add the model the field in question points to to the list of imported models, possibly with a domain. + +If you don't fill in a remote ID, the addon will use the configured local ID for every record of the model (this way, you can for example map all users in the remote system to some import user in the current system). + +For fields that have a uniqueness constraint (like `res.users#login`), set the flag `unique`, then the import will generate a unique value for this field. + +Usage +===== + +To use this module, you need to: + +#. go to an import configuration and hit the button ``Run import`` +#. be patient, this creates a cronjob which will start up to a minutes afterwards +#. reload the form, as soon as the cronjob runs you'll see a field ``Progress`` that lets you inspect what was imported already +#. note that the cronjob also resets the password as soon as it has read it. So for a subsequent import, you'll have to fill it in again +#. running an import a second time won't duplicate data, it should recognize records imported earlier and just update them + +.. image:: https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas + :alt: Try me on Runbot + :target: https://runbot.odoo-community.org/runbot/149/8.0 + +Known issues / Roadmap +====================== + +* Yes of course this duplicates a lot of connector functionality. Rewrite this with the connector framework, probably collaborate with https://github.com/OCA/connector-odoo2odoo +* Do something with workflows +* Probably it's safer and faster to disable recomputation during import, and recompute all fields afterwards + +Bug Tracker +=========== + +Bugs are tracked on `GitHub Issues +`_. In case of trouble, please +check there if your issue has already been reported. If you spotted it first, +help us smashing it by providing a detailed and welcomed feedback. + +Credits +======= + +Images +------ + +* Odoo Community Association: `Icon `_. + +Contributors +------------ + +* Holger Brunn + +Do not contact contributors directly about help with questions or problems concerning this addon, but use the `community mailing list `_ or the `appropriate specialized mailinglist `_ for help, and the bug tracker linked in `Bug Tracker`_ above for technical issues. + +Maintainer +---------- + +.. image:: https://odoo-community.org/logo.png + :alt: Odoo Community Association + :target: https://odoo-community.org + +This module is maintained by the OCA. + +OCA, or the Odoo Community Association, is a nonprofit organization whose +mission is to support the collaborative development of Odoo features and +promote its widespread use. + +To contribute to this module, please visit https://odoo-community.org. diff --git a/base_import_odoo/__init__.py b/base_import_odoo/__init__.py new file mode 100644 index 000000000..86cb334c3 --- /dev/null +++ b/base_import_odoo/__init__.py @@ -0,0 +1,4 @@ +# -*- coding: utf-8 -*- +# © 2017 Therp BV +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). +from . import models diff --git a/base_import_odoo/__openerp__.py b/base_import_odoo/__openerp__.py new file mode 100644 index 000000000..baaba59e5 --- /dev/null +++ b/base_import_odoo/__openerp__.py @@ -0,0 +1,29 @@ +# -*- coding: utf-8 -*- +# © 2017 Therp BV +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). +{ + "name": "Import from Odoo", + "version": "8.0.1.0.0", + "author": "Therp BV,Odoo Community Association (OCA)", + "license": "AGPL-3", + "category": "Tools", + "summary": "Import records from another Odoo instance", + "depends": [ + 'base', + ], + "demo": [ + "demo/res_users.xml", + "demo/import_odoo_database.xml", + "demo/import_odoo_database_field.xml", + "demo/import_odoo_database_model.xml", + ], + "data": [ + "security/ir.model.access.csv", + "views/import_odoo_database.xml", + "views/menu.xml", + ], + "installable": True, + "external_dependencies": { + "python": ['erppeek'], + }, +} diff --git a/base_import_odoo/demo/import_odoo_database.xml b/base_import_odoo/demo/import_odoo_database.xml new file mode 100644 index 000000000..f1f9bb59d --- /dev/null +++ b/base_import_odoo/demo/import_odoo_database.xml @@ -0,0 +1,11 @@ + + + + + http://localhost:8069 + demodb + admin + + + + diff --git a/base_import_odoo/demo/import_odoo_database_field.xml b/base_import_odoo/demo/import_odoo_database_field.xml new file mode 100644 index 000000000..e47c6d070 --- /dev/null +++ b/base_import_odoo/demo/import_odoo_database_field.xml @@ -0,0 +1,23 @@ + + + + + + + + + + + + + + + + + + + + + + + diff --git a/base_import_odoo/demo/import_odoo_database_model.xml b/base_import_odoo/demo/import_odoo_database_model.xml new file mode 100644 index 000000000..c9c856766 --- /dev/null +++ b/base_import_odoo/demo/import_odoo_database_model.xml @@ -0,0 +1,17 @@ + + + + + 1 + + + [(1, '=', 1)] + + + 2 + + + [(1, '=', 1)] + + + diff --git a/base_import_odoo/demo/res_users.xml b/base_import_odoo/demo/res_users.xml new file mode 100644 index 000000000..152e5cbd3 --- /dev/null +++ b/base_import_odoo/demo/res_users.xml @@ -0,0 +1,9 @@ + + + + + Mapped admin + mapped_admin + + + diff --git a/base_import_odoo/models/__init__.py b/base_import_odoo/models/__init__.py new file mode 100644 index 000000000..2fd595611 --- /dev/null +++ b/base_import_odoo/models/__init__.py @@ -0,0 +1,7 @@ +# -*- coding: utf-8 -*- +# © 2017 Therp BV +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). +from . import ir_model_data +from . import import_odoo_database +from . import import_odoo_database_model +from . import import_odoo_database_field diff --git a/base_import_odoo/models/import_odoo_database.py b/base_import_odoo/models/import_odoo_database.py new file mode 100644 index 000000000..8a15b73f4 --- /dev/null +++ b/base_import_odoo/models/import_odoo_database.py @@ -0,0 +1,472 @@ +# -*- coding: utf-8 -*- +# © 2017 Therp BV +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). +try: + from erppeek import Client +except: + pass +import psycopg2 +import traceback +from openerp import _, api, exceptions, fields, models, tools +from collections import namedtuple + + +import_context_tuple = namedtuple( + 'import_context', [ + 'remote', 'model_line', 'ids', 'idmap', 'dummies', 'dummy_instances', + 'to_delete', 'field_context', + ] +) + + +class ImportContext(import_context_tuple): + def with_field_context(self, *args): + return ImportContext(*(self[:-1] + (field_context(*args),))) + + +field_context = namedtuple( + 'field_context', ['record_model', 'field_name', 'record_id'], +) + + +mapping_key = namedtuple('mapping_key', ['model_name', 'remote_id']) + + +dummy_instance = namedtuple( + 'dummy_instance', ['model_name', 'field_name', 'remote_id', 'dummy_id'], +) + + +class ImportOdooDatabase(models.Model): + _name = 'import.odoo.database' + _description = 'An Odoo database to import' + + url = fields.Char(required=True) + database = fields.Char(required=True) + user = fields.Char(default='admin', required=True) + password = fields.Char(default='admin') + import_line_ids = fields.One2many( + 'import.odoo.database.model', 'database_id', string='Import models', + ) + import_field_mappings = fields.One2many( + 'import.odoo.database.field', 'database_id', string='Field mappings', + ) + cronjob_id = fields.Many2one( + 'ir.cron', string='Import job', readonly=True, copy=False, + ) + cronjob_running = fields.Boolean(compute='_compute_cronjob_running') + status_data = fields.Serialized('Status', readonly=True, copy=False) + status_html = fields.Html( + compute='_compute_status_html', readonly=True, sanitize=False, + ) + + @api.multi + def action_import(self): + """Create a cronjob to run the actual import""" + self.ensure_one() + if self.cronjob_id: + return self.cronjob_id.write({ + 'numbercall': 1, + 'doall': True, + 'active': True, + }) + return self.write({ + 'cronjob_id': self._create_cronjob().id, + }) + + @api.multi + def _run_import(self, commit=True, commit_threshold=100): + """Run the import as cronjob, commit often""" + self.ensure_one() + if not self.password: + return + # model name: [ids] + remote_ids = {} + # model name: count + remote_counts = {} + # model name: count + done = {} + # mapping_key: local_id + idmap = {} + # mapping_key: local_id + # this are records created or linked when we need to fill a required + # field, but the local record is not yet created + dummies = {} + # model name: [local_id] + # this happens when we create a dummy we can throw away again + to_delete = {} + # dummy_instance + dummy_instances = [] + remote = self._get_connection() + self.write({'password': False}) + if commit and not tools.config['test_enable']: + # pylint: disable=invalid-commit + self.env.cr.commit() + for model_line in self.import_line_ids: + model = model_line.model_id + remote_ids[model.model] = remote.search( + model.model, + tools.safe_eval(model_line.domain) if model_line.domain else [] + ) + remote_counts[model.model] = len(remote_ids[model.model]) + self.write({ + 'status_data': { + 'counts': remote_counts, + 'ids': remote_ids, + 'error': None, + 'done': {}, + } + }) + if commit and not tools.config['test_enable']: + # pylint: disable=invalid-commit + self.env.cr.commit() + for model_line in self.import_line_ids: + model = self.env[model_line.model_id.model] + done[model._name] = 0 + + for start_index in range( + len(remote_ids[model._name]) / commit_threshold + 1 + ): + index = start_index * commit_threshold + ids = remote_ids[model._name][index:index + commit_threshold] + context = ImportContext( + remote, model_line, ids, idmap, dummies, dummy_instances, + to_delete, field_context(None, None, None), + ) + try: + self._run_import_model(context) + except: + error = traceback.format_exc() + self.env.cr.rollback() + self.write({ + 'status_data': dict(self.status_data, error=error), + }) + # pylint: disable=invalid-commit + self.env.cr.commit() + raise + done[model._name] += len(ids) + self.write({'status_data': dict(self.status_data, done=done)}) + if commit and not tools.config['test_enable']: + # pylint: disable=invalid-commit + self.env.cr.commit() + + @api.multi + def _run_import_model(self, context): + """Import records of a configured model""" + model = self.env[context.model_line.model_id.model] + fields = self._run_import_model_get_fields(context) + for data in context.remote.read( + model._name, context.ids, fields.keys() + ): + self._run_import_get_record(context, model, data) + if (model._name, data['id']) in context.idmap: + # there's a mapping for this record, nothing to do + continue + data = self._run_import_map_values(context, data) + _id = data['id'] + record = self._create_record(context, model, data) + self._run_import_model_cleanup_dummies( + context, model, _id, record.id, + ) + + @api.multi + def _create_record(self, context, model, record): + """Create a record, add an xmlid""" + _id = record.pop('id') + xmlid = '%d-%s-%d' % ( + self.id, model._name.replace('.', '_'), _id, + ) + if self.env.ref('base_import_odoo.%s' % xmlid, False): + new = self.env.ref('base_import_odoo.%s' % xmlid) + new.with_context( + **self._create_record_context(model, record) + ).write(record) + else: + new = model.with_context( + **self._create_record_context(model, record) + ).create(record) + self.env['ir.model.data'].create({ + 'name': xmlid, + 'model': model._name, + 'module': 'base_import_odoo', + 'res_id': new.id, + 'noupdate': True, + 'import_database_id': self.id, + 'import_database_record_id': _id, + }) + context.idmap[mapping_key(model._name, _id)] = new.id + return new + + def _create_record_context(self, model, record): + """Return a context that is used when creating a record""" + context = { + 'tracking_disable': True, + } + if model._name == 'res.users': + context['no_reset_password'] = True + return context + + @api.multi + def _run_import_get_record( + self, context, model, record, create_dummy=True, + ): + """Find the local id of some remote record. Create a dummy if not + available""" + _id = context.idmap.get((model._name, record['id'])) + if not _id: + _id = context.dummies.get((model._name, record['id'])) + if _id: + context.dummy_instances.append( + dummy_instance(*(context.field_context + (_id,))) + ) + if not _id: + mapping = self.import_field_mappings.filtered( + lambda x: x.model_id.model == model._name and + ( + not x.fields_id or + x.fields_id.name == context.field_context.field_name and + x.fields_id.model_id.model == + context.field_context.record_model + ) and + x.local_id and + (x.remote_id == record['id'] or not x.remote_id) + )[:1] + if mapping: + if mapping.local_id: + _id = mapping.local_id + context.idmap[(model._name, record['id'])] = _id + else: + _id = self._run_import_create_dummy( + context, model, record, forcecreate=True, + ) + if not _id: + xmlid = self.env['ir.model.data'].search([ + ('import_database_id', '=', self.id), + ('import_database_record_id', '=', record['id']), + ('model', '=', model._name), + ], limit=1) + if xmlid: + _id = xmlid.res_id + context.idmap[(model._name, record['id'])] = _id + if not _id and create_dummy: + _id = self._run_import_create_dummy(context, model, record) + return _id + + @api.multi + def _run_import_create_dummy( + self, context, model, record, forcecreate=False, + ): + """Either misuse some existing record or create an empty one to satisfy + required links""" + dummy = model.search([ + ( + 'id', 'not in', + [ + v for (model_name, remote_id), v + in context.dummies.iteritems() + if model_name == model._name + ] + ), + ], limit=1) + if dummy and not forcecreate: + context.dummies[mapping_key(model._name, record['id'])] = dummy.id + context.dummy_instances.append( + dummy_instance(*(context.field_context + (dummy.id,))) + ) + return dummy.id + required = [ + name + for name, field in model._fields.iteritems() + if field.required + ] + defaults = model.default_get(required) + values = {'id': record['id']} + for name, field in model._fields.iteritems(): + if name not in required or name in defaults: + continue + value = None + if field.type in ['char', 'text', 'html']: + value = '' + elif field.type in ['boolean']: + value = False + elif field.type in ['integer', 'float']: + value = 0 + elif model._fields[name].type in ['date', 'datetime']: + value = '2000-01-01' + elif field.type in ['many2one']: + new_context = context.with_field_context( + model._name, name, record['id'] + ) + value = self._run_import_get_record( + new_context, + self.env[model._fields[name].comodel_name], + {'id': record.get(name, [None])[0]}, + ) + elif field.type in ['selection'] and not callable(field.selection): + value = field.selection[0][0] + elif field.type in ['selection'] and callable(field.selection): + value = field.selection(model)[0][0] + # TODO: support more types, refactor to one function per type + values[name] = value + dummy = self._create_record(context, model, values) + context.dummies[mapping_key(model._name, record['id'])] = dummy.id + context.to_delete.setdefault(model._name, []) + context.to_delete[model._name].append(dummy.id) + context.dummy_instances.append( + dummy_instance(*(context.field_context + (dummy.id,))) + ) + return dummy.id + + @api.multi + def _run_import_map_values(self, context, data): + model = self.env[context.model_line.model_id.model] + for field_name in data.keys(): + if not isinstance( + model._fields[field_name], fields._Relational + ) or not data[field_name]: + continue + if model._fields[field_name].type == 'one2many': + # don't import one2many fields, use an own configuration + # for this + data.pop(field_name) + continue + ids = data[field_name] if ( + model._fields[field_name].type != 'many2one' + ) else [data[field_name][0]] + new_context = context.with_field_context( + model._name, field_name, data['id'] + ) + data[field_name] = [ + self._run_import_get_record( + new_context, + self.env[model._fields[field_name].comodel_name], + {'id': _id}, + create_dummy=model._fields[field_name].required, + ) + for _id in ids + ] + data[field_name] = filter(None, data[field_name]) + if model._fields[field_name].type == 'many2one': + if data[field_name]: + data[field_name] = data[field_name] and data[field_name][0] + else: + data[field_name] = None + else: + data[field_name] = [(6, 0, data[field_name])] + for mapping in self.import_field_mappings: + if mapping.model_id.model != model._name or not mapping.fields_id: + continue + if mapping.unique: + value = data.get(mapping.fields_id.name, '') + counter = 1 + while model.with_context(active_test=False).search([ + ( + mapping.fields_id.name, '=', + data.get(mapping.fields_id.name, value) + ), + ]): + data[mapping.fields_id.name] = '%s (%d)' % (value, counter) + counter += 1 + return data + + @api.multi + def _run_import_model_get_fields(self, context): + return { + name: field + for name, field + in self.env[context.model_line.model_id.model]._fields.iteritems() + if not field.compute or field.related + } + + @api.multi + def _run_import_model_cleanup_dummies( + self, context, model, remote_id, local_id + ): + for instance in context.dummy_instances: + if ( + instance.model_name != model._name or + instance.remote_id != remote_id + ): + continue + if not context.idmap.get(instance.remote_id): + continue + model = self.env[instance.model_name] + record = model.browse(context.idmap[instance.remote_id]) + field_name = instance.field_id.name + if record._fields[field_name].type == 'many2one': + record.write({field_name: local_id}) + elif record._fields[field_name].type == 'many2many': + record.write({field_name: [ + (3, context.idmap[remote_id]), + (4, local_id), + ]}) + else: + raise exceptions.UserError( + _('Unhandled field type %s') % + record._fields[field_name].type + ) + context.dummy_instances.remove(instance) + dummy_id = context.dummies[(record._model, remote_id)] + if dummy_id in context.to_delete: + model.browse(dummy_id).unlink() + del context.dummies[(record._model, remote_id)] + + def _get_connection(self): + self.ensure_one() + return Client(self.url, self.database, self.user, self.password) + + @api.constrains('url', 'database', 'user', 'password') + @api.multi + def _constrain_url(self): + for this in self: + if this == self.env.ref('base_import_odoo.demodb', False): + continue + if tools.config['test_enable']: + continue + if not this.password: + continue + this._get_connection() + + @api.depends('status_data') + @api.multi + def _compute_status_html(self): + for this in self: + if not this.status_data: + continue + this.status_html = self.env.ref( + 'base_import_odoo.view_import_odoo_database_qweb' + ).render({'object': this}) + + @api.depends('cronjob_id') + @api.multi + def _compute_cronjob_running(self): + for this in self: + if not this.cronjob_id: + continue + try: + with self.env.cr.savepoint(): + self.env.cr.execute( + 'select id from "%s" where id=%%s for update nowait' % + self.env['ir.cron']._table, + (this.cronjob_id.id,), log_exceptions=False, + ) + except psycopg2.OperationalError: + this.cronjob_running = True + + @api.multi + def _create_cronjob(self): + self.ensure_one() + return self.env['ir.cron'].create({ + 'name': self.display_name, + 'model': self._name, + 'function': '_run_import', + 'doall': True, + 'args': str((self.ids,)), + }) + + @api.multi + def name_get(self): + return [ + (this.id, '%s@%s, %s' % (this.user, this.url, this.database)) + for this in self + ] diff --git a/base_import_odoo/models/import_odoo_database_field.py b/base_import_odoo/models/import_odoo_database_field.py new file mode 100644 index 000000000..ff823a9b1 --- /dev/null +++ b/base_import_odoo/models/import_odoo_database_field.py @@ -0,0 +1,35 @@ +# -*- coding: utf-8 -*- +# © 2017 Therp BV +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). +from openerp import fields, models + + +class ImportOdooDatabaseField(models.Model): + _name = 'import.odoo.database.field' + _description = 'A field mapping for records in the remote database' + + database_id = fields.Many2one( + 'import.odoo.database', string='Database', required=True, + ondelete='cascade', + ) + model_id = fields.Many2one( + 'ir.model', string='Model', required=True, ondelete='cascade', + ) + fields_id = fields.Many2one( + 'ir.model.fields', string='Field', help='If set, the mapping is only ' + 'effective when setting said field', ondelete='cascade', + ) + unique = fields.Boolean( + 'Unique', help='If set on a char field, a number is appended until ' + 'the value is unique. Set this for fields with uniqueness constraints', + ) + # TODO: create a reference function field to set this conveniently + local_id = fields.Integer( + 'Local ID', help='If you leave this empty, a new record will be ' + 'created in the local database when this field is set on the remote ' + 'database' + ) + remote_id = fields.Integer( + 'Remote ID', help='If you leave this empty, every (set) field value ' + 'will be mapped to the local ID' + ) diff --git a/base_import_odoo/models/import_odoo_database_model.py b/base_import_odoo/models/import_odoo_database_model.py new file mode 100644 index 000000000..2394ebc65 --- /dev/null +++ b/base_import_odoo/models/import_odoo_database_model.py @@ -0,0 +1,20 @@ +# -*- coding: utf-8 -*- +# © 2017 Therp BV +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). +from openerp import fields, models + + +class ImportOdooDatabaseModel(models.Model): + _name = 'import.odoo.database.model' + _description = 'A model to import from a remote database' + _order = 'sequence' + + sequence = fields.Integer() + model_id = fields.Many2one( + 'ir.model', string='Model', required=True, ondelete='cascade', + ) + database_id = fields.Many2one( + 'import.odoo.database', string='Database', required=True, + ondelete='cascade', + ) + domain = fields.Char(help='Optional filter to import only a subset') diff --git a/base_import_odoo/models/ir_model_data.py b/base_import_odoo/models/ir_model_data.py new file mode 100644 index 000000000..2348232cd --- /dev/null +++ b/base_import_odoo/models/ir_model_data.py @@ -0,0 +1,13 @@ +# -*- coding: utf-8 -*- +# © 2017 Therp BV +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). +from openerp import fields, models + + +class IrModelData(models.Model): + _inherit = 'ir.model.data' + + import_database_id = fields.Many2one( + 'import.odoo.database', string='From remote database', + ) + import_database_record_id = fields.Integer('Remote database ID') diff --git a/base_import_odoo/security/ir.model.access.csv b/base_import_odoo/security/ir.model.access.csv new file mode 100644 index 000000000..b2413c5f8 --- /dev/null +++ b/base_import_odoo/security/ir.model.access.csv @@ -0,0 +1,4 @@ +"id","name","model_id:id","group_id:id","perm_read","perm_write","perm_create","perm_unlink" +access_import_odoo_database,access_import_odoo_database,model_import_odoo_database,base.group_system,1,1,1,1 +access_import_odoo_database_model,access_import_odoo_database_model,model_import_odoo_database_model,base.group_system,1,1,1,1 +access_import_odoo_database_field,access_import_odoo_database_field,model_import_odoo_database_field,base.group_system,1,1,1,1 diff --git a/base_import_odoo/static/description/icon.png b/base_import_odoo/static/description/icon.png new file mode 100644 index 000000000..3a0328b51 Binary files /dev/null and b/base_import_odoo/static/description/icon.png differ diff --git a/base_import_odoo/tests/__init__.py b/base_import_odoo/tests/__init__.py new file mode 100644 index 000000000..905385a18 --- /dev/null +++ b/base_import_odoo/tests/__init__.py @@ -0,0 +1,4 @@ +# -*- coding: utf-8 -*- +# © 2017 Therp BV +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). +from . import test_base_import_odoo diff --git a/base_import_odoo/tests/test_base_import_odoo.py b/base_import_odoo/tests/test_base_import_odoo.py new file mode 100644 index 000000000..f3201ecdc --- /dev/null +++ b/base_import_odoo/tests/test_base_import_odoo.py @@ -0,0 +1,68 @@ +# -*- coding: utf-8 -*- +# © 2017 Therp BV +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). +from mock import patch +from openerp.tests.common import TransactionCase, post_install, at_install + + +class TestBaseImportOdoo(TransactionCase): + @at_install(False) + @post_install(True) + @patch('erppeek.Client.__init__', side_effect=lambda *args: None) + def test_base_import_odoo(self, mock_client_init): + # the mocked functions simply search/read in the current database + # the effect then should be that the models in question are duplicated, + # we just need to try not to be confused by the fact that local and + # remote ids are the same + def _mock_search( + model, domain, offset=0, limit=None, order=None, context=None, + ): + return self.env[model].with_context( + **(context or self.env.context) + ).search( + domain, offset=offset, limit=limit, order=order, + ).ids + + def _mock_read( + model, domain_or_ids, fields=None, offset=0, limit=None, + order=None, context=None, + ): + return self.env[model].with_context( + **(context or self.env.context) + ).browse(domain_or_ids).read(fields=fields) + + for dummy in range(2): + # we run this two times to enter the code path where xmlids exist + self.env.ref('base_import_odoo.demodb').write({ + 'password': 'admin', + }) + with patch('erppeek.Client.search', side_effect=_mock_search): + with patch('erppeek.Client.read', side_effect=_mock_read): + self.env.ref('base_import_odoo.demodb')._run_import() + # here the actual test begins - check that we created new + # objects, check xmlids, check values, check if dummies are + # cleaned up/replaced + self.assertNotEqual( + self.env.ref(self._get_xmlid('base.user_demo')), + self.env.ref('base.user_demo'), + ) + self.assertEqual( + dict(self.env.ref(self._get_xmlid('base.user_demo'))._cache), + dict(self.env.ref('base.user_demo')._cache), + ) + # TODO: test much more + demodb = self.env.ref('base_import_odoo.demodb') + demodb.action_import() + self.assertTrue(demodb.cronjob_id) + demodb.cronjob_id.write({'active': False}) + demodb.action_import() + self.assertTrue(demodb.cronjob_id.active) + self.assertFalse(demodb.cronjob_running) + + def _get_xmlid(self, remote_xmlid): + remote_obj = self.env.ref(remote_xmlid) + return 'base_import_odoo.%d-%s-%s' % ( + self.env.ref('base_import_odoo.demodb').id, + remote_obj._name.replace('.', '_'), + remote_obj.id, + ) diff --git a/base_import_odoo/views/import_odoo_database.xml b/base_import_odoo/views/import_odoo_database.xml new file mode 100644 index 000000000..d5c67077b --- /dev/null +++ b/base_import_odoo/views/import_odoo_database.xml @@ -0,0 +1,89 @@ + + + + + import.odoo.database + + + + + + + + + import.odoo.database + +
+
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + +
+ + + + + diff --git a/base_import_odoo/views/menu.xml b/base_import_odoo/views/menu.xml new file mode 100644 index 000000000..42b79797a --- /dev/null +++ b/base_import_odoo/views/menu.xml @@ -0,0 +1,12 @@ + + + + + + + +