Last modified: 2014-07-24 12:49:51 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T57222, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 55222 - cosmetic_changes.py and <code>
cosmetic_changes.py and <code>
Status: NEW
Product: Pywikibot
Classification: Unclassified
Cosmetic changes (Other open bugs)
unspecified
All All
: Normal normal
: ---
Assigned To: Pywikipedia bugs
:
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2013-10-05 04:45 UTC by Kunal Mehta (Legoktm)
Modified: 2014-07-24 12:49 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Kunal Mehta (Legoktm) 2013-10-05 04:45:47 UTC 
Originally from: http://sourceforge.net/p/pywikipediabot/bugs/1393/
Reported by: ganz-ru
Created on: 2012-01-14 17:37:31
Subject: cosmetic_changes.py and <code>
Original description:
Here:  
http://ru.wikipedia.org/w/index.php?diff=40245627&oldid=39731202 
http://ru.wikipedia.org/w/index.php?diff=40765267&oldid=39425330
cosmetic\_changes.py replaced the internal internal html-text of &lt;code&gt;...&lt;/code&gt; with unicode symbols. That's not good, since the tag &lt;code&gt; is for an original representation of codes, including the html-code. So, I propose to add this tag to the elist of exceptions.
Comment 1 Kunal Mehta (Legoktm) 2013-10-05 04:45:49 UTC 
- **assigned_to**: nobody --> xqt
- **status**: open --> open-accepted
Comment 2 Kunal Mehta (Legoktm) 2013-10-05 04:45:51 UTC 
As I see, it just necessary to add this tag into arrays of exceptions and it does not requir editing of any other parts of code. So please accelerate the processing of of this bug.
Comment 3 Kunal Mehta (Legoktm) 2013-10-05 04:45:52 UTC 
Feel free to submit a patch.
Comment 4 Kunal Mehta (Legoktm) 2013-10-05 04:45:54 UTC 
@ganz-du: unfortunatelly this part of code does not use replaceExcept\(\) and we have no exceptions list where to add that tag.
Comment 5 Kunal Mehta (Legoktm) 2013-10-05 04:45:56 UTC 
- **assigned_to**: xqt --> nobody
Comment 6 Kunal Mehta (Legoktm) 2013-10-05 04:45:58 UTC 
duplicated by bug \#3530791
raising prio.
Comment 7 Kunal Mehta (Legoktm) 2013-10-05 04:45:59 UTC 
- **priority**: 5 --> 6
Comment 8 Kunal Mehta (Legoktm) 2013-10-05 04:46:01 UTC 
CosmeticChangesToolkit.resolveHtmlEntities\(\) deactivated in pyrev:10438 due to this bug
Comment 9 Kunal Mehta (Legoktm) 2013-10-05 04:46:03 UTC 
- **priority**: 6 --> 7
Comment 10 Kunal Mehta (Legoktm) 2013-10-05 04:46:05 UTC 
- **priority**: 7 --> 5
Comment 11 Amir Ladsgroup 2014-07-24 12:49:51 UTC 
It happens because of the html2unicode function, it can be fixed either in textlib or cosmetic_changes.py
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links