From 821fa37cb2079606995d36b3b195df9096d4cb8a Mon Sep 17 00:00:00 2001 From: Alexander Bokovoy Date: Wed, 17 Aug 2011 15:46:41 +0300 Subject: Propagate environment when it is required. https://fedorahosted.org/freeipa/ticket/1549 https://fedorahosted.org/freeipa/ticket/1550 --- ipalib/frontend.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/ipalib/frontend.py b/ipalib/frontend.py index 35343105c..c2ae4e744 100644 --- a/ipalib/frontend.py +++ b/ipalib/frontend.py @@ -344,6 +344,13 @@ class HasParam(Plugin): for spec in get(): param = create_param(spec) if env is None or param.use_in_context(env): + if env is not None and not hasattr(param, 'env'): + # Force specified environment. The way it is done is violation of ReadOnly promise. + # Unfortunately, all alternatives are worse from both performance and code complexity + # points of view. See following threads on freeipa-devel@ for references: + # https://www.redhat.com/archives/freeipa-devel/2011-August/msg00000.html + # https://www.redhat.com/archives/freeipa-devel/2011-August/msg00011.html + object.__setattr__(param, 'env', env) yield param def _create_param_namespace(self, name, env=None): -- cgit