[MediaWiki-commits] [Gerrit] mediawiki...ORES[master]: Clean up === false comparisons

[jenkins-bot (Code Review)]

jenkins-bot has submitted this change and it was merged.

Change subject: Clean up === false comparisons
......................................................................


Clean up === false comparisons

Change-Id: Ib56fba798e6ebeba4879201890c038251089e32f
---
M includes/Hooks.php
1 file changed, 12 insertions(+), 15 deletions(-)

Approvals:
  Ladsgroup: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/Hooks.php b/includes/Hooks.php
index 7692089..8ffe2e2 100644
--- a/includes/Hooks.php
+++ b/includes/Hooks.php
@@ -83,7 +83,7 @@
         * @return bool
         */
        public static function onChangesListSpecialPageFilters( 
ChangesListSpecialPage $clsp, &$filters ) {
-               if ( self::oresEnabled( $clsp->getUser() ) === false ) {
+               if ( !self::oresEnabled( $clsp->getUser() ) ) {
                        return true;
                }
 
@@ -123,7 +123,7 @@
                array &$query_options, array &$join_conds, FormOptions $opts
        ) {
                global $wgUser;
-               if ( self::oresEnabled( $wgUser ) === false ) {
+               if ( !self::oresEnabled( $wgUser ) ) {
                        return true;
                }
 
@@ -179,7 +179,7 @@
        public static function onEnhancedChangesListModifyLineData( 
EnhancedChangesList $ecl, array &$data,
                array $block, RCCacheEntry $rcObj, array &$classes
        ) {
-               if ( self::oresEnabled( $ecl->getUser() ) === false ) {
+               if ( !self::oresEnabled( $ecl->getUser() ) ) {
                        return true;
                }
 
@@ -199,7 +199,7 @@
        public static function onEnhancedChangesListModifyBlockLineData( 
EnhancedChangesList $ecl,
                array &$data, RCCacheEntry $rcObj
        ) {
-               if ( self::oresEnabled( $ecl->getUser() ) === false ) {
+               if ( !self::oresEnabled( $ecl->getUser() ) ) {
                        return true;
                }
 
@@ -222,7 +222,7 @@
        public static function onOldChangesListRecentChangesLine( ChangesList 
&$changesList, &$s,
                $rc, &$classes = []
        ) {
-               if ( self::oresEnabled( $changesList->getUser() ) === false ) {
+               if ( !self::oresEnabled( $changesList->getUser() ) ) {
                        return true;
                }
 
@@ -248,7 +248,7 @@
         * @param array $query
         */
        public static function onContribsGetQueryInfo( ContribsPager $pager, 
&$query ) {
-               if ( self::oresEnabled( $pager->getUser() ) === false ) {
+               if ( !self::oresEnabled( $pager->getUser() ) ) {
                        return true;
                }
 
@@ -290,7 +290,7 @@
        public static function onSpecialContributionsFormatRowFlags(
                RequestContext $context, $row, array &$flags
        ) {
-               if ( self::oresEnabled( $context->getUser() ) === false ) {
+               if ( !self::oresEnabled( $context->getUser() ) ) {
                        return true;
                }
 
@@ -308,7 +308,7 @@
        public static function onContributionsLineEnding(
                ContribsPager $pager, &$ret, $row, array &$classes
        ) {
-               if ( self::oresEnabled( $pager->getUser() ) === false ) {
+               if ( !self::oresEnabled( $pager->getUser() ) ) {
                        return true;
                }
 
@@ -329,7 +329,7 @@
        public static function onSpecialContributionsGetFormFilters(
                SpecialContributions $page, array &$filters
        ) {
-               if ( self::oresEnabled( $page->getUser() ) === false ) {
+               if ( !self::oresEnabled( $page->getUser() ) ) {
                        return true;
                }
 
@@ -398,7 +398,7 @@
        public static function onGetPreferences( $user, &$preferences ) {
                global $wgOresDamagingThresholds;
 
-               if ( self::oresEnabled( $user ) === false ) {
+               if ( !self::oresEnabled( $user ) ) {
                        return true;
                }
                $options = [];
@@ -432,7 +432,7 @@
         * Add CSS styles to output page
         */
        public static function onBeforePageDisplay( OutputPage &$out, Skin 
&$skin ) {
-               if ( self::oresEnabled( $out->getUser() ) === false ) {
+               if ( !self::oresEnabled( $out->getUser() ) ) {
                        return true;
                }
                $out->addModuleStyles( 'ext.ores.styles' );
@@ -464,9 +464,6 @@
         * @return bool
         */
        private static function oresEnabled( User $user ) {
-               if ( BetaFeatures::isFeatureEnabled( $user, 'ores-enabled' ) ) {
-                       return true;
-               }
-               return false;
+               return BetaFeatures::isFeatureEnabled( $user, 'ores-enabled' );
        }
 }

-- 
To view, visit https://gerrit.wikimedia.org/r/311651
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib56fba798e6ebeba4879201890c038251089e32f
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/ORES
Gerrit-Branch: master
Gerrit-Owner: Catrope <r...@wikimedia.org>
Gerrit-Reviewer: Ladsgroup <ladsgr...@gmail.com>
Gerrit-Reviewer: Thiemo Mättig (WMDE) <thiemo.maet...@wikimedia.de>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits