User "Raymond" posted a comment on MediaWiki.r91854.

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

SocialProfile: add a special page, Special:GenerateTopUsersReport, to generate 
the report of the users who earned the most points during the past week or 
month.
This is the only way to update the points_winner_weekly and 
points_winner_monthly columns in the user_stats table. This requires that the 
user_points_archive, user_points_monthly and user_points_weekly tables exist in 
the database; SocialProfile does not yet ship with the schemas for those 
tables, but I'll try to fix that soon-ish.

To access Special:GenerateTopUsersReport, you need the generatetopusersreport 
user right, which isn't given to any user group by default.

Code from 
http://trac.wikia-code.com/browser/wikia/trunk-1.15/extensions/wikia/UserStats/SpecialGenerateTopUserReport.php
 with tweaks and cleanup by me

Comment:

* 'user-stats-weekly-win-congratulations' and others needs PLURAL support for 
"$2 extra points"
** can't think of why, at least when it comes to English and Finnish, although 
some funny sysadmin might set it so that a weekly win earns you one point... 
*** Keep in mind that some languages, like Russian, has 4-5 plural forms, 
partly very complex.

* Numbers should be run through $wgLang->formatNum() or $wgContLang->formatNum()
** Probably the former, can't think of a good reason why they should be in 
content language.
*** Because you call "content": <code>wfMsgExt( 
"user-stats-{$period}-win-congratulations", array( 'parsemag', 'content' ), 
...</code>

* Add a special page alias file
** No need to add a separate file, this can be done by adding 
GenerateTopUsersReport to SocialProfile/SocialProfile.alias.php
*** Ah, I missed this. 

_______________________________________________
MediaWiki-CodeReview mailing list
mediawiki-coderev...@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to