From: Fred Drake <fdrake@acm.org>
Date: Tue, 31 Dec 2002 06:55:41 +0000 (+0000)
Subject: ConfigParser._interpolate():  Pass the missing key to the
X-Git-Tag: v2.3c1~2748
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=00dc5a93c12484c3df57bd3bf2e0c6e78345a98b;p=python

ConfigParser._interpolate():  Pass the missing key to the
    InterpolationError constructor, not the KeyError exception itself.
    (Caught by the new InterpolationError test.)

SafeConfigParser._interpolate_some():  Pass the right number of
    arguments to the InterpolationError constructor.
    (Caught by pychecker.)
---

diff --git a/Lib/ConfigParser.py b/Lib/ConfigParser.py
index 24a4f23eb3..7c2eba5cff 100644
--- a/Lib/ConfigParser.py
+++ b/Lib/ConfigParser.py
@@ -554,8 +554,8 @@ class ConfigParser(RawConfigParser):
             if value.find("%(") != -1:
                 try:
                     value = value % vars
-                except KeyError, key:
-                    raise InterpolationError(key, option, section, rawval)
+                except KeyError, e:
+                    raise InterpolationError(e[0], option, section, rawval)
             else:
                 break
         if value.find("%(") != -1:
@@ -599,8 +599,7 @@ class SafeConfigParser(ConfigParser):
                 try:
                     v = map[var]
                 except KeyError:
-                    raise InterpolationError(
-                        "no value found for %r" % var)
+                    raise InterpolationError(var, option, section, rest)
                 if "%" in v:
                     self._interpolate_some(option, accum, v,
                                            section, map, depth + 1)