First code submission for MVC Pipeline by donker · Pull Request #6749 · dnnsoftware/Dnn.Platform

donker · 2025-10-03T13:34:00Z

Core Purpose

Introduces a new ASP.NET MVC-based rendering pipeline as an alternative to the traditional WebForms pipeline, allowing DNN to modernize its architecture while maintaining backward compatibility.

Key Features

Dual Pipeline Architecture:

Traditional WebForms pipeline continues through /default.aspx
New MVC pipeline routes through /DesktopModules/Default/Page/{tabId}/{locale}

Video showcase the general usage

Module Support:

More info about MVC Module Control Implementation
Razor+ Module Development Guide

HTML module converted to MVC
Sample modules demonstrating Razor+, MVC and SPA patterns

How to compile Samples

Configuration:

Pipeline selection at portal and page level
Settings UI integrated into Site and Page Settings
URL rewriter modifications to route based on pipeline settings

More info about MVC pipeline (routing / settings)

New MVC Pipeline Projects

DotNetNuke.Web.MvcPipeline.csproj - New core MVC pipeline library

Skins : Html Helpers (replacement for skin objects)
Containers : Html Helpers (replacement for skin objects for containers)
ModelFactories : construction of the Page Model (Model of the DefaultController)
ModuleControl : Module Control infrastructure
Utils : MvcViewEngine (Replacement for RazorEngine based on MVC 5 and can only be used inside HttpContext)
UI/Utilities : MvcClientAPI (replacement for ClientAPI for mvc)

Razor Module Sample

Razor-based module sample project

NewDDRMenu

Fork of DDRMenu for MVC navigation support (WIP)

Core Platform Projects

DotNetNuke.Library

Added Site and Page pipeline settings
Added Module Control settings
URL rewriter changes to route based on pipeline settings
Update SPA (Html5) to use new CDF

DotNetNuke.Abstractions

Added PagePipeline property to IPortalSettings

DotNetNuke.Web.Mvc

Integration with new MVC pipeline
Make actual DNN MVC modules run on the MVC Pipeline

HTML

Add support for MVC Pipeline rendering
Add MVC views and controllers
This is implemented as true MVC (with form tags) using AJAX

Dnn.PersonaBar.UI

UI for pipeline settings in Site and Page Settings

DNNConnect.CKE

HTML helper additions for CKEditor in MVC context

Skins/Aperture

Added mvc suport to the skin (Razor files)

Skin conversion guide

Website

MVC Razor Layout
MVC Default skin
mvc.js minimal stuff needed for dnn to work
Terms and privacy pages (razor files)
DefaultController
ModuleActions Controller

AdminExperiance

settings ui for new settings
mvc controls for persona and edit bar

This PR is a first submission of the work of the MVC Pipeline team to bring the main project up to date with what has been developed in a forked repo.

Documentation

MVC Module Control Implementation
MVC pipeline (routing / settings)
Razor Module Development Guide
How to compile Samples

DNN MVC Pipeline - Developer User Guide

…tomizations for the MVC pipeline.

Fix layout issue

- fix case

client dependecy for mvc (because it not exist in nuget)

Mvcpl2

Updated user guide for MVC pipeline with corrections and clarifications.

…pipeline-old

…auwaen/Dnn.Platform into feature/mvc-pipeline-old

# Conflicts: # DNN Platform/DotNetNuke.Web.Mvc/Framework/Controllers/DnnController.cs # DNN Platform/DotNetNuke.Web.Mvc/Framework/Modules/ResultCapturingActionInvoker.cs # DNN Platform/DotNetNuke.Web.Mvc/Routing/MvcRoutingManager.cs # DNN Platform/Library/Entities/Portals/PortalSettings.cs # DNN Platform/Library/Entities/Portals/PortalSettingsController.cs # DNN Platform/Library/Entities/Urls/AdvancedUrlRewriter.cs # DNN Platform/Library/UI/Modules/Html5/Html5HostControl.cs # DNN Platform/Library/UI/Modules/Html5/Html5ModuleControlFactory.cs # DNN Platform/Modules/DDRMenu/Common/DNNContext.cs # DNN Platform/Modules/DDRMenu/Localisation/Localiser.cs # DNN Platform/Modules/DDRMenu/MenuBase.cs # DNN Platform/Modules/DDRMenu/TemplateEngine/TemplateDefinition.cs # DNN Platform/Modules/HTML/Components/HtmlTextController.cs # DNN Platform/Providers/HtmlEditorProviders/DNNConnect.CKE/Web/EditorControl.cs # Dnn.AdminExperience/Dnn.PersonaBar.Extensions/admin/personaBar/Dnn.SiteSettings/App_LocalResources/SiteSettings.resx

sachatrauwaen · 2026-04-02T09:52:34Z

