Refactor deprecated App::isAjax() to DI::mode()->isAjax()

This commit is contained in:
Philipp Holzer 2019-12-16 00:30:39 +01:00
parent 8e6973b774
commit 8b6e123b3f
No known key found for this signature in database
GPG key ID: D8365C3D36B77D90
7 changed files with 12 additions and 17 deletions

View file

@ -878,7 +878,7 @@ function item_content(App $a)
$o = ''; $o = '';
if (($a->argc >= 3) && ($a->argv[1] === 'drop') && intval($a->argv[2])) { if (($a->argc >= 3) && ($a->argv[1] === 'drop') && intval($a->argv[2])) {
if ($a->isAjax()) { if (DI::mode()->isAjax()) {
$o = Item::deleteForUser(['id' => $a->argv[2]], local_user()); $o = Item::deleteForUser(['id' => $a->argv[2]], local_user());
} else { } else {
if (!empty($a->argv[3])) { if (!empty($a->argv[3])) {
@ -889,7 +889,7 @@ function item_content(App $a)
} }
} }
if ($a->isAjax()) { if (DI::mode()->isAjax()) {
// ajax return: [<item id>, 0 (no perm) | <owner id>] // ajax return: [<item id>, 0 (no perm) | <owner id>]
echo json_encode([intval($a->argv[2]), intval($o)]); echo json_encode([intval($a->argv[2]), intval($o)]);
exit(); exit();

View file

@ -543,15 +543,6 @@ class App
return Core\Theme::getStylesheetPath($this->getCurrentTheme()); return Core\Theme::getStylesheetPath($this->getCurrentTheme());
} }
/**
* @deprecated 2019.09 - use App\Mode->isAjax() instead
* @see App\Mode::isAjax()
*/
public function isAjax()
{
return $this->mode->isAjax();
}
/** /**
* @deprecated use Arguments->get() instead * @deprecated use Arguments->get() instead
* *

View file

@ -31,6 +31,8 @@ class Authentication
{ {
/** @var Configuration */ /** @var Configuration */
private $config; private $config;
/** @var App\Mode */
private $mode;
/** @var App\BaseURL */ /** @var App\BaseURL */
private $baseUrl; private $baseUrl;
/** @var L10n */ /** @var L10n */
@ -48,6 +50,7 @@ class Authentication
* Authentication constructor. * Authentication constructor.
* *
* @param Configuration $config * @param Configuration $config
* @param App\Mode $mode
* @param App\BaseURL $baseUrl * @param App\BaseURL $baseUrl
* @param L10n $l10n * @param L10n $l10n
* @param Database $dba * @param Database $dba
@ -55,9 +58,10 @@ class Authentication
* @param User\Cookie $cookie * @param User\Cookie $cookie
* @param Session\ISession $session * @param Session\ISession $session
*/ */
public function __construct(Configuration $config, App\BaseURL $baseUrl, L10n $l10n, Database $dba, LoggerInterface $logger, User\Cookie $cookie, Session\ISession $session) public function __construct(Configuration $config, App\Mode $mode, App\BaseURL $baseUrl, L10n $l10n, Database $dba, LoggerInterface $logger, User\Cookie $cookie, Session\ISession $session)
{ {
$this->config = $config; $this->config = $config;
$this->mode = $mode;
$this->baseUrl = $baseUrl; $this->baseUrl = $baseUrl;
$this->l10n = $l10n; $this->l10n = $l10n;
$this->dba = $dba; $this->dba = $dba;
@ -404,7 +408,7 @@ class Authentication
} }
// Case 2: No valid 2FA session: redirect to code verification page // Case 2: No valid 2FA session: redirect to code verification page
if ($a->isAjax()) { if ($this->mode->isAjax()) {
throw new HTTPException\ForbiddenException(); throw new HTTPException\ForbiddenException();
} else { } else {
$this->baseUrl->redirect('2fa'); $this->baseUrl->redirect('2fa');

View file

@ -32,7 +32,7 @@ class Details extends BaseAdminModule
info(L10n::t('Theme settings updated.')); info(L10n::t('Theme settings updated.'));
if ($a->isAjax()) { if (DI::mode()->isAjax()) {
return; return;
} }

View file

@ -46,7 +46,7 @@ class Embed extends BaseAdminModule
info(L10n::t('Theme settings updated.')); info(L10n::t('Theme settings updated.'));
if ($a->isAjax()) { if (DI::mode()->isAjax()) {
return; return;
} }

View file

@ -31,7 +31,7 @@ class ItemBody extends BaseModule
$item = Item::selectFirst(['body'], ['uid' => local_user(), 'id' => $itemId]); $item = Item::selectFirst(['body'], ['uid' => local_user(), 'id' => $itemId]);
if (!empty($item)) { if (!empty($item)) {
if ($app->isAjax()) { if (DI::mode()->isAjax()) {
echo str_replace("\n", '<br />', $item['body']); echo str_replace("\n", '<br />', $item['body']);
exit(); exit();
} else { } else {

View file

@ -24,7 +24,7 @@ class Group extends BaseModule
{ {
$a = DI::app(); $a = DI::app();
if ($a->isAjax()) { if (DI::mode()->isAjax()) {
self::ajaxPost(); self::ajaxPost();
} }