You've already forked phpmyadmin
mirror of
https://github.com/phpmyadmin/phpmyadmin.git
synced 2026-10-29 21:24:44 +03:00
Remove unneeded writing to $_POST in Database\Structure controllers
Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
This commit is contained in:
@@ -36,8 +36,6 @@ final class AddPrefixTableController extends AbstractController
|
||||
|
||||
public function __invoke(ServerRequest $request): void
|
||||
{
|
||||
$GLOBALS['message'] = $GLOBALS['message'] ?? null;
|
||||
|
||||
$selected = $_POST['selected'] ?? [];
|
||||
|
||||
$GLOBALS['sql_query'] = '';
|
||||
@@ -55,10 +53,6 @@ final class AddPrefixTableController extends AbstractController
|
||||
|
||||
$GLOBALS['message'] = Message::success();
|
||||
|
||||
if (empty($_POST['message'])) {
|
||||
$_POST['message'] = $GLOBALS['message'];
|
||||
}
|
||||
|
||||
($this->structureController)($request);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -36,8 +36,6 @@ final class CopyTableController extends AbstractController
|
||||
|
||||
public function __invoke(ServerRequest $request): void
|
||||
{
|
||||
$GLOBALS['message'] = $GLOBALS['message'] ?? null;
|
||||
|
||||
$selected = $_POST['selected'] ?? [];
|
||||
$targetDb = $_POST['target_db'] ?? null;
|
||||
$selectedCount = count($selected);
|
||||
@@ -63,10 +61,6 @@ final class CopyTableController extends AbstractController
|
||||
|
||||
$GLOBALS['message'] = Message::success();
|
||||
|
||||
if (empty($_POST['message'])) {
|
||||
$_POST['message'] = $GLOBALS['message'];
|
||||
}
|
||||
|
||||
($this->structureController)($request);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -32,8 +32,6 @@ final class CopyTableWithPrefixController extends AbstractController
|
||||
|
||||
public function __invoke(ServerRequest $request): void
|
||||
{
|
||||
$GLOBALS['message'] = $GLOBALS['message'] ?? null;
|
||||
|
||||
$selected = $_POST['selected'] ?? [];
|
||||
$fromPrefix = $_POST['from_prefix'] ?? null;
|
||||
$toPrefix = $_POST['to_prefix'] ?? null;
|
||||
@@ -58,10 +56,6 @@ final class CopyTableWithPrefixController extends AbstractController
|
||||
|
||||
$GLOBALS['message'] = Message::success();
|
||||
|
||||
if (empty($_POST['message'])) {
|
||||
$_POST['message'] = $GLOBALS['message'];
|
||||
}
|
||||
|
||||
($this->structureController)($request);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -45,8 +45,6 @@ final class DropTableController extends AbstractController
|
||||
|
||||
public function __invoke(ServerRequest $request): void
|
||||
{
|
||||
$GLOBALS['message'] = $GLOBALS['message'] ?? null;
|
||||
|
||||
$GLOBALS['reload'] = $_POST['reload'] ?? $GLOBALS['reload'] ?? null;
|
||||
$multBtn = $_POST['mult_btn'] ?? '';
|
||||
$selected = $_POST['selected'] ?? [];
|
||||
@@ -56,10 +54,6 @@ final class DropTableController extends AbstractController
|
||||
if ($multBtn !== __('Yes')) {
|
||||
$GLOBALS['message'] = Message::success(__('No change'));
|
||||
|
||||
if (empty($_POST['message'])) {
|
||||
$_POST['message'] = Message::success();
|
||||
}
|
||||
|
||||
unset($_POST['mult_btn']);
|
||||
|
||||
($this->structureController)($request);
|
||||
@@ -121,10 +115,6 @@ final class DropTableController extends AbstractController
|
||||
|
||||
$GLOBALS['message'] = Message::success();
|
||||
|
||||
if (empty($_POST['message'])) {
|
||||
$_POST['message'] = $GLOBALS['message'];
|
||||
}
|
||||
|
||||
unset($_POST['mult_btn']);
|
||||
|
||||
($this->structureController)($request);
|
||||
|
||||
@@ -64,8 +64,6 @@ final class EmptyTableController extends AbstractController
|
||||
|
||||
public function __invoke(ServerRequest $request): void
|
||||
{
|
||||
$GLOBALS['message'] = $GLOBALS['message'] ?? null;
|
||||
|
||||
$multBtn = $_POST['mult_btn'] ?? '';
|
||||
$selected = $_POST['selected'] ?? [];
|
||||
|
||||
@@ -107,10 +105,6 @@ final class EmptyTableController extends AbstractController
|
||||
|
||||
$GLOBALS['message'] = Message::success();
|
||||
|
||||
if (empty($_POST['message'])) {
|
||||
$_POST['message'] = $GLOBALS['message'];
|
||||
}
|
||||
|
||||
unset($_POST['mult_btn']);
|
||||
|
||||
($this->structureController)($request);
|
||||
|
||||
@@ -38,8 +38,6 @@ final class ReplacePrefixController extends AbstractController
|
||||
|
||||
public function __invoke(ServerRequest $request): void
|
||||
{
|
||||
$GLOBALS['message'] = $GLOBALS['message'] ?? null;
|
||||
|
||||
$selected = $_POST['selected'] ?? [];
|
||||
$fromPrefix = $_POST['from_prefix'] ?? '';
|
||||
$toPrefix = $_POST['to_prefix'] ?? '';
|
||||
@@ -67,10 +65,6 @@ final class ReplacePrefixController extends AbstractController
|
||||
|
||||
$GLOBALS['message'] = Message::success();
|
||||
|
||||
if (empty($_POST['message'])) {
|
||||
$_POST['message'] = $GLOBALS['message'];
|
||||
}
|
||||
|
||||
($this->structureController)($request);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user