I’d like to check the status of this PR. There hasn’t been any review activity for several months—are there any issues or blockers? How can I help move things forward?

It might also be worth bringing this up during a TAG meeting ?

I completely understand that reviewing this PR is a significant effort. At the same time, on the otherhand inoticing quite a few major changes being made elsewhere, which also require substantial merge work on my side.

I believe it’s generally more efficient to process PRs in order (except for bug fixes, of course). Otherwise, each new PR tends to create additional merge overhead for those still pending.

thienvc · 2026-04-04T14:32:47Z

I’d like to check the status of this PR. There hasn’t been any review activity for several months—are there any issues or blockers? How can I help move things forward?

It might also be worth bringing this up during a TAG meeting ?

I completely understand that reviewing this PR is a significant effort. At the same time, on the otherhand inoticing quite a few major changes being made elsewhere, which also require substantial merge work on my side.

I believe it’s generally more efficient to process PRs in order (except for bug fixes, of course). Otherwise, each new PR tends to create additional merge overhead for those still pending.

I wish this PR was a custom package that could be installed and uninstalled like a plugin.
If that were the case, I think it wouldn't affect the official version.

mitchelsellers · 2026-04-04T22:38:26Z

DNN Platform/DotNetNuke.ContentSecurityPolicy/CspExtensions.cs

+
+            if (isAuthenticated)
+            {
+                csp.FrameSource.AddHost("https://dnndocs.com").AddHost("https://docs.dnncommunity.org");


I don't think this should be a global approval, this will result in some really funny security responses most likely

mitchelsellers · 2026-04-04T22:42:07Z

DNN Platform/DotNetNuke.Web.Mvc/StandardTabAndModuleInfoProvider.cs

                value = requestBase.Headers[key];
            }

+            if (requestBase.Form[key] != null)


I would like to review/discuss the possible risks wit htis change, by accpeting the value from the form in any situation , as I believe this might expose a different method of manipulation that could have unintended consequences.

mitchelsellers · 2026-04-04T22:46:44Z

DNN Platform/DotNetNuke.Web.MvcPipeline/Commons/PropertyHelper.cs

+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+// See the LICENSE file in the project root for more information
+namespace DotNetNuke.Web.MvcPipeline.Commons


This entire implementation already exists within the solution at DotNetNuke.Web.Mvc.Common

Should this be a move? Should the old file be marked as Obsolete?

mitchelsellers · 2026-04-04T22:48:11Z

DNN Platform/DotNetNuke.Web.MvcPipeline/Commons/TypeHelper.cs

+// The .NET Foundation licenses this file to you under the MIT license.
+// See the LICENSE file in the project root for more information
+
+namespace DotNetNuke.Web.MvcPipeline.Commons


To match other naming conventions if needed this namespace/folder should be Common not Commons

mitchelsellers · 2026-04-04T22:52:23Z

DNN Platform/DotNetNuke.Web.MvcPipeline/Controllers/DnnPageController.cs

+    /// <summary>
+    /// Base controller for DNN MVC page controllers, exposing common services and portal context.
+    /// </summary>
+    public abstract class DnnPageController : Controller, IMvcController


Similar question here for the fate of the existing IDNNController and related items, as I can see a lot of confusion without some level of guidance being provided at a programming perspective with different behaviors and purposes. For example this ONLY provides PortalSettings the legacy one provides much more rich context (User, etc)

mitchelsellers · 2026-04-04T22:55:27Z

DNN Platform/DotNetNuke.Web.MvcPipeline/Controllers/ModuleViewControllerBase.cs

+        /// Builds an edit URL for the current module instance.
+        /// </summary>
+        /// <returns>The edit URL.</returns>
+        public string EditUrl()


I echo @valadas comments on this duplication of exploding items that are already there, the long-term maintenance of this, as we have seen with other changes, is not exactly "fun". I would prefer exposing the ModuleContext and leaving it at that personally

mitchelsellers · 2026-04-04T22:58:26Z

DNN Platform/DotNetNuke.Web.MvcPipeline/Framework/JavascriptLibraries/MvcJavaScript.cs

