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.