Siebrand Mazeland wrote:
https://bugzilla.wikimedia.org/show_bug.cgi?id=17865
-- Mismatched input
syntax for Cite error messages
* Some discussion. Cite is seen as scary! No one to take this yet.
Please give this some TLC.
Cite is not /that/ scary. What's TLC?
I don't see the problem with those messages, though. They look like the
rest of Cite errors. Just through the parser instead of the wfMsg()
calls (which would have its own problems). Seems that ParserBeforeTidy
is too late for doing a recursiveTagParse().
Just replacing the call to Cite::parse() of Cite::error with a call to
MessageCache::singleton()->parse() looks like the best solution.
We don't need to keep the parser context for errors.
And now MessageCache::parse() doesn't pollute the $wgParser state.
== Harder issues and discussion ==
https://bugzilla.wikimedia.org/show_bug.cgi?id=28428 -- Allow saving pages
with LRM and RLM in titles, showing a warning and requiring a user right
https://bugzilla.wikimedia.org/show_bug.cgi?id=28411 -- titles of articles
with LTR titles in RTL wikis may be displayed incorrectly in categories
and special pages
* It looks like a schema change is not needed, and that the
page_props table can and should be used. Use batches where needed;
integrate that in Linker, LinkBatch and probably Title as well.
Ambitious according to Roan. Roan agrees that having displaytitle as a
page_ field makes more sense *conceptually*. In practice, he prefers
page_props because that avoids a schema change and is still cheap.
Niklas indicated the discussion gave him ideas on how to work towards
resolving these issues.
I agree with the idea of storing the displaytitle in page_props.