Fix inappropriate use of empty(), it should only be used when you want to suppress...

diff --git a/ConfirmEdit_body.php b/ConfirmEdit_body.php
index 57655c681023ea56f4d4290919666a621bee7a3b..2d2124623a46a17146954c266d39476bd9d00098 100644 (file)
--- a/ConfirmEdit_body.php
+++ b/ConfirmEdit_body.php
@@ -218,7 +218,7 @@ class SimpleCaptcha {
         */
        function isIPWhitelisted() {
                global $wgCaptchaWhitelistIP;
-               if( !empty( $wgCaptchaWhitelistIP ) ) {
+               if( $wgCaptchaWhitelistIP ) {
                        $ip = wfGetIp();
                        foreach ( $wgCaptchaWhitelistIP as $range ) {
                                if ( IP::isInRange( $ip, $range ) ) {
@@ -349,7 +349,7 @@ class SimpleCaptcha {
                }
 
                global $wgCaptchaRegexes;
-               if( !empty( $wgCaptchaRegexes ) ) {
+               if( $wgCaptchaRegexes ) {
                        // Custom regex checks
                        $oldtext = $this->loadText( $editPage, $section );