jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/398225 )
Change subject: Compatible with php7.2, avoid passing null to count()
......................................................................
Compatible with php7.2, avoid passing null to count()
Bug: T182847
Change-Id: I0ab4555fc0669605528d5a14bbaa771cc88e0708
---
M BlueSkyTemplate.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
jenkins-bot: Verified
Thiemo Kreuz (WMDE): Looks good to me, approved
diff --git a/BlueSkyTemplate.php b/BlueSkyTemplate.php
index 4bdd4bd..e4fdaa9 100644
--- a/BlueSkyTemplate.php
+++ b/BlueSkyTemplate.php
@@ -32,7 +32,7 @@
$title = $skin->getTitle();
$request = $skin->getRequest();
$action = $request->getVal( 'action', 'view' );
- if ( count( $request->getVal( 'diff' ) ) > 0 ) {
+ if ( count( $request->getVal( 'diff',[] ) ) > 0 ) {
$action = 'diff';
}
// We'll treat the mainpage like any other page if they're
doing something besides looking at it
--
To view, visit https://gerrit.wikimedia.org/r/398225
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0ab4555fc0669605528d5a14bbaa771cc88e0708
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/skins/BlueSky
Gerrit-Branch: master
Gerrit-Owner: 星耀晨曦 <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: Thiemo Kreuz (WMDE) <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
