mirror of
https://we.phorge.it/source/phorge.git
synced 2024-11-19 05:12:41 +01:00
Removing deprecated method calls
Summary: Removed call to the deprecated buildStandardPageResponse method from XHProfProfileController Test Plan: Install, configure, and use XHProf. I'll need some guidance with this Reviewers: epriestley, #blessed_reviewers Reviewed By: epriestley, #blessed_reviewers Subscribers: Korvin, epriestley Differential Revision: https://secure.phabricator.com/D16432
This commit is contained in:
parent
5d93290e42
commit
d26cca27d7
3 changed files with 11 additions and 29 deletions
|
@ -2,26 +2,4 @@
|
|||
|
||||
abstract class PhabricatorXHProfController extends PhabricatorController {
|
||||
|
||||
public function buildStandardPageResponse($view, array $data) {
|
||||
$page = $this->buildStandardPageView();
|
||||
|
||||
$page->setApplicationName(pht('XHProf'));
|
||||
$page->setBaseURI('/xhprof/');
|
||||
$page->setTitle(idx($data, 'title'));
|
||||
$page->setGlyph("\xE2\x98\x84");
|
||||
$page->appendChild($view);
|
||||
$page->setDeviceReady(true);
|
||||
|
||||
$response = new AphrontWebpageResponse();
|
||||
|
||||
if (isset($data['frame'])) {
|
||||
$response->setFrameable(true);
|
||||
$page->setFrameable(true);
|
||||
$page->setShowChrome(false);
|
||||
$page->setDisableConsole(true);
|
||||
}
|
||||
|
||||
return $response->setContent($page->render());
|
||||
}
|
||||
|
||||
}
|
||||
|
|
|
@ -47,11 +47,14 @@ final class PhabricatorXHProfProfileController
|
|||
$crumbs = $this->buildApplicationCrumbs();
|
||||
$crumbs->addTextCrumb(pht('%s Profile', $symbol));
|
||||
|
||||
return $this->buildStandardPageResponse(
|
||||
array($crumbs, $view),
|
||||
array(
|
||||
'title' => pht('Profile'),
|
||||
'frame' => $is_framed,
|
||||
));
|
||||
$title = pht('Profile');
|
||||
|
||||
return $this->newPage()
|
||||
->setTitle($title)
|
||||
->setCrumbs($crumbs)
|
||||
->setFrameable(true)
|
||||
->setShowChrome(false)
|
||||
->setDisableConsole(true)
|
||||
->appendChild($view);
|
||||
}
|
||||
}
|
||||
|
|
|
@ -890,7 +890,8 @@ final class PhabricatorStandardPageView extends PhabricatorBarePageView
|
|||
} else {
|
||||
$content = $this->render();
|
||||
$response = id(new AphrontWebpageResponse())
|
||||
->setContent($content);
|
||||
->setContent($content)
|
||||
->setFrameable($this->getFrameable());
|
||||
}
|
||||
|
||||
return $response;
|
||||
|
|
Loading…
Reference in a new issue