Bugfix for vanished media setting allowedextrathumbnailsizes when saving media settings in admin area

Michael-Herzog · Michael-Herzog · commit 0a96a70d9fc2 · 2022-06-14T21:22:21.000+02:00
diff --git a/src/Presentation/SmartStore.Web/Administration/Controllers/SettingController.cs b/src/Presentation/SmartStore.Web/Administration/Controllers/SettingController.cs
@@ -913,7 +913,7 @@ public ActionResult Media(MediaSettings mediaSettings)
         [Permission(Permissions.Configuration.Setting.Update)]
         [ValidateAntiForgeryToken]
         [HttpPost, FormValueRequired("save")]
-        [SaveSetting(UpdateParameterFromStore = false)]
+        [SaveSetting]
         public ActionResult Media(MediaSettings settings, MediaSettingsModel model)
         {
             if (!ModelState.IsValid)
@@ -922,7 +922,7 @@ public ActionResult Media(MediaSettings settings, MediaSettingsModel model)
             }
 
             ModelState.Clear();
-            settings = model.ToEntity();
+            model.ToEntity(settings);
 
             return NotifyAndRedirect("Media");
         }

Original file line number	Diff line number	Diff line change
`@@ -913,7 +913,7 @@ public ActionResult Media(MediaSettings mediaSettings)`
`913`	`913`	`[Permission(Permissions.Configuration.Setting.Update)]`
`914`	`914`	`[ValidateAntiForgeryToken]`
`915`	`915`	`[HttpPost, FormValueRequired("save")]`
`916`		`- [SaveSetting(UpdateParameterFromStore = false)]`
	`916`	`+ [SaveSetting]`
`917`	`917`	`public ActionResult Media(MediaSettings settings, MediaSettingsModel model)`
`918`	`918`	`{`
`919`	`919`	`if (!ModelState.IsValid)`
`@@ -922,7 +922,7 @@ public ActionResult Media(MediaSettings settings, MediaSettingsModel model)`
`922`	`922`	`}`
`923`	`923`
`924`	`924`	`ModelState.Clear();`
`925`		`- settings = model.ToEntity();`
	`925`	`+ model.ToEntity(settings);`
`926`	`926`
`927`	`927`	`return NotifyAndRedirect("Media");`
`928`	`928`	`}`