Improve markpatrolled hook

diff --git a/docs/hooks.txt b/docs/hooks.txt
index c9daed9..41f5587 100644 (file)
--- a/docs/hooks.txt
+++ b/docs/hooks.txt
@@ -897,7 +897,7 @@ $variableIDs: array of strings
                of Skin::makeVariablesScript
 
 'MarkPatrolled': before an edit is marked patrolled
-$rcid: ID of the revision to be marked patrolled
+$rc: RecentChange object of the revision to be marked patrolled
 $user: the user (object) marking the revision as patrolled
 $wcOnlySysopsCanPatrol: config setting indicating whether the user
  needs to be a sysop in order to mark an edit patrolled

diff --git a/includes/RecentChange.php b/includes/RecentChange.php
index 010770c..ff4624f 100644 (file)
--- a/includes/RecentChange.php
+++ b/includes/RecentChange.php
@@ -268,7 +268,7 @@ class RecentChange
                // Automatic patrol needs "autopatrol", ordinary patrol needs "patrol"
                $right = $auto ? 'autopatrol' : 'patrol';
                $errors = array_merge( $errors, $this->getTitle()->getUserPermissionsErrors( $right, $wgUser ) );
-               if( !wfRunHooks('MarkPatrolled', array($this->getAttribute('rc_id'), &$wgUser, false)) ) {
+               if( !wfRunHooks('MarkPatrolled', array($this, &$wgUser, false)) ) {
                        $errors[] = array('hookaborted');
                }
                // Users without the 'autopatrol' right can't patrol their