"Krinkle" posted a comment on MediaWiki.r105239. URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/105239#c26996
Commit summary for MediaWiki.r105239: [HarvardResearch][CentralNotice] Temporary wgNoticeBanner_Harvard2011 data. -- Banner display is handled through JavaScript because it's CentralNotice that is also for anonymous users and page output must be unaffected. -- Banner submission to Harvard/Science PO needs a hash of which the salt must not be in cleartext in the front-end code (also generating hashes in JS is ugly). So we need at least one temporary server component. -- Instead of requesting things from 2 or three different APIs on all pages + outputting a hash from PHP, might as well do it all in PHP and output an object in mw.config with all the needed data -- Cached in the session. Not sure whether it should be in memcached or session. Choose session for now since this is going to happen for all logged-in users on en.wiki, not sure wether it's good to have a gazillion memcached entries only used by 1 user only needed for about a week. -- Also notices that session is somewhat tied into memcached Krinkle's comment: Agreed. Taken from [http://svn.wikimedia.org/viewvc/mediawiki/trunk/extensions/UserDailyContribs/api/ApiUserDailyContribs.php?revision=104622&view=markup#l34 ApiUserDailyContribs.php] (originally like [//svn.wikimedia.org/viewvc/mediawiki/trunk/extensions/UserDailyContribs/api/ApiUserDailyContribs.php?revision=84932&view=markup#l21 this], [//svn.wikimedia.org/viewvc/mediawiki/trunk/extensions/UserDailyContribs/api/ApiUserDailyContribs.php?revision=91244&view=markup#l21 broken] and [//svn.wikimedia.org/viewvc/mediawiki/trunk/extensions/UserDailyContribs/api/ApiUserDailyContribs.php?revision=99005&view=markup#l22 fixed]). Fixed in r105267. _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
