Issue 1635639: ConfigParser does not quote % (original) (raw)

Created on 2007-01-15 07:43 by mark-roberts, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
bug_1603688_cfgparser_errors.patch mark-roberts,2007-01-15 07:43 ValueError version of the patch
bug_1603688_cfgparser_munges.patch mark-roberts,2007-01-15 07:44 Munging version of the patch
Messages (5)
msg51746 - (view) Author: Mark Roberts (mark-roberts) Date: 2007-01-15 07:43
This is covered by bug 1603688 (https://sourceforge.net/tracker/?func=detail&atid=105470&aid=1603688&group_id=5470) I implemented 2 versions of this patch. One version raises ValueError when an invalid interpolation syntax is encountered (such as foo%, foo%bar, and %foo, but not %%foo and %(dir)foo). The other version simply replaces appropriate %s with %%s. Initially, I believed ValueError was the appropriate way to go with this. However, when I thought about how I use ConfigParser, I realized that it would be far nicer if it simply worked. I'm +0.5 to ValueError, and +1 to munging the values.
msg51747 - (view) Author: Mark Roberts (mark-roberts) Date: 2007-01-15 07:44
File Added: bug_1603688_cfgparser_munges.patch
msg51748 - (view) Author: Mark Roberts (mark-roberts) Date: 2007-01-16 02:17
For the record, this was supposed to be a patch. I don't know if the admins have any way of moving it to that category. I guess that explained the funky categories and groups. Sorry for the inconvenience.
msg51749 - (view) Author: A.M. Kuchling (akuchling) * (Python committer) Date: 2007-01-22 19:44
Turning into a patch.
msg51750 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2007-03-13 17:44
Explicit is better than implicit. I committed a similar patch that raises ValueError, and your new test cases in rev. 54339.
History
Date User Action Args
2022-04-11 14:56:22 admin set github: 44469
2007-01-15 07:43:36 mark-roberts create