Don't inherit backward compatibility controllers from BasePublicContr…

…oller, since they only redirect requests.
chan2will · Dec 10, 2019 · f6edf56 · f6edf56
1 parent ee3d403
commit f6edf56
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/Presentation/Nop.Web/Controllers/BackwardCompatibility1XController.cs b/src/Presentation/Nop.Web/Controllers/BackwardCompatibility1XController.cs
@@ -12,7 +12,9 @@
 
 namespace Nop.Web.Controllers
 {
-    public partial class BackwardCompatibility1XController : BasePublicController
+    //do not inherit it from BasePublicController. otherwise a lot of extra action filters will be called
+    //they can create guest account(s), etc
+    public partial class BackwardCompatibility1XController : Controller
     {
         #region Fields
 

diff --git a/src/Presentation/Nop.Web/Controllers/BackwardCompatibility2XController.cs b/src/Presentation/Nop.Web/Controllers/BackwardCompatibility2XController.cs
@@ -8,7 +8,9 @@
 
 namespace Nop.Web.Controllers
 {
-    public partial class BackwardCompatibility2XController : BasePublicController
+    //do not inherit it from BasePublicController. otherwise a lot of extra action filters will be called
+    //they can create guest account(s), etc
+    public partial class BackwardCompatibility2XController : Controller
     {
         #region Fields