From c47518d9f1bc4df781b5a5a458db3c02dfb608e5 Mon Sep 17 00:00:00 2001 From: Daniel-CA Date: Tue, 28 Mar 2017 12:45:38 +0200 Subject: [PATCH] [8.0][FIX] mrp_operations_extension: Check option "Calc Cycles by BoM Quantity" does not update correctly + Test update --- mrp_operations_extension/models/res_config.py | 2 +- .../tests/test_mrp_operations_extension.py | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/mrp_operations_extension/models/res_config.py b/mrp_operations_extension/models/res_config.py index 5bad75f6b..2f011bec8 100644 --- a/mrp_operations_extension/models/res_config.py +++ b/mrp_operations_extension/models/res_config.py @@ -30,7 +30,7 @@ class MrpConfigSettings(models.TransientModel): def get_default_parameter_cycle_bom(self): def get_value(key, default=''): rec = self._get_parameter(key) - return rec and rec.value or default + return rec and rec.value and rec.value != 'False' or default return {'cycle_by_bom': get_value('cycle.by.bom', False)} @api.multi diff --git a/mrp_operations_extension/tests/test_mrp_operations_extension.py b/mrp_operations_extension/tests/test_mrp_operations_extension.py index 492549d7e..67374d4b0 100644 --- a/mrp_operations_extension/tests/test_mrp_operations_extension.py +++ b/mrp_operations_extension/tests/test_mrp_operations_extension.py @@ -239,9 +239,9 @@ class TestMrpOperationsExtension(common.TransactionCase): 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') + data = record.get_default_parameter_cycle_bom() + self.assertFalse(data['cycle_by_bom'], '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') + data = record.get_default_parameter_cycle_bom() + self.assertTrue(data['cycle_by_bom'], 'Error cycle.by.bom not marked')