OrchardCore.Cms.Web, Mvc.Web and Templates with .NET 6

[agriffard]

Use only Program.cs

Options in csproj:

    <Nullable>enable</Nullable>
    <ImplicitUsings>enable</ImplicitUsings>

Docs updated.
Templates updated.

Fixes #11932

[agriffard]

1. I figured out that it would only work with .NET 6.0 so we could not support .net5.0 and 3.1 as currently, right?

2. Launching it using this line, with VS and IIS Express:
   builder.WebHost.UseNLogWeb();

seems to cause an error:

NotSupportedException: The web root changed from "D:\Orchard\OrchardCore\src\OrchardCore.Cms.Web\wwwroot" to "D:\Orchard\OrchardCore\src\OrchardCore.Cms.Web\". Changing the host configuration using WebApplicationBuilder.WebHost is not supported. Use WebApplication.CreateBuilder(WebApplicationOptions) instead.

[Skrypt]

We can't have this because of multi-targeting. Same issue as on OrchardCore.Commerce recently.

[agriffard]

Example repo to repro the error using builder.WebHost.UseNLogWeb(); : https://github.com/agriffard/Cms.Web

[agriffard]

@deanmarcussen or @Skrypt Can you please have a look to the NLog error?

[snakefoot]

WebHost is legacy and now obsolete with NET6. Instead use Host:

builder.Host.UseNLogWeb();

And change from IWebHostBuilder to IHostBuilder:

    public static class WebHostBuilderExtensions
    {
        public static IHostBuilder UseNLogWeb(this IHostBuilder builder)
        {
            LayoutRenderer.Register<TenantLayoutRenderer>(TenantLayoutRenderer.LayoutRendererName);
            builder.UseNLog();
            builder.ConfigureAppConfiguration((context, configuration) =>
            {
                var environment = context.HostingEnvironment;
                NLog.LogManager.Setup()
                      .LoadConfigurationFromFile(System.IO.Path.Combine(environment.ContentRootPath, "NLog.config"))
                      .LoadConfiguration(builder => builder.Configuration.Variables["configDir"] = environment.ContentRootPath);
            });

            return builder;
        }
    }

Notice NLog.Web.AspNetCore ver. 5.0 will automatically attempt to load NLog.config from ContentRootPath. And one can use ${aspnet-appbasepath} as an alternative to ${var:configDir}

[deanmarcussen]

@snakefoot thanks for the assist.

@agriffard this needs the same for the Mvc templates / program.cs / startup.cs

You could take this out of draft :)

[agriffard]

Mvc sample and template updated.

[agriffard]

Could be merged, now that we use .NET 6.

[Skrypt]

I'm ok with it if we merge we will have time to adjust if any breaking change is introduced.

[sebastienros]

Needs testing around razor compilation with implicit using. Niraj mentions there is an issue with that.

[ns8482e]

@agriffard @Skrypt Updated Program.cs and fix merge conflict

[ns8482e]

@agriffard @Skrypt @sebastienros @deanmarcussen @hishamco Can you please review the updates?.

[ns8482e]

Needs testing around razor compilation with implicit using. Niraj mentions there is an issue with that.

@sebastienros for OrchardCore.Web conditional enable implicit using based on RazorCompilation value.
For templates removed Razor compilation