User "Nikerabbit" posted a comment on MediaWiki.r96712.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/96712#c22305
Commit summary:

Conversion script between Tifinagh and Latin for the Tachelhit language
Preferences: made getLanguageNames include those that don't have a Messages file

Comment:

<pre>Preferences: made getLanguageNames include those that don't have a 
Messages file</pre>
Why and why embedded in a big commit?

<pre>
$wgHooks['ArticleSaveComplete'][] = $this->mConverter;
</pre>
Having that in class constructor looks very suspicious.

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to