Bsitu has uploaded a new change for review.
https://gerrit.wikimedia.org/r/66142
Change subject: Add new parameter $patrolFooterShown to hook ArticleViewFooter
......................................................................
Add new parameter $patrolFooterShown to hook ArticleViewFooter
Some extensions like PageTriage will be able to use this new
parameter to determine whether to show its own curation toolbar.
Change-Id: I07e85aa37edc994c5a2bf9d011976b91ff01ab14
---
M RELEASE-NOTES-1.22
M docs/hooks.txt
M includes/Article.php
3 files changed, 15 insertions(+), 8 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/42/66142/1
diff --git a/RELEASE-NOTES-1.22 b/RELEASE-NOTES-1.22
index 4e04940..dd4dbbb 100644
--- a/RELEASE-NOTES-1.22
+++ b/RELEASE-NOTES-1.22
@@ -86,6 +86,8 @@
HTTP caches when a page is changed.
* Changed the patrolling system to always show the link for patrolling in case
the
current revision is patrollable. This also removed the usage of the rcid URI
parameters.
+* Add a new parameter $patrolFooterShown to hook ArticleViewFooter so the hook
+ handlers can take further action based on the status of the patrol footer
=== Bug fixes in 1.22 ===
* Disable Special:PasswordReset when $wgEnableEmail is false. Previously one
diff --git a/docs/hooks.txt b/docs/hooks.txt
index 640d642..3c77985 100644
--- a/docs/hooks.txt
+++ b/docs/hooks.txt
@@ -615,6 +615,7 @@
'ArticleViewFooter': After showing the footer section of an ordinary page view
$article: Article object
+$patrolFooterShown: boolean whether patrol footer is shown
'ArticleViewHeader': Before the parser cache is about to be tried for article
viewing.
diff --git a/includes/Article.php b/includes/Article.php
index c4b0835..cd9a995 100644
--- a/includes/Article.php
+++ b/includes/Article.php
@@ -1038,9 +1038,9 @@
}
// Show a footer allowing the user to patrol the shown revision
or page if possible
- $this->showPatrolFooter();
+ $patrolFooterShown = $this->showPatrolFooter();
- wfRunHooks( 'ArticleViewFooter', array( $this ) );
+ wfRunHooks( 'ArticleViewFooter', array( $this,
$patrolFooterShown ) );
}
@@ -1050,6 +1050,8 @@
* desired, does nothing.
* Side effect: When the patrol link is build, this method will call
* OutputPage::preventClickjacking() and load
mediawiki.page.patrol.ajax.
+ *
+ * @return bool
*/
public function showPatrolFooter() {
global $wgUseRCPatrol, $wgUseNPPatrol, $wgRCMaxAge;
@@ -1066,7 +1068,7 @@
if ( !$this->getTitle()->quickUserCan( 'patrol', $user ) || (
!$wgUseNPPatrol && !$wgUseRCPatrol ) ) {
// Patrolling is fully disabled or the user isn't
allowed to
- return;
+ return false;
}
wfProfileIn( __METHOD__ );
@@ -1075,7 +1077,7 @@
// Check for cached results
if ( $cache->get( wfMemcKey( 'NotPatrollableRevId',
$this->getRevIdFetched() ) ) ) {
wfProfileOut( __METHOD__ );
- return;
+ return false;
}
// We make use of a little index trick over here:
@@ -1091,7 +1093,7 @@
// The revision is more than 6 hours older than
the Max RC age
// no need to torture the DB any further (6h
because the RC might not be cleaned out regularly)
wfProfileOut( __METHOD__ );
- return;
+ return false;
}
$rc = RecentChange::newFromConds(
array(
@@ -1115,7 +1117,7 @@
// Check for cached results
if ( $cache->get( wfMemcKey( 'NotPatrollablePage',
$this->getTitle()->getArticleID() ) ) ) {
wfProfileOut( __METHOD__ );
- return;
+ return false;
}
$dbr = wfGetDB( DB_SLAVE );
@@ -1130,7 +1132,7 @@
// We either didn't find the oldest revision
for the given page
// or it's to old for the RC table (with 6h
tolerance)
wfProfileOut( __METHOD__ );
- return;
+ return false;
}
$rc = RecentChange::newFromConds(
@@ -1159,7 +1161,7 @@
$cache->set( wfMemcKey( 'NotPatrollablePage',
$this->getTitle()->getArticleID() ), '1', $wgRCMaxAge );
}
- return;
+ return false;
}
$rcid = $rc->getAttribute( 'rc_id' );
@@ -1185,6 +1187,8 @@
wfMessage( 'markaspatrolledlink' )->rawParams(
$link )->escaped() .
'</div>'
);
+
+ return true;
}
/**
--
To view, visit https://gerrit.wikimedia.org/r/66142
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I07e85aa37edc994c5a2bf9d011976b91ff01ab14
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: wmf/1.22wmf5
Gerrit-Owner: Bsitu <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
