]> granicus.if.org Git - python/commitdiff
Fix namedtuple bug reported by Glenn Linderman. Template did not form correctly...
authorRaymond Hettinger <python@rcn.com>
Thu, 25 Sep 2008 23:31:52 +0000 (23:31 +0000)
committerRaymond Hettinger <python@rcn.com>
Thu, 25 Sep 2008 23:31:52 +0000 (23:31 +0000)
Lib/collections.py
Lib/test/test_collections.py

index 24088183900a30f43c3242ddc33bd81ba88b6bf6..ace2b2a446b6229977a81b105755a78acbb7e3c1 100644 (file)
@@ -38,7 +38,7 @@ def namedtuple(typename, field_names, verbose=False):
     # generating informative error messages and preventing template injection attacks.
     if isinstance(field_names, basestring):
         field_names = field_names.replace(',', ' ').split() # names separated by whitespace and/or commas
-    field_names = tuple(field_names)
+    field_names = tuple(map(str, field_names))
     for name in (typename,) + field_names:
         if not all(c.isalnum() or c=='_' for c in name):
             raise ValueError('Type names and field names can only contain alphanumeric characters and underscores: %r' % name)
index d689add41d20266b256d92c5f531daad461d42cf..7dffd73720ba6a435427a5eeef1bb573d2545b30 100644 (file)
@@ -34,6 +34,11 @@ class TestNamedTuple(unittest.TestCase):
         namedtuple('Point0', 'x1 y2')   # Verify that numbers are allowed in names
         namedtuple('_', 'a b c')        # Test leading underscores in a typename
 
+        nt = namedtuple('nt', u'the quick brown fox')                       # check unicode input
+        self.assert_("u'" not in repr(nt._fields))
+        nt = namedtuple('nt', (u'the', u'quick'))                           # check unicode input
+        self.assert_("u'" not in repr(nt._fields))
+
         self.assertRaises(TypeError, Point._make, [11])                     # catch too few args
         self.assertRaises(TypeError, Point._make, [11, 22, 33])             # catch too many args