Skip to content
Snippets Groups Projects
Commit 967192b1 authored by Kevin Ditscheid's avatar Kevin Ditscheid
Browse files

[BUGFIX] Do not add all the arguments to the redirect on template save

The only arguments to be sent should be the required ones:
selectedExtension and selectedTemplate to prevent the redirect url
from becoming to long for some web servers to handle
parent 835201e8
No related branches found
No related tags found
No related merge requests found
......@@ -257,6 +257,13 @@ class MailController extends ActionController {
* @throws \TYPO3\CMS\Core\Cache\Exception\NoSuchCacheException
*/
public function sendTestMailAction(array $parameters = []) {
$arguments = [];
if ($parameters['selectedExtension']) {
$arguments['selectedExtension'] = $parameters['selectedExtension'];
}
if ($parameters['selectedTemplate']) {
$arguments['selectedTemplate'] = $parameters['selectedTemplate'];
}
foreach ((array) $parameters['templates'] as $parameter) {
$ccAddresses = GeneralUtility::trimExplode(',', $parameter['cc']);
if (\count($ccAddresses) > 0) {
......@@ -266,7 +273,6 @@ class MailController extends ActionController {
$message = LocalizationUtility::translate('backend.error_cc', 'sg_mail');
$this->addFlashMessage($message, '', FlashMessage::WARNING);
$arguments = $this->request->getArguments();
$this->redirect('index', NULL, NULL, $arguments);
}
}
......@@ -279,7 +285,6 @@ class MailController extends ActionController {
$message = LocalizationUtility::translate('backend.error_bcc', 'sg_mail');
$this->addFlashMessage($message, '', FlashMessage::WARNING);
$arguments = $this->request->getArguments();
$this->redirect('index', NULL, NULL, $arguments);
}
}
......@@ -298,7 +303,6 @@ class MailController extends ActionController {
if (!$this->request->hasArgument('saveOnly')) {
if (!filter_var($parameters['emailAddress'], FILTER_VALIDATE_EMAIL)) {
$arguments = $this->request->getArguments();
$this->redirect('index', NULL, NULL, $arguments);
}
......@@ -326,7 +330,6 @@ class MailController extends ActionController {
$this->addFlashMessage($message, '', FlashMessage::ERROR);
}
}
$arguments = $this->request->getArguments();
$this->redirect('index', NULL, NULL, $arguments);
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment