(bug 14778) 'limit' parameter now applies to history feeds as well as history pages

diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index b95022e..9e60525 100644 (file)
--- a/RELEASE-NOTES
+++ b/RELEASE-NOTES
@@ -432,6 +432,8 @@ it from source control: http://www.mediawiki.org/wiki/Download_from_SVN
 * (bug 14763) Child classes of Database (DatabasePostgres and DatabaseOracle)
   had stict standards issues with setFakeSlaveLag() and setFakeMaster().
 * (bug 451) Improve the phrase mappings of the Chinese converter arrays.
+* (bug 14778) 'limit' parameter now applies to history feeds as well as 
+  history pages
 
 === API changes in 1.13 ===
 

diff --git a/includes/PageHistory.php b/includes/PageHistory.php
index 0819441..7cd8891 100644 (file)
--- a/includes/PageHistory.php
+++ b/includes/PageHistory.php
@@ -501,7 +501,7 @@ class PageHistory {
         * @param string $type
         */
        function feed( $type ) {
-               global $wgFeedClasses;
+               global $wgFeedClasses, $wgRequest, $wgFeedLimit;
                if ( !FeedUtils::checkFeedOutput($type) ) {
                        return;
                }
@@ -512,7 +512,14 @@ class PageHistory {
                        wfMsgForContent( 'history-feed-description' ),
                        $this->mTitle->getFullUrl( 'action=history' ) );
 
-               $items = $this->fetchRevisions(10, 0, PageHistory::DIR_NEXT);
+               // Get a limit on number of feed entries. Provide a sane default
+               // of 10 if none is defined (but limit to $wgFeedLimit max)
+               $limit = $wgRequest->getInt( 'limit', 10 );
+               if( $limit > $wgFeedLimit ) {
+                       $limit = $wgFeedLimit;
+               }
+               $items = $this->fetchRevisions($limit, 0, PageHistory::DIR_NEXT);
+
                $feed->outHeader();
                if( $items ) {
                        foreach( $items as $row ) {