T152540 migrated section IDs, but citation ids (from the Cite extension) are still using the old .7B.7B style encoding.
This makes the address bar very hard to read for wikis which don't use the latin script for writing.
Description
Description
Details
Details
Related Changes in Gerrit:
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Use HTML5 id attributes; remove use of deprecated Sanitizer::escapeId() | mediawiki/extensions/Cite | master | +72 -24 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | MaxSem | T75092 Anchors to section names for non-ASCII letters are encoded in the URL | |||
| Resolved | matmarex | T172493 mw.util.escapeId should log deprecation warning | |||
| Resolved | MaxSem | T152540 Migrate to HTML5 section ids | |||
| Resolved | cscott | T176170 Migrate citation IDs to HTML5 encoding |
Event Timeline
Comment Actions
Change 378774 had a related patch set uploaded (by C. Scott Ananian; owner: C. Scott Ananian):
[mediawiki/extensions/Cite@master] Use HTML5 id attributes; remove use of deprecated Sanitizer::escapeId()
Comment Actions
Current patch mostly fixes this but if you set $wgFragmentMode = ['html5']....
$ echo '<sup id="cite_ref-{|a}}_1-0" class="reference">[[#cite_note-{{echo|a}}-1|[1]]]</sup>' | php maintenance/parse.php <div class="mw-parser-output"><p><sup id="cite_ref-{{echo|a}}_1-0" class="reference"><a href="#cite_notea}1">[1]</a></sup> </p></div>
Look at the href attribute: something in the core parser is double-expanding the attribute value, but it is usually protected (when wgFragmentMode = legacy) by the URL escaping.
So that needs to be fixed in the core parser (or else I need to come up with a better way of protecting the wikilink): {T176176: HTML5 ids seems to change how wikilink fragments are parsed (when LanguageConverter is enabled)}
Comment Actions
Change 378774 merged by jenkins-bot:
[mediawiki/extensions/Cite@master] Use HTML5 id attributes; remove use of deprecated Sanitizer::escapeId()
Comment Actions
Appears to be, yes. In current master:
$ echo '<sup id="cite_ref-{|a}}_1-0" class="reference">[[#cite_note-{{echo|a}}-1|[1]]]</sup>' | php maintenance/parse.php <div class="mw-parser-output"><p><sup id="cite_ref-{|a}}_1-0" class="reference"><a href="#cite_note-{{echo|a}}-1">[1]</a></sup> </p></div>
The href attribute is now correct.