From 6c1a7cb4a54c3ff86c7a2137e50d9777fcbe18b4 Mon Sep 17 00:00:00 2001 From: Jeremy Fincher Date: Tue, 31 Aug 2004 15:40:23 +0000 Subject: [PATCH] Cleaner interface for Config.list. --- src/Config.py | 37 ++++++++++++------------------------- 1 file changed, 12 insertions(+), 25 deletions(-) diff --git a/src/Config.py b/src/Config.py index f6a50b4f6..f8aa45f2a 100644 --- a/src/Config.py +++ b/src/Config.py @@ -108,38 +108,25 @@ class Config(callbacks.Privmsg): name = 'supybot.' + name return name - def _list(self, name, groups=False): + def _list(self, name): name = self._canonicalizeName(name) group = getWrapper(name) - if groups: - L = [] - for (vname, v) in group._children.iteritems(): - if v._added: - L.append(vname) - utils.sortBy(str.lower, L) - return L - else: - try: - L = [t[0] for t in group.getValues(fullNames=False)] - utils.sortBy(str.lower, L) - return L - except TypeError: - return [] + L = [] + for (vname, v) in group._children.iteritems(): + if v._added: + vname = '@' + vname + L.append(vname) + utils.sortBy(str.lower, L) + return L def list(self, irc, msg, args): - """[--groups] + """ Returns the configuration variables available under the given - configuration . If --groups is given, return the subgroups of - the . + configuration . Subgroups are indicated by a preceding @. """ - (optlist, rest) = getopt.getopt(args, '', ['groups']) - groups = False - for (name, arg) in optlist: - if name == '--groups': - groups = True - name = privmsgs.getArgs(rest) - L = self._list(name, groups=groups) + name = privmsgs.getArgs(args) + L = self._list(name) if L: irc.reply(utils.commaAndify(L)) elif groups: