http://www.mediawiki.org/wiki/Special:Code/MediaWiki/71515

Revision: 71515
Author:   platonides
Date:     2010-08-23 21:54:45 +0000 (Mon, 23 Aug 2010)

Log Message:
-----------
Try to improve the funciton comment of getETag() per r70809 CR. Adaptation of 
Nikerabbit proposal.

Modified Paths:
--------------
    trunk/phase3/includes/parser/ParserCache.php

Modified: trunk/phase3/includes/parser/ParserCache.php
===================================================================
--- trunk/phase3/includes/parser/ParserCache.php        2010-08-23 21:52:15 UTC 
(rev 71514)
+++ trunk/phase3/includes/parser/ParserCache.php        2010-08-23 21:54:45 UTC 
(rev 71515)
@@ -55,10 +55,14 @@
        }
 
        /**
-        * Provides an E-Tag suitable for the whole page, even if $article is
-        * just the main wikitext. So it uses the complete set of user options.
-        * Most importantly, that includes the user language, but other options
-        * would give problems on some setups, too.
+        * Provides an E-Tag suitable for the whole page. Note that $article 
+        * is just the main wikitext. The E-Tag has to be unique to the whole 
+        * page, even if the article itself is the same, so it uses the 
+        * complete set of user options. We don't want to use the preference 
+        * of a different user on a message just because it wasn't used in 
+        * $article. For example give a Chinese interface to a user with 
+        * English preferences. That's why we take into account *all* user 
+        * options. (r70809 CR)
         */
        function getETag( $article, $popts ) {
                return 'W/"' . $this->getParserOutputKey( $article, 



_______________________________________________
MediaWiki-CVS mailing list
MediaWiki-CVS@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs

Reply via email to