Message170324
| Author |
chris.jerdonek |
| Recipients |
Arfrever, barry, benjamin.peterson, bethard, chris.jerdonek, georg.brandl |
| Date |
2012年09月11日.16:40:28 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1347381631.69.0.75948744462.issue15906@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
Along the lines of my previous comment, I notice that the following str type check was removed in the patch for issue 12776:
- if isinstance(action.default, str):
- default = self._get_value(action, default)
- setattr(namespace, action.dest, default)
But it was not preserved when the call to _get_value() was added back elsewhere:
+ if (action.default is not None and
+ hasattr(namespace, action.dest) and
+ action.default is getattr(namespace, action.dest)):
+ setattr(namespace, action.dest,
+ self._get_value(action, action.default)) |
|