Merge "Don't use isset() to check whether an existing variable is null"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Tue, 18 Mar 2014 11:39:20 +0000 (11:39 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Tue, 18 Mar 2014 11:39:20 +0000 (11:39 +0000)
includes/UserMailer.php

index 4c2ecba..de4457d 100644 (file)
@@ -275,7 +275,7 @@ class UserMailer {
                                $headers = $mime->headers( $headers );
                        }
                }
-               if ( !isset( $mime ) ) {
+               if ( $mime === null ) {
                        // sending text only, either deliberately or as a fallback
                        if ( wfIsWindows() ) {
                                $body = str_replace( "\n", "\r\n", $body );