Bugs item #3399799, was opened at 2011-08-28 10:13 Message generated for change (Comment added) made by bachinchi You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=603138&aid=3399799...

Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Nobody/Anonymous (nobody) Assigned to: Nobody/Anonymous (nobody) Summary: Self closing tags

Initial Comment: Since MW 1.16 wgHtm5 and wgWellFormedXml are turned on by default. And cosmetic_changes.py still changes things like <br> to <br />, I don't see any reason for that. There are three options we have here: * Stop making these kind of changes * Make these changes "complete" and add some tags that are not fixed (eg. <hr> to <hr />) * Do just the opposite, and let the MW fix the tags (eg. a<br/> to <br>, etc) I like the third option, because it makes the code easier to read for editors. In the attachment there's a patch to apply the third option.

python version.py: Pywikipedia [http] trunk/pywikipedia (r9462, 2011/08/28, 11:34:03) Python 2.7 (r27:82500, Aug 07 2010, 16:54:59) [GCC] config-settings: use_api = True use_api_login = True

----------------------------------------------------------------------

Comment By: B Baron (bachinchi) Date: 2011-08-28 10:35

Message: Ups, didn't remember that (since I have seen plenty of hr in templates), so you can't ignore the second line in patch.

----------------------------------------------------------------------

Comment By: xqt (xqt) Date: 2011-08-28 10:27

Message: <hr> will be replaced by mediawiki syntax ---- look at cosmetic_changes.fixHtml()

----------------------------------------------------------------------

You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=603138&aid=3399799...