"Kaldari" posted a comment on MediaWiki.r109786. URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/109786#c30082
Commit summary for MediaWiki.r109786: [ApiSandbox] bug fix * Looks like API is returning HTML here, got a couple of <br> and <li>'s * Wrapping parens only if there is a link, got a couple of loose () before Kaldari's comment: descHtml.replace( /^([^\r\n\.]*)/, '$1' ) I give up. What does this actually do? It looks like it's matching to any set of characters that don't include '\r', '\n', or '.', and then replacing that set of characters with itself. Does that ever actually change the output? _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
