View Options

	diff --git a/src/applications/console/controller/DarkConsoleController.php b/src/applications/console/controller/DarkConsoleController.php
	index 026083cb59..3a677920ee 100644
	--- a/src/applications/console/controller/DarkConsoleController.php
	+++ b/src/applications/console/controller/DarkConsoleController.php
	@@ -1,64 +1,75 @@
	<?php

	final class DarkConsoleController extends PhabricatorController {

	protected $op;
	protected $data;

	public function shouldRequireLogin() {
	return !PhabricatorEnv::getEnvConfig('darkconsole.always-on');
	}

	public function shouldRequireEnabledUser() {
	return !PhabricatorEnv::getEnvConfig('darkconsole.always-on');
	}

	public function shouldAllowPartialSessions() {
	return true;
	}

	public function handleRequest(AphrontRequest $request) {
	$viewer = $this->getViewer();
	$response = id(new AphrontAjaxResponse())->setDisableConsole(true);

	if (!$viewer->isLoggedIn()) {
	return $response;
	}

	$visible = $request->getStr('visible');
	if (strlen($visible)) {
	$this->writeDarkConsoleSetting(
	PhabricatorDarkConsoleVisibleSetting::SETTINGKEY,
	(int)$visible);
	return $response;
	}

	$tab = $request->getStr('tab');
	if (strlen($tab)) {
	$this->writeDarkConsoleSetting(
	PhabricatorDarkConsoleTabSetting::SETTINGKEY,
	$tab);
	return $response;
	}

	return new Aphront404Response();
	}

	private function writeDarkConsoleSetting($key, $value) {
	$viewer = $this->getViewer();
	$request = $this->getRequest();

	$preferences = PhabricatorUserPreferences::loadUserPreferences($viewer);

	$editor = id(new PhabricatorUserPreferencesEditor())
	->setActor($viewer)
	->setContentSourceFromRequest($request)
	->setContinueOnNoEffect(true)
	->setContinueOnMissingFields(true);

	$xactions = array();
	$xactions[] = $preferences->newTransaction($key, $value);
	$editor->applyTransactions($preferences, $xactions);
	+
	+ // Reload the user to regenerate their preferences cache. If we don't
	+ // do this, the "Services" tab gets misleadingly spammed up with cache
	+ // fills that are only filling because you toggled the console or switched
	+ // tabs. This makes it harder to see what's really going on, so just force
	+ // a cache regeneration here.
	+ id(new PhabricatorPeopleQuery())
	+ ->setViewer($viewer)
	+ ->withPHIDs(array($viewer->getPHID()))
	+ ->needUserSettings(true)
	+ ->execute();
	}

	}

File Metadata

Mime Type: text/x-diff
Expires: Mon, Mar 17, 3:37 AM (1 d, 14 h)
Storage Engine: blob
Storage Format: Raw Data
Storage Handle: 72250
Default Alt Text: (2 KB)

No OneTemporary
Actions

View Options

File Metadata

Event Timeline

No OneTemporaryActions

View Options

File Metadata

Event Timeline

No OneTemporary
Actions