Fixes "Server cannot modify cookies after HTTP headers have been sent".

mgesing · mgesing · commit 7b44b30391fd · 2018-07-18T16:01:10.000+02:00
A child-action should never have a post-action of the same name.
diff --git a/changelog.md b/changelog.md
@@ -36,6 +36,7 @@
 * Switching the language always redirected to the home page if SEO friendly URLs was deactivated.
 * File upload of a checkout attribute was not stored on cart page.
 * Redirecting within checkout may have displayed an incorrect URL in the browser.
+* Server cannot modify cookies after HTTP headers have been sent.
 
 
 ## SmartStore.NET 3.1.5
diff --git a/src/Presentation/SmartStore.Web/Controllers/CommonController.cs b/src/Presentation/SmartStore.Web/Controllers/CommonController.cs
@@ -855,7 +855,7 @@ public ActionResult CookieConsentBadge()
 		}
 
 		[HttpPost]
-		public ActionResult CookieConsentBadge(CookieConsentModel model)
+		public ActionResult SetCookieConsentBadge(CookieConsentModel model)
 		{
 			CookieConsent.SetCookieConsent(Response, true);
 
diff --git a/src/Presentation/SmartStore.Web/Views/Common/Partials/CookieConsentBadge.cshtml b/src/Presentation/SmartStore.Web/Views/Common/Partials/CookieConsentBadge.cshtml
@@ -3,7 +3,7 @@
 	Layout = "";
 }
 
-@using (Ajax.BeginForm("CookieConsentBadge", "Common", null, new AjaxOptions()
+@using (Ajax.BeginForm("SetCookieConsentBadge", "Common", null, new AjaxOptions()
 {
 	HttpMethod = "POST",
 	OnComplete = "OnConsented"

Original file line number	Diff line number	Diff line change
`@@ -855,7 +855,7 @@ public ActionResult CookieConsentBadge()`
`855`	`855`	`}`
`856`	`856`
`857`	`857`	`[HttpPost]`
`858`		`- public ActionResult CookieConsentBadge(CookieConsentModel model)`
	`858`	`+ public ActionResult SetCookieConsentBadge(CookieConsentModel model)`
`859`	`859`	`{`
`860`	`860`	`CookieConsent.SetCookieConsent(Response, true);`
`861`	`861`
Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@`
`3`	`3`	`Layout = "";`
`4`	`4`	`}`
`5`	`5`
`6`		`-@using (Ajax.BeginForm("CookieConsentBadge", "Common", null, new AjaxOptions()`
	`6`	`+@using (Ajax.BeginForm("SetCookieConsentBadge", "Common", null, new AjaxOptions()`
`7`	`7`	`{`
`8`	`8`	`HttpMethod = "POST",`
`9`	`9`	`OnComplete = "OnConsented"`