From c911cb8064bd31ddbdcbc5f272c34f8e9e2f76cb Mon Sep 17 00:00:00 2001 From: Daniel Date: Thu, 23 Feb 2017 10:28:21 +0100 Subject: [PATCH] [FIX] mrp_operations_extension: Don't check option "Calc Cycles by BoM Quantity" even if not desired. --- mrp_operations_extension/models/res_config.py | 9 +++------ .../tests/test_mrp_operations_extension.py | 15 +++++++++++++++ 2 files changed, 18 insertions(+), 6 deletions(-) diff --git a/mrp_operations_extension/models/res_config.py b/mrp_operations_extension/models/res_config.py index d7cf55080..5bad75f6b 100644 --- a/mrp_operations_extension/models/res_config.py +++ b/mrp_operations_extension/models/res_config.py @@ -22,12 +22,9 @@ class MrpConfigSettings(models.TransientModel): param_obj = self.env['ir.config_parameter'] rec = self._get_parameter(key) if rec: - if not value: - rec.unlink() - else: - rec.value = value - elif value: - param_obj.create({'key': key, 'value': value}) + rec.value = str(value) + else: + param_obj.create({'key': key, 'value': str(value)}) @api.multi def get_default_parameter_cycle_bom(self): diff --git a/mrp_operations_extension/tests/test_mrp_operations_extension.py b/mrp_operations_extension/tests/test_mrp_operations_extension.py index df908c820..492549d7e 100644 --- a/mrp_operations_extension/tests/test_mrp_operations_extension.py +++ b/mrp_operations_extension/tests/test_mrp_operations_extension.py @@ -230,3 +230,18 @@ class TestMrpOperationsExtension(common.TransactionCase): 'Error work order moves quantity do not match') self.assertEqual(workorder2.state, 'done', 'Error work center line not in done state') + + def test_param_config(self): + wiz_config_obj = self.env['mrp.config.settings'] + param_obj = self.env['ir.config_parameter'] + rec = param_obj.search([('key', '=', 'cycle.by.bom')]) + self.assertEqual(rec.value, 'True', 'Error cycle.by.bom is not marked') + rec.unlink() + record = wiz_config_obj.new() + record.set_parameter_cycle_bom() + rec = param_obj.search([('key', '=', 'cycle.by.bom')]) + self.assertEqual(rec.value, 'False', 'Error cycle.by.bom is marked') + record.cycle_by_bom = True + record.set_parameter_cycle_bom() + rec = param_obj.search([('key', '=', 'cycle.by.bom')]) + self.assertEqual(rec.value, 'True', 'Error cycle.by.bom not marked')