Last modified: 2011-09-21 20:52:19 UTC
Don't set up language converter hooks in Language object constructor. Looks very ugly and error prone. Raised in r96712.
The most puzzling thing is that it seems to be there to trigger a generic-looking onArticleSaveComplete method on LanguageConverter, to recache some objects if you're saving to MediaWiki:Conversiontable/<variant-name> But since it triggers for any Language object that's created where the language supports a converter, this may trigger inappropriately -- eg based on somebody's UI language where the converter isn't even available to use. Seems funky in the extreme to me. :)
Note this dates back at least to r8347; and before that it looks like it was done as a standalone hook function that'd get set up as soon as LanguageZh.php got loaded -- without even necessarily instantiating the class. :)