diff options
author | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2013-01-08 13:16:35 -0500 |
---|---|---|
committer | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2013-01-08 13:16:35 -0500 |
commit | 923e48916f19bedfe5b239b1baca18dc38c39491 (patch) | |
tree | 2d1db7141c14925e59262aae5f38eca3ddef9a2b /src/lib/Bcfg2/Server/Plugins/Base.py | |
parent | 40bfc1dc57dd6a05a01f72d6332af7d586a15768 (diff) | |
download | bcfg2-923e48916f19bedfe5b239b1baca18dc38c39491.tar.gz bcfg2-923e48916f19bedfe5b239b1baca18dc38c39491.tar.bz2 bcfg2-923e48916f19bedfe5b239b1baca18dc38c39491.zip |
replace unintuitive reduce() calls with slightly less unintuitive chain() calls
Diffstat (limited to 'src/lib/Bcfg2/Server/Plugins/Base.py')
-rw-r--r-- | src/lib/Bcfg2/Server/Plugins/Base.py | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/src/lib/Bcfg2/Server/Plugins/Base.py b/src/lib/Bcfg2/Server/Plugins/Base.py index 66515ede2..d662da60a 100644 --- a/src/lib/Bcfg2/Server/Plugins/Base.py +++ b/src/lib/Bcfg2/Server/Plugins/Base.py @@ -2,9 +2,8 @@ import copy import lxml.etree -import sys -from Bcfg2.Compat import reduce import Bcfg2.Server.Plugin +from itertools import chain class Base(Bcfg2.Server.Plugin.Plugin, @@ -18,7 +17,6 @@ class Base(Bcfg2.Server.Plugin.Plugin, __child__ = Bcfg2.Server.Plugin.StructFile deprecated = True - """Base creates independent clauses based on client metadata.""" def __init__(self, core, datastore): Bcfg2.Server.Plugin.Plugin.__init__(self, core, datastore) Bcfg2.Server.Plugin.Structure.__init__(self) @@ -33,8 +31,8 @@ class Base(Bcfg2.Server.Plugin.Plugin, def BuildStructures(self, metadata): """Build structures for client described by metadata.""" ret = lxml.etree.Element("Independent", version='2.0') - fragments = reduce(lambda x, y: x + y, - [base.Match(metadata) for base - in list(self.entries.values())], []) - [ret.append(copy.copy(frag)) for frag in fragments] + fragments = list(chain(*[base.Match(metadata) + for base in list(self.entries.values())])) + for frag in fragments: + ret.append(copy.copy(frag)) return [ret] |