wfProfile fixes:
0543c7b0 added a wfProfileIn() that should have been wfProfileOut()
Reorder Http::request() so that wfProfileOut() is at the same
level as the above wfProfileIn() [checked by check-vars]
Change-Id: Ia91445d792e524b9c570890397f1513af0d8cd73
This commit is contained in:
parent
0543c7b039
commit
cebf04f334
1 changed files with 4 additions and 6 deletions
|
|
@ -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__ );
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
|||
Loading…
Reference in a new issue