revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-22 06:42:41 +01:00
Code Issues Releases Wiki Activity

Fix a missing "pht()" parameter in HTTPSFuture

Browse source 
Summary: Ref T13577. This call is missing a parameter. After D21453, this is detected properly by lint. Provide the parameter.

Test Plan: Ran `arc lint` on HTTPSFuture before and after the change.

Maniphest Tasks: T13577

Differential Revision: https://secure.phabricator.com/D21454
This commit is contained in:
epriestley 2020-09-04 17:01:21 -07:00
parent 73847a4b19
commit 1c327208d7
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/future/http/HTTPSFuture.php
View file

@ -132,7 +132,7 @@ final class HTTPSFuture extends BaseHTTPFuture {
* @return string|false * @return string|false
*/ */
public static function loadContent($uri, $timeout = null) { public static function loadContent($uri, $timeout = null) {
$future = new HTTPSFuture($uri); $future = new self($uri);
if ($timeout !== null) { if ($timeout !== null) {
$future->setTimeout($timeout); $future->setTimeout($timeout);
} }
@ -151,7 +151,8 @@ final class HTTPSFuture extends BaseHTTPFuture {
throw new Exception( throw new Exception(
pht( pht(
'Specified download path "%s" already exists, refusing to '. 'Specified download path "%s" already exists, refusing to '.
'overwrite.')); 'overwrite.',
$download_path));
} }
return $this; return $this;