Last modified: 2012-04-16 09:15:40 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 T34802, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 32802 - $1 in message editsectionhint shouldn't be parsed
$1 in message editsectionhint shouldn't be parsed
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Internationalization (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch, patch-need-review
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-12-05 08:45 UTC by Liangent
Modified: 2012-04-16 09:15 UTC (History)
6 users (show)

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


Attachments
Patch (didn't test whether this breaks other stuff) (642 bytes, patch)
2012-01-26 21:25 UTC, Jelle Zijlstra
Details
Patch to fix the issue (990 bytes, patch)
2012-01-26 21:35 UTC, Brad Jorsch
Details

Description Liangent 2011-12-05 08:45:43 UTC
In this message, if $1 contains a template call (also magic words etc.), it shouldn't be expanded. (If a template call can be passed in, it must be already escaped somehow, eg., with <nowiki>). See <http://zh.wikipedia.org/w/index.php?title=Wikipedia:%E6%B2%99%E7%9B%92&oldid=18527982>
Comment 2 Brion Vibber 2011-12-05 16:09:27 UTC
Can you point out exactly what on those pages looks wrong?
Comment 3 Liangent 2011-12-05 16:36:53 UTC
(In reply to comment #2)
> Can you point out exactly what on those pages looks wrong?

See the tooltip of the "edit" link of the section and compare it with the section title.
Comment 4 Brion Vibber 2011-12-05 18:40:05 UTC
There is no section edit link on old page views such as your sample URLs.

If I go to the current version of https://zh.wikipedia.org/wiki/Wikipedia:%E6%B2%99%E7%9B%92, I see:

Source: =测试区=
Section title: 测试区
Edit link tooltip: 编辑段落:测试区

which seems entirely consistent.
Comment 5 Liangent 2011-12-06 00:20:49 UTC
(In reply to comment #4)
> There is no section edit link on old page views such as your sample URLs.
> 
> If I go to the current version of
> https://zh.wikipedia.org/wiki/Wikipedia:%E6%B2%99%E7%9B%92, I see:
> 
> Source: =测试区=
> Section title: 测试区
> Edit link tooltip: 编辑段落:测试区
> 
> which seems entirely consistent.

I forgot that old pages do not have edit links. Those revisions are specially designed. You can copy their source to a currect page (not restricted to zhwiki) and see the bug.
Comment 6 Jelle Zijlstra 2012-01-26 21:25:08 UTC
Created attachment 9914 [details]
Patch (didn't test whether this breaks other stuff)

See https://en.wikipedia.org/wiki/User:Ucucha/bug32802 for another example. The "title" attribute for the editsection link is "Edit section: foo ==Notice of discussion at the Administrators' Noticeboard == Hello. This message is being sent to inform you that there is currently a discussion at [[Wikipedia:Administrators' noticeboard/Incidents]] regarding an issue with which you may have been involved.    Thank you. bar", which expands the ANI-notice template.

I've attached a simple patch that fixes this issue on my local installation of MW trunk. I haven't been able to run parser tests, so I'm not sure whether this may break other stuff.
Comment 7 Brad Jorsch 2012-01-26 21:35:15 UTC
Created attachment 9915 [details]
Patch to fix the issue

(In reply to comment #6)
> 
> I've attached a simple patch that fixes this issue on my local installation of
> MW trunk. I haven't been able to run parser tests, so I'm not sure whether this
> may break other stuff.

It causes a regression of bug 25462. I've attached a patch that seems to work correctly.
Comment 8 Sumana Harihareswara 2012-02-11 01:53:02 UTC
Brad, sorry for the wait; there's been a bit of a delay in the review of patches here.  As we prepare to get a new version out, we're in a "code slush" during which we concentrate on reviewing code that has already been committed to our source code repository (see
http://thread.gmane.org/gmane.science.linguistics.wikipedia.technical/57950 for
more details).

Thanks for your patch, and I hope to get it reviewed soon.
Comment 9 Santhosh Thottingal 2012-03-22 06:46:34 UTC
I see that this bug is addressed in r112395 and related to Bug 34528. I am not able to reproduce it the issue mentioned. I am closing it now.
Comment 10 Liangent 2012-03-22 06:51:12 UTC
(In reply to comment #9)
> I see that this bug is addressed in r112395 and related to Bug 34528. I am not
> able to reproduce it the issue mentioned. I am closing it now.

Yeah I think this is actually fixed in that commit.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links