2011-01-16 13:51:39 -08:00
|
|
|
<?php
|
|
|
|
|
2012-10-16 10:33:47 -07:00
|
|
|
/**
|
|
|
|
* This is a standard Phabricator page with menus, Javelin, DarkConsole, and
|
|
|
|
* basic styles.
|
|
|
|
*
|
|
|
|
*/
|
|
|
|
final class PhabricatorStandardPageView extends PhabricatorBarePageView {
|
2011-01-16 13:51:39 -08:00
|
|
|
|
|
|
|
private $baseURI;
|
|
|
|
private $applicationName;
|
|
|
|
private $glyph;
|
2012-07-31 06:18:38 -07:00
|
|
|
private $menuContent;
|
Countdown tweaks
Summary:
A few tweaks to hsb's Countdown implementation:
- Allow the page to be rendered "chromeless", suitable for display on one of
the dozens of monitors everyone has laying around.
- Show title of countdown in deletion dialog.
- When creating a new countdown default to time(), not Dec 31, 1969.
- Add extra "/" after editing to avoid needless redirect.
- Tweak some page titles.
- Show countdown author in list view.
- Highlight tab in list view.
- Tweak menu copy.
- Link countdown title in list view, separate buttons into different columns
so they pick up padding.
Test Plan:
Created, edited and deleted a timer. Viewed a timer and toggled chrome mode.
Viewed timer list.
Reviewed By: hsb
Reviewers: hsb, aran, jungejason, tuomaspelkonen
CC: aran, hsb, epriestley
Differential Revision: 454
2011-06-13 17:35:13 -07:00
|
|
|
private $showChrome = true;
|
2013-03-04 10:35:50 -08:00
|
|
|
private $layDownSomeDust = false;
|
2011-07-09 09:45:19 -07:00
|
|
|
private $disableConsole;
|
2012-02-14 17:00:12 -08:00
|
|
|
private $searchDefaultScope;
|
2012-06-13 17:28:21 -07:00
|
|
|
private $pageObjects = array();
|
2012-12-07 13:34:44 -08:00
|
|
|
private $applicationMenu;
|
|
|
|
|
|
|
|
public function setApplicationMenu(PhabricatorMenuView $application_menu) {
|
|
|
|
$this->applicationMenu = $application_menu;
|
|
|
|
return $this;
|
|
|
|
}
|
|
|
|
|
|
|
|
public function getApplicationMenu() {
|
|
|
|
return $this->applicationMenu;
|
|
|
|
}
|
2011-01-16 13:51:39 -08:00
|
|
|
|
|
|
|
public function setApplicationName($application_name) {
|
|
|
|
$this->applicationName = $application_name;
|
|
|
|
return $this;
|
|
|
|
}
|
|
|
|
|
2011-07-09 09:45:19 -07:00
|
|
|
public function setDisableConsole($disable) {
|
|
|
|
$this->disableConsole = $disable;
|
|
|
|
return $this;
|
|
|
|
}
|
|
|
|
|
2011-01-16 13:51:39 -08:00
|
|
|
public function getApplicationName() {
|
|
|
|
return $this->applicationName;
|
|
|
|
}
|
|
|
|
|
|
|
|
public function setBaseURI($base_uri) {
|
|
|
|
$this->baseURI = $base_uri;
|
|
|
|
return $this;
|
|
|
|
}
|
|
|
|
|
|
|
|
public function getBaseURI() {
|
|
|
|
return $this->baseURI;
|
|
|
|
}
|
|
|
|
|
Countdown tweaks
Summary:
A few tweaks to hsb's Countdown implementation:
- Allow the page to be rendered "chromeless", suitable for display on one of
the dozens of monitors everyone has laying around.
- Show title of countdown in deletion dialog.
- When creating a new countdown default to time(), not Dec 31, 1969.
- Add extra "/" after editing to avoid needless redirect.
- Tweak some page titles.
- Show countdown author in list view.
- Highlight tab in list view.
- Tweak menu copy.
- Link countdown title in list view, separate buttons into different columns
so they pick up padding.
Test Plan:
Created, edited and deleted a timer. Viewed a timer and toggled chrome mode.
Viewed timer list.
Reviewed By: hsb
Reviewers: hsb, aran, jungejason, tuomaspelkonen
CC: aran, hsb, epriestley
Differential Revision: 454
2011-06-13 17:35:13 -07:00
|
|
|
public function setShowChrome($show_chrome) {
|
|
|
|
$this->showChrome = $show_chrome;
|
|
|
|
return $this;
|
|
|
|
}
|
|
|
|
|
2013-03-04 10:35:50 -08:00
|
|
|
public function setDust($is_dusty) {
|
|
|
|
$this->layDownSomeDust = $is_dusty;
|
|
|
|
return $this;
|
|
|
|
}
|
|
|
|
|
Countdown tweaks
Summary:
A few tweaks to hsb's Countdown implementation:
- Allow the page to be rendered "chromeless", suitable for display on one of
the dozens of monitors everyone has laying around.
- Show title of countdown in deletion dialog.
- When creating a new countdown default to time(), not Dec 31, 1969.
- Add extra "/" after editing to avoid needless redirect.
- Tweak some page titles.
- Show countdown author in list view.
- Highlight tab in list view.
- Tweak menu copy.
- Link countdown title in list view, separate buttons into different columns
so they pick up padding.
Test Plan:
Created, edited and deleted a timer. Viewed a timer and toggled chrome mode.
Viewed timer list.
Reviewed By: hsb
Reviewers: hsb, aran, jungejason, tuomaspelkonen
CC: aran, hsb, epriestley
Differential Revision: 454
2011-06-13 17:35:13 -07:00
|
|
|
public function getShowChrome() {
|
|
|
|
return $this->showChrome;
|
|
|
|
}
|
|
|
|
|
2013-03-04 10:35:50 -08:00
|
|
|
public function getDust() {
|
|
|
|
return $this->layDownSomeDust;
|
|
|
|
}
|
|
|
|
|
2012-02-14 17:00:12 -08:00
|
|
|
public function setSearchDefaultScope($search_default_scope) {
|
|
|
|
$this->searchDefaultScope = $search_default_scope;
|
|
|
|
return $this;
|
|
|
|
}
|
|
|
|
|
|
|
|
public function getSearchDefaultScope() {
|
|
|
|
return $this->searchDefaultScope;
|
|
|
|
}
|
|
|
|
|
2012-06-13 17:28:21 -07:00
|
|
|
public function appendPageObjects(array $objs) {
|
|
|
|
foreach ($objs as $obj) {
|
|
|
|
$this->pageObjects[] = $obj;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2011-01-16 13:51:39 -08:00
|
|
|
public function getTitle() {
|
2011-03-30 19:21:09 -07:00
|
|
|
$use_glyph = true;
|
|
|
|
$request = $this->getRequest();
|
|
|
|
if ($request) {
|
|
|
|
$user = $request->getUser();
|
|
|
|
if ($user && $user->loadPreferences()->getPreference(
|
|
|
|
PhabricatorUserPreferences::PREFERENCE_TITLES) !== 'glyph') {
|
|
|
|
$use_glyph = false;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return ($use_glyph ?
|
|
|
|
$this->getGlyph() : '['.$this->getApplicationName().']').
|
|
|
|
' '.parent::getTitle();
|
2011-01-16 13:51:39 -08:00
|
|
|
}
|
|
|
|
|
2011-01-25 11:31:40 -08:00
|
|
|
|
2011-01-25 09:59:31 -08:00
|
|
|
protected function willRenderPage() {
|
2012-10-16 10:33:47 -07:00
|
|
|
parent::willRenderPage();
|
2011-02-02 13:48:52 -08:00
|
|
|
|
|
|
|
if (!$this->getRequest()) {
|
|
|
|
throw new Exception(
|
2013-03-01 15:37:32 -08:00
|
|
|
pht(
|
|
|
|
"You must set the Request to render a PhabricatorStandardPageView."));
|
2011-02-02 13:48:52 -08:00
|
|
|
}
|
|
|
|
|
2011-07-09 09:45:19 -07:00
|
|
|
$console = $this->getConsole();
|
2011-02-02 13:48:52 -08:00
|
|
|
|
2011-01-25 09:59:31 -08:00
|
|
|
require_celerity_resource('phabricator-core-css');
|
2012-12-30 09:30:21 -08:00
|
|
|
require_celerity_resource('phabricator-zindex-css');
|
2011-01-25 11:31:40 -08:00
|
|
|
require_celerity_resource('phabricator-core-buttons-css');
|
2013-03-30 07:49:32 -07:00
|
|
|
require_celerity_resource('spacing-css');
|
2012-12-07 13:35:49 -08:00
|
|
|
require_celerity_resource('sprite-gradient-css');
|
2011-01-25 11:31:40 -08:00
|
|
|
require_celerity_resource('phabricator-standard-page-view');
|
|
|
|
|
2012-11-19 17:05:10 -08:00
|
|
|
Javelin::initBehavior('workflow', array());
|
|
|
|
|
Prevent CSRF uploads via /file/dropupload/
Summary:
We don't currently validate CSRF tokens on this workflow. This allows an
attacker to upload arbitrary files on the user's behalf. Although I believe the
tight list of servable mime-types means that's more or less the end of the
attack, this is still a vulnerability.
In the long term, the right solution is probably to pass CSRF tokens on all Ajax
requests in an HTTP header (or just a GET param) or something like that.
However, this endpoint is unique and this is the quickest and most direct way to
close the hole.
Test Plan:
- Drop-uploaded files to Files, Maniphest, Phriction and Differential.
- Modified CSRF vaidator to use __csrf__.'x' and verified uploads and form
submissions don't work.
Reviewers: andrewjcg, aran, jungejason, tuomaspelkonen, erling
Commenters: andrewjcg, pedram
CC: aran, epriestley, andrewjcg, pedram
Differential Revision: 758
2011-08-01 20:23:01 -07:00
|
|
|
$request = $this->getRequest();
|
2013-01-28 18:12:09 -08:00
|
|
|
$user = null;
|
Prevent CSRF uploads via /file/dropupload/
Summary:
We don't currently validate CSRF tokens on this workflow. This allows an
attacker to upload arbitrary files on the user's behalf. Although I believe the
tight list of servable mime-types means that's more or less the end of the
attack, this is still a vulnerability.
In the long term, the right solution is probably to pass CSRF tokens on all Ajax
requests in an HTTP header (or just a GET param) or something like that.
However, this endpoint is unique and this is the quickest and most direct way to
close the hole.
Test Plan:
- Drop-uploaded files to Files, Maniphest, Phriction and Differential.
- Modified CSRF vaidator to use __csrf__.'x' and verified uploads and form
submissions don't work.
Reviewers: andrewjcg, aran, jungejason, tuomaspelkonen, erling
Commenters: andrewjcg, pedram
CC: aran, epriestley, andrewjcg, pedram
Differential Revision: 758
2011-08-01 20:23:01 -07:00
|
|
|
if ($request) {
|
|
|
|
$user = $request->getUser();
|
|
|
|
}
|
|
|
|
|
2013-01-28 18:12:09 -08:00
|
|
|
if ($user) {
|
|
|
|
$default_img_uri =
|
|
|
|
PhabricatorEnv::getCDNURI(
|
2013-02-19 13:33:10 -08:00
|
|
|
'/rsrc/image/icon/fatcow/document_black.png');
|
2013-01-30 11:30:38 -08:00
|
|
|
$download_form = phabricator_form(
|
2013-01-28 18:12:09 -08:00
|
|
|
$user,
|
|
|
|
array(
|
|
|
|
'action' => '#',
|
|
|
|
'method' => 'POST',
|
|
|
|
'class' => 'lightbox-download-form',
|
|
|
|
'sigil' => 'download',
|
|
|
|
),
|
|
|
|
phutil_tag(
|
|
|
|
'button',
|
|
|
|
array(),
|
|
|
|
pht('Download')));
|
|
|
|
|
|
|
|
Javelin::initBehavior(
|
|
|
|
'lightbox-attachments',
|
|
|
|
array(
|
|
|
|
'defaultImageUri' => $default_img_uri,
|
|
|
|
'downloadForm' => $download_form,
|
|
|
|
));
|
|
|
|
}
|
|
|
|
|
|
|
|
Javelin::initBehavior('aphront-form-disable-on-submit');
|
2012-07-30 16:08:42 -07:00
|
|
|
Javelin::initBehavior('toggle-class', array());
|
2012-08-22 08:20:23 -07:00
|
|
|
Javelin::initBehavior('konami', array());
|
2013-02-28 18:58:00 -08:00
|
|
|
Javelin::initBehavior('history-install');
|
Add support for device swipe events
Summary:
Ref T2700. Allow JS to listen for swipes on devices.
There are a bunch of tricky cases here and I probably didn't get them all totally right, but this interaction broadly looks like this:
- We implement gesture recognition for the mouse in device modes (narrow browser), and for touch events from an actual device.
- The sigil `touchable` indicates that a node wants to react to touch events.
- When the user touches a `touchable` node, we start listening for moves. They might be tapping/clicking (in which case we don't care), but they might also be gesturing.
- Once the user moves their finger/pointer far enough away from the tap origin, we recognize it as a gesture. I hardcoded this at 20px; I wasn't able to find any "official" Apple value, but 20px seems like a common default.
- At this point, we look at where their finger has moved.
- If they moved it mostly up/down, we interpret the gesture as "scroll" and just stop listening. The device does its own thing.
- However, if they moved it mostly left/right, we interpret it as a "swipe". We start killing the moves so the device doesn't scroll.
- Once we've recognized that a gesture is underway, we send a "gesture.swipe.start" event and then "gesture.swipe.move" events for every move.
- When the user ends the gesture, we send "gesture.swipe.end".
- If the user cancels the gesture (currently, only by tapping with a second finger), we send "gesture.swipe.cancel".
- Gesture events have raw position data and some convenience fields.
Test Plan:
Wrote UI example and used it from the Desktop, iPhone simulator, and a real iphone.
- The code always seems to get "scroll" vs "swipe" correct (i.e., consistent with my intentions).
- The threshold feels pretty good to me.
- Tapping with a second finger cancels the action.
Reviewers: chad, btrahan
Reviewed By: chad
CC: aran
Maniphest Tasks: T2700
Differential Revision: https://secure.phabricator.com/D5308
2013-03-09 13:53:15 -08:00
|
|
|
Javelin::initBehavior('phabricator-gesture');
|
2013-01-28 18:12:09 -08:00
|
|
|
|
|
|
|
$current_token = null;
|
|
|
|
if ($user) {
|
|
|
|
$current_token = $user->getCSRFToken();
|
|
|
|
}
|
|
|
|
|
Prevent CSRF uploads via /file/dropupload/
Summary:
We don't currently validate CSRF tokens on this workflow. This allows an
attacker to upload arbitrary files on the user's behalf. Although I believe the
tight list of servable mime-types means that's more or less the end of the
attack, this is still a vulnerability.
In the long term, the right solution is probably to pass CSRF tokens on all Ajax
requests in an HTTP header (or just a GET param) or something like that.
However, this endpoint is unique and this is the quickest and most direct way to
close the hole.
Test Plan:
- Drop-uploaded files to Files, Maniphest, Phriction and Differential.
- Modified CSRF vaidator to use __csrf__.'x' and verified uploads and form
submissions don't work.
Reviewers: andrewjcg, aran, jungejason, tuomaspelkonen, erling
Commenters: andrewjcg, pedram
CC: aran, epriestley, andrewjcg, pedram
Differential Revision: 758
2011-08-01 20:23:01 -07:00
|
|
|
Javelin::initBehavior(
|
|
|
|
'refresh-csrf',
|
|
|
|
array(
|
|
|
|
'tokenName' => AphrontRequest::getCSRFTokenName(),
|
|
|
|
'header' => AphrontRequest::getCSRFHeaderName(),
|
|
|
|
'current' => $current_token,
|
|
|
|
));
|
2013-01-28 18:12:09 -08:00
|
|
|
|
2012-12-11 14:01:51 -08:00
|
|
|
Javelin::initBehavior('device');
|
2012-03-14 20:47:17 -07:00
|
|
|
|
2011-02-02 13:48:52 -08:00
|
|
|
if ($console) {
|
|
|
|
require_celerity_resource('aphront-dark-console-css');
|
2013-02-09 13:29:47 -08:00
|
|
|
|
|
|
|
$headers = array();
|
2013-02-09 15:47:55 -08:00
|
|
|
if (DarkConsoleXHProfPluginAPI::isProfilerStarted()) {
|
2013-02-09 13:29:47 -08:00
|
|
|
$headers[DarkConsoleXHProfPluginAPI::getProfilerHeader()] = 'page';
|
|
|
|
}
|
|
|
|
|
2011-02-02 13:48:52 -08:00
|
|
|
Javelin::initBehavior(
|
|
|
|
'dark-console',
|
|
|
|
array(
|
DarkConsole: fix rendering, move request log, load over ajax
Summary:
This accomplishes three major goals:
# Fixes phutil_render_tag -> phutil_tag callsites in DarkConsole.
# Moves the Ajax request log to a new panel on the left. This panel (and the tabs panel) get scrollbars when they get large, instead of making the page constantly scroll down.
# Loads the panel content over ajax, instead of dumping it into the page body / ajax response body. I've been planning to do this for about 3 years, which is why the plugins are architected the way they are. This should make debugging easier by making response bodies not be 50%+ darkconsole stuff.
Additionally, load the plugins dynamically (the old method predates library maps and PhutilSymbolLoader).
Test Plan:
{F30675}
- Switched between requests and tabs, reloaded page, saw same tab.
- Used "analyze queries", "profile page", triggered errors.
- Verified page does not load anything by default if dark console is closed with Charles.
- Generally banged on it a bit.
Reviewers: vrana, btrahan, chad
Reviewed By: vrana
CC: aran
Maniphest Tasks: T2432
Differential Revision: https://secure.phabricator.com/D4692
2013-01-28 18:45:32 -08:00
|
|
|
'uri' => $request ? (string)$request->getRequestURI() : '?',
|
|
|
|
'selected' => $user ? $user->getConsoleTab() : null,
|
|
|
|
'visible' => $user ? (int)$user->getConsoleVisible() : true,
|
2013-02-09 13:29:47 -08:00
|
|
|
'headers' => $headers,
|
2011-02-02 13:48:52 -08:00
|
|
|
));
|
2011-04-05 12:24:21 -07:00
|
|
|
|
|
|
|
// Change this to initBehavior when there is some behavior to initialize
|
|
|
|
require_celerity_resource('javelin-behavior-error-log');
|
2011-02-02 13:48:52 -08:00
|
|
|
}
|
|
|
|
|
2012-12-07 13:34:44 -08:00
|
|
|
$menu = id(new PhabricatorMainMenuView())
|
2012-12-07 13:33:03 -08:00
|
|
|
->setUser($request->getUser())
|
2012-12-07 13:34:44 -08:00
|
|
|
->setDefaultSearchScope($this->getSearchDefaultScope());
|
|
|
|
|
2012-12-07 15:29:41 -08:00
|
|
|
if ($this->getController()) {
|
|
|
|
$menu->setController($this->getController());
|
|
|
|
}
|
|
|
|
|
2012-12-07 13:34:44 -08:00
|
|
|
if ($this->getApplicationMenu()) {
|
|
|
|
$menu->setApplicationMenu($this->getApplicationMenu());
|
|
|
|
}
|
|
|
|
|
|
|
|
$this->menuContent = $menu->render();
|
2011-01-25 09:59:31 -08:00
|
|
|
}
|
|
|
|
|
|
|
|
|
2011-01-16 13:51:39 -08:00
|
|
|
protected function getHead() {
|
2012-07-06 15:39:43 -07:00
|
|
|
$monospaced = PhabricatorEnv::getEnvConfig('style.monospace');
|
2013-04-01 13:48:57 -07:00
|
|
|
$monospaced_win = PhabricatorEnv::getEnvConfig('style.monospace.windows');
|
2011-03-30 19:21:09 -07:00
|
|
|
|
|
|
|
$request = $this->getRequest();
|
|
|
|
if ($request) {
|
|
|
|
$user = $request->getUser();
|
|
|
|
if ($user) {
|
2013-04-01 13:48:57 -07:00
|
|
|
$pref = $user->loadPreferences()->getPreference(
|
|
|
|
PhabricatorUserPreferences::PREFERENCE_MONOSPACED);
|
|
|
|
$monospaced = nonempty($pref, $monospaced);
|
|
|
|
$monospaced_win = nonempty($pref, $monospaced_win);
|
2011-03-30 19:21:09 -07:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2012-10-16 10:33:47 -07:00
|
|
|
$response = CelerityAPI::getStaticResourceResponse();
|
2012-07-25 11:51:27 -07:00
|
|
|
|
2013-02-13 14:50:15 -08:00
|
|
|
return hsprintf(
|
2013-04-01 13:48:57 -07:00
|
|
|
'%s<style type="text/css">'.
|
|
|
|
'.PhabricatorMonospaced { font: %s; } '.
|
|
|
|
'.platform-windows .PhabricatorMonospaced { font: %s; }'.
|
|
|
|
'</style>%s',
|
2012-10-16 10:33:47 -07:00
|
|
|
parent::getHead(),
|
2013-02-13 14:50:15 -08:00
|
|
|
phutil_safe_html($monospaced),
|
2013-04-01 13:48:57 -07:00
|
|
|
phutil_safe_html($monospaced_win),
|
2013-02-13 14:50:15 -08:00
|
|
|
$response->renderSingleResource('javelin-magical-init'));
|
2011-01-16 13:51:39 -08:00
|
|
|
}
|
|
|
|
|
|
|
|
public function setGlyph($glyph) {
|
|
|
|
$this->glyph = $glyph;
|
|
|
|
return $this;
|
|
|
|
}
|
|
|
|
|
|
|
|
public function getGlyph() {
|
|
|
|
return $this->glyph;
|
|
|
|
}
|
|
|
|
|
2011-02-02 13:48:52 -08:00
|
|
|
protected function willSendResponse($response) {
|
DarkConsole: fix rendering, move request log, load over ajax
Summary:
This accomplishes three major goals:
# Fixes phutil_render_tag -> phutil_tag callsites in DarkConsole.
# Moves the Ajax request log to a new panel on the left. This panel (and the tabs panel) get scrollbars when they get large, instead of making the page constantly scroll down.
# Loads the panel content over ajax, instead of dumping it into the page body / ajax response body. I've been planning to do this for about 3 years, which is why the plugins are architected the way they are. This should make debugging easier by making response bodies not be 50%+ darkconsole stuff.
Additionally, load the plugins dynamically (the old method predates library maps and PhutilSymbolLoader).
Test Plan:
{F30675}
- Switched between requests and tabs, reloaded page, saw same tab.
- Used "analyze queries", "profile page", triggered errors.
- Verified page does not load anything by default if dark console is closed with Charles.
- Generally banged on it a bit.
Reviewers: vrana, btrahan, chad
Reviewed By: vrana
CC: aran
Maniphest Tasks: T2432
Differential Revision: https://secure.phabricator.com/D4692
2013-01-28 18:45:32 -08:00
|
|
|
$request = $this->getRequest();
|
2012-10-16 10:33:47 -07:00
|
|
|
$response = parent::willSendResponse($response);
|
|
|
|
|
DarkConsole: fix rendering, move request log, load over ajax
Summary:
This accomplishes three major goals:
# Fixes phutil_render_tag -> phutil_tag callsites in DarkConsole.
# Moves the Ajax request log to a new panel on the left. This panel (and the tabs panel) get scrollbars when they get large, instead of making the page constantly scroll down.
# Loads the panel content over ajax, instead of dumping it into the page body / ajax response body. I've been planning to do this for about 3 years, which is why the plugins are architected the way they are. This should make debugging easier by making response bodies not be 50%+ darkconsole stuff.
Additionally, load the plugins dynamically (the old method predates library maps and PhutilSymbolLoader).
Test Plan:
{F30675}
- Switched between requests and tabs, reloaded page, saw same tab.
- Used "analyze queries", "profile page", triggered errors.
- Verified page does not load anything by default if dark console is closed with Charles.
- Generally banged on it a bit.
Reviewers: vrana, btrahan, chad
Reviewed By: vrana
CC: aran
Maniphest Tasks: T2432
Differential Revision: https://secure.phabricator.com/D4692
2013-01-28 18:45:32 -08:00
|
|
|
$console = $request->getApplicationConfiguration()->getConsole();
|
|
|
|
|
2011-02-02 13:48:52 -08:00
|
|
|
if ($console) {
|
2013-02-13 14:50:15 -08:00
|
|
|
$response = PhutilSafeHTML::applyFunction(
|
|
|
|
'str_replace',
|
|
|
|
hsprintf('<darkconsole />'),
|
DarkConsole: fix rendering, move request log, load over ajax
Summary:
This accomplishes three major goals:
# Fixes phutil_render_tag -> phutil_tag callsites in DarkConsole.
# Moves the Ajax request log to a new panel on the left. This panel (and the tabs panel) get scrollbars when they get large, instead of making the page constantly scroll down.
# Loads the panel content over ajax, instead of dumping it into the page body / ajax response body. I've been planning to do this for about 3 years, which is why the plugins are architected the way they are. This should make debugging easier by making response bodies not be 50%+ darkconsole stuff.
Additionally, load the plugins dynamically (the old method predates library maps and PhutilSymbolLoader).
Test Plan:
{F30675}
- Switched between requests and tabs, reloaded page, saw same tab.
- Used "analyze queries", "profile page", triggered errors.
- Verified page does not load anything by default if dark console is closed with Charles.
- Generally banged on it a bit.
Reviewers: vrana, btrahan, chad
Reviewed By: vrana
CC: aran
Maniphest Tasks: T2432
Differential Revision: https://secure.phabricator.com/D4692
2013-01-28 18:45:32 -08:00
|
|
|
$console->render($request),
|
2011-02-02 13:48:52 -08:00
|
|
|
$response);
|
|
|
|
}
|
2012-10-16 10:33:47 -07:00
|
|
|
|
2011-02-02 13:48:52 -08:00
|
|
|
return $response;
|
|
|
|
}
|
|
|
|
|
2011-01-16 13:51:39 -08:00
|
|
|
protected function getBody() {
|
2011-07-09 09:45:19 -07:00
|
|
|
$console = $this->getConsole();
|
2011-01-16 13:51:39 -08:00
|
|
|
|
2011-02-26 20:57:21 -08:00
|
|
|
$user = null;
|
2013-01-17 16:25:47 -08:00
|
|
|
$request = $this->getRequest();
|
2011-01-25 17:17:19 -08:00
|
|
|
if ($request) {
|
|
|
|
$user = $request->getUser();
|
2011-01-26 13:21:12 -08:00
|
|
|
}
|
2011-02-05 12:20:18 -08:00
|
|
|
|
Countdown tweaks
Summary:
A few tweaks to hsb's Countdown implementation:
- Allow the page to be rendered "chromeless", suitable for display on one of
the dozens of monitors everyone has laying around.
- Show title of countdown in deletion dialog.
- When creating a new countdown default to time(), not Dec 31, 1969.
- Add extra "/" after editing to avoid needless redirect.
- Tweak some page titles.
- Show countdown author in list view.
- Highlight tab in list view.
- Tweak menu copy.
- Link countdown title in list view, separate buttons into different columns
so they pick up padding.
Test Plan:
Created, edited and deleted a timer. Viewed a timer and toggled chrome mode.
Viewed timer list.
Reviewed By: hsb
Reviewers: hsb, aran, jungejason, tuomaspelkonen
CC: aran, hsb, epriestley
Differential Revision: 454
2011-06-13 17:35:13 -07:00
|
|
|
$header_chrome = null;
|
|
|
|
if ($this->getShowChrome()) {
|
2012-07-31 06:18:38 -07:00
|
|
|
$header_chrome = $this->menuContent;
|
Countdown tweaks
Summary:
A few tweaks to hsb's Countdown implementation:
- Allow the page to be rendered "chromeless", suitable for display on one of
the dozens of monitors everyone has laying around.
- Show title of countdown in deletion dialog.
- When creating a new countdown default to time(), not Dec 31, 1969.
- Add extra "/" after editing to avoid needless redirect.
- Tweak some page titles.
- Show countdown author in list view.
- Highlight tab in list view.
- Tweak menu copy.
- Link countdown title in list view, separate buttons into different columns
so they pick up padding.
Test Plan:
Created, edited and deleted a timer. Viewed a timer and toggled chrome mode.
Viewed timer list.
Reviewed By: hsb
Reviewers: hsb, aran, jungejason, tuomaspelkonen
CC: aran, hsb, epriestley
Differential Revision: 454
2011-06-13 17:35:13 -07:00
|
|
|
}
|
|
|
|
|
2012-01-04 07:35:52 -08:00
|
|
|
$developer_warning = null;
|
2013-02-01 09:34:06 -08:00
|
|
|
if (PhabricatorEnv::getEnvConfig('phabricator.developer-mode') &&
|
2012-01-04 07:35:52 -08:00
|
|
|
DarkConsoleErrorLogPluginAPI::getErrors()) {
|
2013-01-28 18:42:57 -08:00
|
|
|
$developer_warning = phutil_tag(
|
|
|
|
'div',
|
|
|
|
array(
|
|
|
|
'class' => 'aphront-developer-error-callout',
|
|
|
|
),
|
|
|
|
pht(
|
|
|
|
'This page raised PHP errors. Find them in DarkConsole '.
|
|
|
|
'or the error log.'));
|
2012-01-04 07:35:52 -08:00
|
|
|
}
|
|
|
|
|
2013-01-17 16:25:47 -08:00
|
|
|
// Render the "you have unresolved setup issues..." warning.
|
|
|
|
$setup_warning = null;
|
|
|
|
if ($user && $user->getIsAdmin()) {
|
2013-01-19 08:39:27 -08:00
|
|
|
$open = PhabricatorSetupCheck::getOpenSetupIssueCount();
|
|
|
|
if ($open) {
|
2013-01-25 05:50:50 -08:00
|
|
|
$setup_warning = phutil_tag(
|
2013-01-19 08:39:27 -08:00
|
|
|
'div',
|
|
|
|
array(
|
|
|
|
'class' => 'setup-warning-callout',
|
|
|
|
),
|
2013-01-25 05:50:50 -08:00
|
|
|
phutil_tag(
|
2013-01-19 08:39:27 -08:00
|
|
|
'a',
|
2013-01-17 16:25:47 -08:00
|
|
|
array(
|
2013-01-19 08:39:27 -08:00
|
|
|
'href' => '/config/issue/',
|
2013-01-17 16:25:47 -08:00
|
|
|
),
|
2013-01-19 08:39:27 -08:00
|
|
|
pht('You have %d unresolved setup issue(s)...', $open)));
|
2013-01-17 16:25:47 -08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
Countdown tweaks
Summary:
A few tweaks to hsb's Countdown implementation:
- Allow the page to be rendered "chromeless", suitable for display on one of
the dozens of monitors everyone has laying around.
- Show title of countdown in deletion dialog.
- When creating a new countdown default to time(), not Dec 31, 1969.
- Add extra "/" after editing to avoid needless redirect.
- Tweak some page titles.
- Show countdown author in list view.
- Highlight tab in list view.
- Tweak menu copy.
- Link countdown title in list view, separate buttons into different columns
so they pick up padding.
Test Plan:
Created, edited and deleted a timer. Viewed a timer and toggled chrome mode.
Viewed timer list.
Reviewed By: hsb
Reviewers: hsb, aran, jungejason, tuomaspelkonen
CC: aran, hsb, epriestley
Differential Revision: 454
2011-06-13 17:35:13 -07:00
|
|
|
return
|
2013-02-13 14:50:15 -08:00
|
|
|
phutil_tag(
|
2012-07-25 11:51:27 -07:00
|
|
|
'div',
|
|
|
|
array(
|
|
|
|
'id' => 'base-page',
|
2012-12-11 14:01:51 -08:00
|
|
|
'class' => 'phabricator-standard-page',
|
2012-07-25 11:51:27 -07:00
|
|
|
),
|
2013-02-13 14:50:15 -08:00
|
|
|
hsprintf(
|
|
|
|
'%s%s%s'.
|
|
|
|
'<div class="phabricator-standard-page-body">'.
|
|
|
|
'%s%s<div style="clear: both;"></div>'.
|
|
|
|
'</div>',
|
|
|
|
$developer_warning,
|
|
|
|
$setup_warning,
|
|
|
|
$header_chrome,
|
|
|
|
($console ? hsprintf('<darkconsole />') : null),
|
|
|
|
parent::getBody()));
|
2011-01-16 13:51:39 -08:00
|
|
|
}
|
|
|
|
|
|
|
|
protected function getTail() {
|
2012-07-30 16:09:14 -07:00
|
|
|
$request = $this->getRequest();
|
|
|
|
$user = $request->getUser();
|
|
|
|
|
|
|
|
$container = null;
|
2012-12-07 16:27:01 -08:00
|
|
|
if ($user->isLoggedIn()) {
|
2012-07-30 16:09:14 -07:00
|
|
|
|
|
|
|
$aphlict_object_id = celerity_generate_unique_node_id();
|
|
|
|
$aphlict_container_id = celerity_generate_unique_node_id();
|
|
|
|
|
|
|
|
$client_uri = PhabricatorEnv::getEnvConfig('notification.client-uri');
|
|
|
|
$client_uri = new PhutilURI($client_uri);
|
|
|
|
if ($client_uri->getDomain() == 'localhost') {
|
|
|
|
$this_host = $this->getRequest()->getHost();
|
|
|
|
$this_host = new PhutilURI('http://'.$this_host.'/');
|
|
|
|
$client_uri->setDomain($this_host->getDomain());
|
|
|
|
}
|
|
|
|
|
|
|
|
$enable_debug = PhabricatorEnv::getEnvConfig('notification.debug');
|
|
|
|
Javelin::initBehavior(
|
|
|
|
'aphlict-listen',
|
|
|
|
array(
|
|
|
|
'id' => $aphlict_object_id,
|
|
|
|
'containerID' => $aphlict_container_id,
|
|
|
|
'server' => $client_uri->getDomain(),
|
|
|
|
'port' => $client_uri->getPort(),
|
|
|
|
'debug' => $enable_debug,
|
|
|
|
'pageObjects' => array_fill_keys($this->pageObjects, true),
|
|
|
|
));
|
2013-01-17 18:57:09 -08:00
|
|
|
$container = phutil_tag(
|
2012-07-30 16:09:14 -07:00
|
|
|
'div',
|
|
|
|
array(
|
|
|
|
'id' => $aphlict_container_id,
|
|
|
|
'style' => 'position: absolute; width: 0; height: 0;',
|
|
|
|
),
|
2012-07-31 06:18:38 -07:00
|
|
|
'');
|
2012-07-30 16:09:14 -07:00
|
|
|
}
|
|
|
|
|
2011-01-25 09:59:31 -08:00
|
|
|
$response = CelerityAPI::getStaticResourceResponse();
|
2012-10-16 10:33:47 -07:00
|
|
|
|
|
|
|
$tail = array(
|
|
|
|
parent::getTail(),
|
|
|
|
$container,
|
|
|
|
$response->renderHTMLFooter(),
|
|
|
|
);
|
|
|
|
|
2013-02-13 14:50:15 -08:00
|
|
|
return phutil_implode_html("\n", $tail);
|
2011-01-16 13:51:39 -08:00
|
|
|
}
|
|
|
|
|
Countdown tweaks
Summary:
A few tweaks to hsb's Countdown implementation:
- Allow the page to be rendered "chromeless", suitable for display on one of
the dozens of monitors everyone has laying around.
- Show title of countdown in deletion dialog.
- When creating a new countdown default to time(), not Dec 31, 1969.
- Add extra "/" after editing to avoid needless redirect.
- Tweak some page titles.
- Show countdown author in list view.
- Highlight tab in list view.
- Tweak menu copy.
- Link countdown title in list view, separate buttons into different columns
so they pick up padding.
Test Plan:
Created, edited and deleted a timer. Viewed a timer and toggled chrome mode.
Viewed timer list.
Reviewed By: hsb
Reviewers: hsb, aran, jungejason, tuomaspelkonen
CC: aran, hsb, epriestley
Differential Revision: 454
2011-06-13 17:35:13 -07:00
|
|
|
protected function getBodyClasses() {
|
|
|
|
$classes = array();
|
|
|
|
|
|
|
|
if (!$this->getShowChrome()) {
|
|
|
|
$classes[] = 'phabricator-chromeless-page';
|
|
|
|
}
|
|
|
|
|
2013-03-04 10:35:50 -08:00
|
|
|
if ($this->getDust()) {
|
|
|
|
$classes[] = 'make-me-sneeze';
|
|
|
|
}
|
|
|
|
|
2013-02-09 15:01:57 -08:00
|
|
|
$agent = AphrontRequest::getHTTPHeader('User-Agent');
|
2012-12-11 14:01:51 -08:00
|
|
|
|
|
|
|
// Try to guess the device resolution based on UA strings to avoid a flash
|
|
|
|
// of incorrectly-styled content.
|
|
|
|
$device_guess = 'device-desktop';
|
|
|
|
if (preg_match('@iPhone|iPod|(Android.*Chrome/[.0-9]* Mobile)@', $agent)) {
|
|
|
|
$device_guess = 'device-phone device';
|
|
|
|
} else if (preg_match('@iPad|(Android.*Chrome/)@', $agent)) {
|
|
|
|
$device_guess = 'device-tablet device';
|
|
|
|
}
|
|
|
|
|
|
|
|
$classes[] = $device_guess;
|
|
|
|
|
2013-04-01 13:48:57 -07:00
|
|
|
if (preg_match('@Windows@', $agent)) {
|
|
|
|
$classes[] = 'platform-windows';
|
|
|
|
} else if (preg_match('@Macintosh@', $agent)) {
|
|
|
|
$classes[] = 'platform-mac';
|
|
|
|
} else if (preg_match('@X11@', $agent)) {
|
|
|
|
$classes[] = 'platform-linux';
|
|
|
|
}
|
|
|
|
|
Countdown tweaks
Summary:
A few tweaks to hsb's Countdown implementation:
- Allow the page to be rendered "chromeless", suitable for display on one of
the dozens of monitors everyone has laying around.
- Show title of countdown in deletion dialog.
- When creating a new countdown default to time(), not Dec 31, 1969.
- Add extra "/" after editing to avoid needless redirect.
- Tweak some page titles.
- Show countdown author in list view.
- Highlight tab in list view.
- Tweak menu copy.
- Link countdown title in list view, separate buttons into different columns
so they pick up padding.
Test Plan:
Created, edited and deleted a timer. Viewed a timer and toggled chrome mode.
Viewed timer list.
Reviewed By: hsb
Reviewers: hsb, aran, jungejason, tuomaspelkonen
CC: aran, hsb, epriestley
Differential Revision: 454
2011-06-13 17:35:13 -07:00
|
|
|
return implode(' ', $classes);
|
|
|
|
}
|
|
|
|
|
2011-07-09 09:45:19 -07:00
|
|
|
private function getConsole() {
|
|
|
|
if ($this->disableConsole) {
|
|
|
|
return null;
|
|
|
|
}
|
|
|
|
return $this->getRequest()->getApplicationConfiguration()->getConsole();
|
|
|
|
}
|
2012-07-30 16:09:14 -07:00
|
|
|
|
2011-01-16 13:51:39 -08:00
|
|
|
}
|