Last modified: 2014-09-09 15:45:58 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 T70361, 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 68361 - Section edit containing "/*" is mis-parsed
Section edit containing "/*" is mis-parsed
Status: PATCH_TO_REVIEW
Product: MediaWiki
Classification: Unclassified
Parser (Other open bugs)
unspecified
All All
: Low normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2014-07-22 04:17 UTC by MZMcBride
Modified: 2014-09-09 15:45 UTC (History)
4 users (show)

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

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

Description MZMcBride 2014-07-22 04:17:06 UTC 
From <https://www.mediawiki.org/w/index.php?title=Gerrit/Project_ownership&action=history>:

---
<span class="comment">(/* PleaseStand for +2 for mediawiki<a href="/wiki/Gerrit/Project_ownership" title="Gerrit/Project ownership">→</a>‎<span dir="auto"><span class="autocomment">: </span> new section</span>)</span>
---

The link is in the wrong place. It seems to be picking up "PleaseStand for +2 for mediawiki/*" and treating the "/*" in "mediawiki/*" as the start of the section.

The pre-filled section-edit summary field populates correctly, making me think that the database is storing this data correctly. That makes this is a parsing bug then, I believe. Perhaps we can tweak the section-edit edit summary parser to be smarter with this edge case?
Comment 1 Gerrit Notification Bot 2014-09-08 15:14:52 UTC 
Change 147362 had a related patch set uploaded by Jackmcbarn:
Prefer earlier potential sections in summaries

https://gerrit.wikimedia.org/r/147362
Comment 2 Gerrit Notification Bot 2014-09-09 15:45:58 UTC 
Change 159326 had a related patch set uploaded by Anomie:
Format multiple autocomments in edit summaries

https://gerrit.wikimedia.org/r/159326
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