This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author patrick.vrijlandt
Recipients docs@python, patrick.vrijlandt
Date 2012-01-19.08:51:45
SpamBayes Score 5.353302e-09
Marked as misclassified No
Message-id <1326963107.43.0.928827185581.issue13823@psf.upfronthosting.co.za>
In-reply-to
Content
(1) The docs say:
xml_declaration controls if an XML declaration should be added to the file. Use False for never, True for always, None for only if not US-ASCII or UTF-8 or Unicode (default is None). 

The method also accepts other values, like xml_declaration = "yes". This behavior should be documented, or raise a ValueError (up to effbot, I think)

(2) The docs say (in a note):
The encoding string included in XML output should conform to the appropriate standards. For example, “UTF-8” is valid, but “UTF8” is not. See http://www.w3.org/

But the method accepts both values, (“UTF-8” and “UTF8”). Since this will result in invalid xml, (but not invalid python) it should probably raise ValueError too.

(3) Open issue 9458 also refers to this method. It might be wise to raise ValueError if the encoding does not match the (mode of the) file target (binary or text).
History
Date User Action Args
2012-01-19 08:51:47patrick.vrijlandtsetrecipients: + patrick.vrijlandt, docs@python
2012-01-19 08:51:47patrick.vrijlandtsetmessageid: <1326963107.43.0.928827185581.issue13823@psf.upfronthosting.co.za>
2012-01-19 08:51:46patrick.vrijlandtlinkissue13823 messages
2012-01-19 08:51:45patrick.vrijlandtcreate