+                  .Register();
+
+            /*
+            if (Host.CdnEnabled && !string.IsNullOrEmpty(jsl.ObjectName))


This commented out code should be removed

donker and others added 30 commits March 20, 2025 11:36

Initial port of MVC-PAGE branch to DNN 10

96b3954

Fixes

806a567

With previous

18dda02

More moving stuff around

2b93b83

Moving more classes to Mvc Pipeline project

e66429e

Add build file

195b407

Move MVC skin detection to separate class

192a1fe

Added the controllers for the core website

ed9dfad

NewDDRMenu project which forks the original DDR menu project with cus…

0b39bb2

…tomizations for the MVC pipeline.

Rewire the way we detect MVC skins

23b6b13

Ability to replace the urlrewriter

825ae25

Changes that hard wired to ascx files for the skin

26a4165

Revert changes to core project

261a156

Merge branch 'release/10.0.0' into MVCPL

9ec0a3d

Rolling back mvc skin detection to what it was previously

44c90c7

Fixing routing

bcba2c6

Fix layout issue

6b1f317

Merge pull request #1 from donker/fix1

4700236

Fix layout issue

- avoid dublicate key error with standard DDRMENU

af884e5

- fix case

client dependecy for mvc (because it not exist in nuget)

78bcb42

Merge pull request #3 from sachatrauwaen/mvcppl1

419c044

client dependecy for mvc (because it not exist in nuget)

restrore use of clientdependency.core.mvc

5e3b2d4

add use of css variables

d17225a

add support for report-to

e903cda

update web.config for mvc

55308ab

port all what is needed to run from dnn9 version

e107bd4

first part of the html module : view control

c91f68a

Merge pull request #4 from sachatrauwaen/mvcpl2

09c4c04

Mvcpl2

use dnn10 default.css

4aae3e8

make edit in mvc work again

0fd3b47

sachatrauwaen and others added 21 commits January 29, 2026 23:18

Revise DNN MVC Pipeline User Guide

5ebe3a9

Updated user guide for MVC pipeline with corrections and clarifications.

Update documentation for Razor Module Development

b810b27

add partialskin helper with parameters

acb6441

fixes to guide

f207370

ensure the form is not shown

0924c7f

Merge branch 'dnnsoftware:feature/mvc-pipeline-old' into feature/mvc-…

23ddc6b

…pipeline-old

cleaup project file + apply style rules

bd2e392

Merge branch 'feature/mvc-pipeline-old' of https://github.com/sachatr…

86cb47e

…auwaen/Dnn.Platform into feature/mvc-pipeline-old

remove unused usings

a9dab74

remove unused usings

7d10f79

remove all logic from models

c262bf8

fix: add area to route

8d7ccdd

cleanup

5c9337b

cleanup + fix build

bff34c6

fix routing issues - redifine more specific route for the mvcpipeline

0c2e42f

fix csp separator and make it CLS compliant

ef94a5b

add CSP to mvc pipeline

9d42098

Merge branch 'develop' into feature/mvc-pipeline-old

3957da9

Merge branch 'develop' into feature/mvc-pipeline-old

421ccc3

Merge branch 'develop' into feature/mvc-pipeline-old

bac2fca

mitchelsellers reviewed Apr 4, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

First code submission for MVC Pipeline#6749

First code submission for MVC Pipeline#6749
donker wants to merge 248 commits intofeature/mvc-pipelinefrom
feature/mvc-pipeline-old

donker commented Oct 3, 2025 •

edited by sachatrauwaen

Loading

Uh oh!

sachatrauwaen commented Apr 2, 2026

Uh oh!

thienvc commented Apr 4, 2026

Uh oh!

mitchelsellers Apr 4, 2026

Uh oh!

mitchelsellers Apr 4, 2026

Uh oh!

mitchelsellers Apr 4, 2026

Uh oh!

mitchelsellers Apr 4, 2026

Uh oh!

mitchelsellers Apr 4, 2026

Uh oh!

mitchelsellers Apr 4, 2026

Uh oh!

mitchelsellers Apr 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Uh oh!

Conversation

donker commented Oct 3, 2025 • edited by sachatrauwaen Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Core Purpose

Key Features

Module Support:

Configuration:

New MVC Pipeline Projects

DotNetNuke.Web.MvcPipeline.csproj - New core MVC pipeline library

Razor Module Sample

NewDDRMenu

Core Platform Projects

DotNetNuke.Library

DotNetNuke.Abstractions

DotNetNuke.Web.Mvc

HTML

Dnn.PersonaBar.UI

DNNConnect.CKE

Skins/Aperture

Website

AdminExperiance

Documentation

Uh oh!

sachatrauwaen commented Apr 2, 2026

Uh oh!

thienvc commented Apr 4, 2026

Uh oh!

mitchelsellers Apr 4, 2026

Choose a reason for hiding this comment

Uh oh!

mitchelsellers Apr 4, 2026

Choose a reason for hiding this comment

Uh oh!

mitchelsellers Apr 4, 2026

Choose a reason for hiding this comment

Uh oh!

mitchelsellers Apr 4, 2026

Choose a reason for hiding this comment

Uh oh!

mitchelsellers Apr 4, 2026

Choose a reason for hiding this comment

Uh oh!

mitchelsellers Apr 4, 2026

Choose a reason for hiding this comment

Uh oh!

mitchelsellers Apr 4, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

donker commented Oct 3, 2025 •

edited by sachatrauwaen

Loading