Call doMarkPatrolled() on the RecentChange object instead of calling RecentChange...

diff --git a/includes/api/ApiPatrol.php b/includes/api/ApiPatrol.php
index 04afd1d..58e0499 100644 (file)
--- a/includes/api/ApiPatrol.php
+++ b/includes/api/ApiPatrol.php
@@ -42,13 +42,15 @@ class ApiPatrol extends ApiBase {
         * Patrols the article or provides the reason the patrol failed.
         */
        public function execute() {
+               global $wgUser;
+
                $params = $this->extractRequestParams();
 
                $rc = RecentChange::newFromID( $params['rcid'] );
                if ( !$rc instanceof RecentChange ) {
                        $this->dieUsageMsg( array( 'nosuchrcid', $params['rcid'] ) );
                }
-               $retval = RecentChange::markPatrolled( $params['rcid'] );
+               $retval = $rc->doMarkPatrolled( $wgUser );
 
                if ( $retval ) {
                        $this->dieUsageMsg( reset( $retval ) );