User "IAlex" posted a comment on MediaWiki.r101037. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/101037#c25077 Commit summary:
Per Nikerabbit's comment on r100621: * Removed OutputPage::setPageTitleMsg() and OutputPage::setHTMLTitleMsg() and make OutputPage::setPageTitle() and OutputPage::setHTMLTitle() accept a Message object * Updated core calls (including some that I missed last time because of non-matching case) * Added Message::setContext() and use it in RequestContext so that I don't need to duplicate the call in OutputPage * Yes, I'm calling $this->msg() on places and then setting the context one more time in OutputPage::setPageTitle() or OutputPage::setHTMLTitle(), but at least I won't be confused about which objects $ Comment: The HTML title is escaped in OutputPage::headElement() and OutputPage::setPageTitle() does its own escaping as it allows only whitelisted tags (more or the same as in wikitext); so I'm considering these two methods safe. _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
