[TASK] EXT:form - remove unused url parameters

Set the url parameter "format" only if is set in the configuration
and skip the default extbase arguments.

Resolves: #70474
Releases: master
Change-Id: Ia8cdc3f75725da41c8ef5701cec864a39a0f3d35
Reviewed-on: http://review.typo3.org/43894


Reviewed-by: Bjoern Jacob <bjoern.jacob@tritum.de>
Tested-by: Bjoern Jacob <bjoern.jacob@tritum.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>

typo3/sysext/form/Configuration/TypoScript/setup.txt

@@ -20,6 +20,10 @@ tt_content.mailform.20 {
 <INCLUDE_TYPOSCRIPT: source="DIR: EXT:form/Configuration/TypoScript/Validators" extensions="ts">
 
 plugin.tx_form {
+	features {
+		skipDefaultArguments = 1
+	}
+
 	view {
 		templateRootPaths {
 			10 = {$plugin.tx_form.view.templateRootPath}

typo3/sysext/form/Resources/Private/Partials/Compatibility/Confirmation/ContainerElements/Form.html

@@ -26,7 +26,7 @@
 	controller="Frontend"
 	extensionName="Form"
 
-	format="{model.additionalArguments.format}"
+	format="{f:if(condition:'{model.additionalArguments.format}',then:'{model.additionalArguments.format}')}"
 	hiddenFieldClassName="{model.additionalArguments.hiddenFieldClassName}"
 
 	object="{model}"

typo3/sysext/form/Resources/Private/Partials/Compatibility/Show/ContainerElements/Form.html

@@ -23,17 +23,17 @@
 	additionalParams="{model.additionalArguments.additionalParams}"
 	arguments="{model.additionalArguments.arguments}"
 	argumentsToBeExcludedFromQueryString="{model.additionalArguments.argumentsToBeExcludedFromQueryString}"
-	controller="Frontend"
-	extensionName="Form"
+	controller="{model.additionalArguments.controller}"
+	extensionName="{model.additionalArguments.extensionName}"
 
-	format="{model.additionalArguments.format}"
+	format="{f:if(condition:'{model.additionalArguments.format}',then:'{model.additionalArguments.format}')}"
 	hiddenFieldClassName="{model.additionalArguments.hiddenFieldClassName}"
 
 	object="{model}"
 
 	pageType="{model.additionalArguments.pageType}"
 	pageUid="{model.additionalArguments.pageUid}"
-	pluginName="Form"
+	pluginName="{model.additionalArguments.pluginName}"
 	section="{model.additionalArguments.section}">
 
 	<f:format.raw>{model.layout.containerInnerWrap.0}</f:format.raw>

typo3/sysext/form/Resources/Private/Partials/Default/Confirmation/ContainerElements/Form.html

@@ -26,7 +26,7 @@
 	controller="Frontend"
 	extensionName="Form"
 
-	format="{model.additionalArguments.format}"
+	format="{f:if(condition:'{model.additionalArguments.format}',then:'{model.additionalArguments.format}')}"
 	hiddenFieldClassName="{model.additionalArguments.hiddenFieldClassName}"
 
 	object="{model}"

typo3/sysext/form/Resources/Private/Partials/Default/Show/ContainerElements/Form.html

@@ -23,17 +23,17 @@
 	additionalParams="{model.additionalArguments.additionalParams}"
 	arguments="{model.additionalArguments.arguments}"
 	argumentsToBeExcludedFromQueryString="{model.additionalArguments.argumentsToBeExcludedFromQueryString}"
-	controller="Frontend"
-	extensionName="Form"
+	controller="{model.additionalArguments.controller}"
+	extensionName="{model.additionalArguments.extensionName}"
 
-	format="{model.additionalArguments.format}"
+	format="{f:if(condition:'{model.additionalArguments.format}',then:'{model.additionalArguments.format}')}"
 	hiddenFieldClassName="{model.additionalArguments.hiddenFieldClassName}"
 
 	object="{model}"
 
 	pageType="{model.additionalArguments.pageType}"
 	pageUid="{model.additionalArguments.pageUid}"
-	pluginName="Form"
+	pluginName="{model.additionalArguments.pluginName}"
 	section="{model.additionalArguments.section}">
 
 	<ol>