Removed unused dependency on IResourceFactory from JsonApiResourceService

Author: Bart Koelman

docs/usage/extensibility/services.md

@@ -23,10 +23,9 @@ public class TodoItemService : JsonApiResourceService<TodoItem>
         ILoggerFactory loggerFactory,
         IJsonApiRequest request,
         IResourceChangeTracker<TodoItem> resourceChangeTracker,
-        IResourceFactory resourceFactory,
         IResourceHookExecutorFacade hookExecutor)
         : base(repositoryAccessor, queryLayerComposer, paginationContext, options, loggerFactory,
-            request, resourceChangeTracker, resourceFactory, hookExecutor)
+            request, resourceChangeTracker, hookExecutor)
     {
         _notificationService = notificationService;
     }

src/Examples/JsonApiDotNetCoreExample/Services/CustomArticleService.cs

@@ -22,10 +22,9 @@ public CustomArticleService(
             ILoggerFactory loggerFactory,
             IJsonApiRequest request,
             IResourceChangeTracker<Article> resourceChangeTracker,
-            IResourceFactory resourceFactory,
             IResourceHookExecutorFacade hookExecutor)
             : base(repositoryAccessor, queryLayerComposer, paginationContext, options, loggerFactory, request,
-                resourceChangeTracker, resourceFactory, hookExecutor)
+                resourceChangeTracker, hookExecutor)
         { }
 
         public override async Task<Article> GetAsync(int id, CancellationToken cancellationToken)

src/JsonApiDotNetCore/Services/JsonApiResourceService.cs

@@ -30,7 +30,6 @@ public class JsonApiResourceService<TResource, TId> :
         private readonly TraceLogWriter<JsonApiResourceService<TResource, TId>> _traceWriter;
         private readonly IJsonApiRequest _request;
         private readonly IResourceChangeTracker<TResource> _resourceChangeTracker;
-        private readonly IResourceFactory _resourceFactory;
         private readonly IResourceHookExecutorFacade _hookExecutor;
 
         public JsonApiResourceService(
@@ -41,7 +40,6 @@ public JsonApiResourceService(
             ILoggerFactory loggerFactory,
             IJsonApiRequest request,
             IResourceChangeTracker<TResource> resourceChangeTracker,
-            IResourceFactory resourceFactory,
             IResourceHookExecutorFacade hookExecutor)
         {
             if (loggerFactory == null) throw new ArgumentNullException(nameof(loggerFactory));
@@ -53,7 +51,6 @@ public JsonApiResourceService(
             _traceWriter = new TraceLogWriter<JsonApiResourceService<TResource, TId>>(loggerFactory);
             _request = request ?? throw new ArgumentNullException(nameof(request));
             _resourceChangeTracker = resourceChangeTracker ?? throw new ArgumentNullException(nameof(resourceChangeTracker));
-            _resourceFactory = resourceFactory ?? throw new ArgumentNullException(nameof(resourceFactory));
             _hookExecutor = hookExecutor ?? throw new ArgumentNullException(nameof(hookExecutor));
         }
 
@@ -469,10 +466,9 @@ public JsonApiResourceService(
             ILoggerFactory loggerFactory,
             IJsonApiRequest request,
             IResourceChangeTracker<TResource> resourceChangeTracker,
-            IResourceFactory resourceFactory,
             IResourceHookExecutorFacade hookExecutor)
             : base(repositoryAccessor, queryLayerComposer, paginationContext, options, loggerFactory, request,
-                resourceChangeTracker, resourceFactory, hookExecutor)
+                resourceChangeTracker, hookExecutor)
         {
         }
     }

test/DiscoveryTests/ServiceDiscoveryFacadeTests.cs

@@ -155,10 +155,9 @@ public TestModelService(
                 ILoggerFactory loggerFactory,
                 IJsonApiRequest request,
                 IResourceChangeTracker<TestModel> resourceChangeTracker,
-                IResourceFactory resourceFactory,
                 IResourceHookExecutorFacade hookExecutor)
                 : base(repositoryAccessor, queryLayerComposer, paginationContext, options, loggerFactory, request,
-                    resourceChangeTracker, resourceFactory, hookExecutor)
+                    resourceChangeTracker, hookExecutor)
             {
             }
         }