User "Reedy" posted a comment on MediaWiki.r91223. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/91223#c19190 Commit summary:
Move parameter stuff around so we can do the multi cases... Comment: "Intended for deployment" is unclear at the moment. There is possibility this will be enabled side wide. In the near future (next few months), it's only supposed to be for dev/WMF internal stats use. Hence "deferred" is appropriate as there is no reason for anyone to spend (waste?) any time reviewing this code at the moment, and if/when it needs full review for deployment, CR can be left on the later revision, or an appropriate bug. Also, quite a bit of code will be turned over, so reviewing old code is pointless. Again, for the codes, not my decision, I'm working against the given spec - [[Manual:Wikistats/API]], and this project isn't supposed to be much of my time in total. I can bring it up, but I'm not sure whether these are just how they are scraped from the Squid logs etc, but granted, the conversion *could* be done at the import/update part, rather than in place by the api (though, it could be, also). _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
