Merge "wfProfile fixes: 0543c7b0 added a wfProfileIn() that should have been wfProfi...
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Tue, 19 Mar 2013 17:01:59 +0000 (17:01 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Tue, 19 Mar 2013 17:01:59 +0000 (17:01 +0000)
includes/HttpFunctions.php

index 98d36d3..9e280db 100644 (file)
@@ -69,14 +69,12 @@ class Http {
                $req = MWHttpRequest::factory( $url, $options );
                $status = $req->execute();
 
+               $content = false;
                if ( $status->isOK() ) {
                        $content = $req->getContent();
-                       wfProfileOut( __METHOD__ . "-$method" );
-                       return $content;
-               } else {
-                       wfProfileOut( __METHOD__ . "-$method" );
-                       return false;
                }
+               wfProfileOut( __METHOD__ . "-$method" );
+               return $content;
        }
 
        /**
@@ -629,7 +627,7 @@ class MWHttpRequest {
                        }
                }
 
-               wfProfileIn( __METHOD__ );
+               wfProfileOut( __METHOD__ );
        }
 
        /**