Page MenuHomePhabricator

D16571.diff
No OneTemporary

D16571.diff

diff --git a/src/applications/base/controller/PhabricatorRedirectController.php b/src/applications/base/controller/PhabricatorRedirectController.php
--- a/src/applications/base/controller/PhabricatorRedirectController.php
+++ b/src/applications/base/controller/PhabricatorRedirectController.php
@@ -2,9 +2,6 @@
final class PhabricatorRedirectController extends PhabricatorController {
- private $uri;
- private $allowExternal;
-
public function shouldRequireLogin() {
return false;
}
@@ -13,15 +10,12 @@
return false;
}
- public function willProcessRequest(array $data) {
- $this->uri = $data['uri'];
- $this->allowExternal = idx($data, 'external', false);
- }
-
- public function processRequest() {
+ public function handleRequest(AphrontRequest $request) {
+ $uri = $request->getURIData('uri');
+ $external = $request->getURIData('external', false);
return id(new AphrontRedirectResponse())
- ->setURI($this->uri)
- ->setIsExternal($this->allowExternal);
+ ->setURI($uri)
+ ->setIsExternal($external);
}
}

File Metadata

Mime Type
text/plain
Expires
Tue, Mar 25, 7:58 PM (4 w, 1 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7719799
Default Alt Text
D16571.diff (1 KB)

Event Timeline