yv01p/Umbraco-CMS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove public ILogger from PluginController

Browse Source 
Having a public ILogger doesn't align with MS ILogger, and the logger is never used
This commit is contained in:
Mole
2020-09-21 11:46:29 +02:00
parent a12ea89517
commit 581775e08c
12 changed files with 22 additions and 35 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/Umbraco.Web.Common/Controllers/PluginController.cs
View File

@@ -49,11 +49,6 @@ namespace Umbraco.Web.Common.Controllers
/// </summary>
public AppCaches AppCaches { get; }
/// <summary>
/// Gets or sets the logger.
/// </summary>
public ILogger Logger { get; }
/// <summary>
/// Gets or sets the profiling logger.
/// </summary>
@@ -64,13 +59,12 @@ namespace Umbraco.Web.Common.Controllers
/// </summary>
internal PluginControllerMetadata Metadata => GetMetadata(GetType());
protected PluginController(IUmbracoContextAccessor umbracoContextAccessor, IUmbracoDatabaseFactory databaseFactory, ServiceContext services, AppCaches appCaches, ILogger logger, IProfilingLogger profilingLogger)
protected PluginController(IUmbracoContextAccessor umbracoContextAccessor, IUmbracoDatabaseFactory databaseFactory, ServiceContext services, AppCaches appCaches, IProfilingLogger profilingLogger)
{
UmbracoContextAccessor = umbracoContextAccessor;
DatabaseFactory = databaseFactory;
Services = services;
AppCaches = appCaches;
Logger = logger;
ProfilingLogger = profilingLogger;
}