diff options
| author | Alexander Bokovoy <abokovoy@redhat.com> | 2011-08-17 15:46:41 +0300 |
|---|---|---|
| committer | Rob Crittenden <rcritten@redhat.com> | 2011-08-17 02:05:52 -0400 |
| commit | 821fa37cb2079606995d36b3b195df9096d4cb8a (patch) | |
| tree | 19505c248e322b1bf88e83255761d961ba300c2f /ipalib/frontend.py | |
| parent | a4b025404da2b707aaeea655e7c9298b5697e04c (diff) | |
| download | freeipa-821fa37cb2079606995d36b3b195df9096d4cb8a.tar.gz freeipa-821fa37cb2079606995d36b3b195df9096d4cb8a.tar.xz freeipa-821fa37cb2079606995d36b3b195df9096d4cb8a.zip | |
Propagate environment when it is required.
https://fedorahosted.org/freeipa/ticket/1549
https://fedorahosted.org/freeipa/ticket/1550
Diffstat (limited to 'ipalib/frontend.py')
| -rw-r--r-- | ipalib/frontend.py | 7 |
1 files changed, 7 insertions, 0 deletions
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): |