From 40193db94f587f84b85e04df96a3047e3677f7b8 Mon Sep 17 00:00:00 2001
From: Samuele Locatelli <samuele@steamware.net>
Date: Thu, 19 Mar 2026 14:36:03 +0100
Subject: [PATCH] Semplificazione init Repo/Servizi x DI

---
 .../DataServiceCollectionExtensions.cs        |  95 +++++++++++++++
 Lux.API/Program.cs                            |  96 +---------------
 Lux.UI/Program.cs                             | 108 +-----------------
 3 files changed, 103 insertions(+), 196 deletions(-)
 create mode 100644 EgwCoreLib.Lux.Data/DataServiceCollectionExtensions.cs

diff --git a/EgwCoreLib.Lux.Data/DataServiceCollectionExtensions.cs b/EgwCoreLib.Lux.Data/DataServiceCollectionExtensions.cs
new file mode 100644
index 0000000..574d898
--- /dev/null
+++ b/EgwCoreLib.Lux.Data/DataServiceCollectionExtensions.cs
@@ -0,0 +1,95 @@
+using EgwCoreLib.Lux.Data.Repository.Config;
+using EgwCoreLib.Lux.Data.Repository.Cost;
+using EgwCoreLib.Lux.Data.Repository.Items;
+using EgwCoreLib.Lux.Data.Repository.Job;
+using EgwCoreLib.Lux.Data.Repository.Production;
+using EgwCoreLib.Lux.Data.Repository.Sales;
+using EgwCoreLib.Lux.Data.Repository.Utils;
+using EgwCoreLib.Lux.Data.Services;
+using EgwCoreLib.Lux.Data.Services.Config;
+using EgwCoreLib.Lux.Data.Services.Cost;
+using EgwCoreLib.Lux.Data.Services.Items;
+using EgwCoreLib.Lux.Data.Services.Job;
+using EgwCoreLib.Lux.Data.Services.Production;
+using EgwCoreLib.Lux.Data.Services.Sales;
+using EgwCoreLib.Lux.Data.Services.Utils;
+using Microsoft.Extensions.DependencyInjection;
+using Microsoft.Extensions.DependencyInjection.Extensions;
+
+namespace EgwCoreLib.Lux.Data
+{
+    public static class DataServiceCollectionExtensions
+    {
+        public static IServiceCollection AddLuxData(this IServiceCollection services, string connectionString)
+        {
+            //// DbContextFactory: preferibile in Blazor Server e scenari concorrenti
+            //services.AddDbContextFactory<DataLayerContext>(options =>
+            //    options.UseMySql(connectionString, ServerVersion.AutoDetect(connectionString)));
+
+            // Repository con interfaccia
+            services.TryAddScoped<IConfGlassRepository, ConfGlassRepository>();
+            services.TryAddScoped<IConfProfileRepository, ConfProfileRepository>();
+            services.TryAddScoped<IConfWoodRepository, ConfWoodRepository>();
+            services.TryAddScoped<ICostDriverRepository, CostDriverRepository>();
+            services.TryAddScoped<ICustomerRepository, CustomerRepository>();
+            services.TryAddScoped<IDealerRepository, DealerRepository>();
+            services.TryAddScoped<IEnvirParamRepository, EnvirParamRepository>();
+            services.TryAddScoped<IGenClassRepository, GenClassRepository>();
+            services.TryAddScoped<IGenValRepository, GenValRepository>();
+            services.TryAddScoped<IItemGroupRepository, ItemGroupRepository>();
+            services.TryAddScoped<IItemRepository, ItemRepository>();
+            services.TryAddScoped<IJobStepRepository, JobStepRepository>();
+            services.TryAddScoped<IJobTaskRepository, JobTaskRepository>();
+            services.TryAddScoped<IOfferRepository, OfferRepository>();
+            services.TryAddScoped<IOfferRowRepository, OfferRowRepository>();
+            services.TryAddScoped<IOrderRepository, OrderRepository>();
+            services.TryAddScoped<IOrderRowRepository, OrderRowRepository>();
+            services.TryAddScoped<IProductionBatchRepository, ProductionBatchRepository>();
+            services.TryAddScoped<IProductionGroupRepository, ProductionGroupRepository>();
+            services.TryAddScoped<IProductionItemRepository, ProductionItemRepository>();
+            services.TryAddScoped<IProductionOdlRepository, ProductionOdlRepository>();
+            services.TryAddScoped<IProductionPlantRepository, ProductionPlantRepository>();
+            services.TryAddScoped<IResourceRepository, ResourceRepository>();
+            services.TryAddScoped<ISellingItemRepository, SellingItemRepository>();
+            services.TryAddScoped<ITagRepository, TagRepository>();
+            services.TryAddScoped<ITemplateRepository, TemplateRepository>();
+            services.TryAddScoped<ITemplateRowRepository, TemplateRowRepository>();
+
+            // Servizi dati con interfaccia
+            services.TryAddScoped<IConfGlassService, ConfGlassService>();
+            services.TryAddScoped<IConfProfileService, ConfProfileService>();
+            services.TryAddScoped<IConfWoodService, ConfWoodService>();
+            services.TryAddScoped<ICostDriverService, CostDriverService>();
+            services.TryAddScoped<ICustomerService, CustomerService>();
+            services.TryAddScoped<IDealerService, DealerService>();
+            services.TryAddScoped<IEnvirParamService, EnvirParamService>();
+            services.TryAddScoped<IGenClassService, GenClassService>();
+            services.TryAddScoped<IGenValService, GenValService>();
+            services.TryAddScoped<IItemService, ItemService>();
+            services.TryAddScoped<IItemGroupService, ItemGroupService>();
+            services.TryAddScoped<IJobStepService, JobStepService>();
+            services.TryAddScoped<IJobTaskService, JobTaskService>();
+            services.TryAddScoped<IOfferService, OfferService>();
+            services.TryAddScoped<IOfferRowService, OfferRowService>();
+            services.TryAddScoped<IOrderService, OrderService>();
+            services.TryAddScoped<IOrderRowService, OrderRowService>();
+            services.TryAddScoped<IProductionGroupService, ProductionGroupService>();
+            services.TryAddScoped<IProductionItemService, ProductionItemService>();
+            services.TryAddScoped<IProductionBatchService, ProductionBatchService>();
+            services.TryAddScoped<IProductionOdlService, ProductionOdlService>();
+            services.TryAddScoped<IProductionPlantService, ProductionPlantService>();
+            services.TryAddScoped<IResourceService, ResourceService>();
+            services.TryAddScoped<ISellingItemService, SellingItemService>();
+            services.TryAddScoped<ITagService, TagService>();
+            services.TryAddScoped<ITemplateService, TemplateService>();
+            services.TryAddScoped<ITemplateRowService, TemplateRowService>();
+
+            // Facade / DataLayerService
+            //builder.Services.AddSingleton<DataLayerServices>();
+            services.TryAddScoped<DataLayerServices>();
+            //services.TryAddScoped<IDataLayerServices, DataLayerServices>();
+
+            return services;
+        }
+    }
+}
diff --git a/Lux.API/Program.cs b/Lux.API/Program.cs
index 50ec9dd..3888bb2 100644
--- a/Lux.API/Program.cs
+++ b/Lux.API/Program.cs
@@ -1,19 +1,5 @@
 using EgwCoreLib.Lux.Data;
-using EgwCoreLib.Lux.Data.Repository.Config;
-using EgwCoreLib.Lux.Data.Repository.Cost;
-using EgwCoreLib.Lux.Data.Repository.Items;
-using EgwCoreLib.Lux.Data.Repository.Job;
-using EgwCoreLib.Lux.Data.Repository.Production;
-using EgwCoreLib.Lux.Data.Repository.Sales;
-using EgwCoreLib.Lux.Data.Repository.Utils;
 using EgwCoreLib.Lux.Data.Services;
-using EgwCoreLib.Lux.Data.Services.Config;
-using EgwCoreLib.Lux.Data.Services.Cost;
-using EgwCoreLib.Lux.Data.Services.Items;
-using EgwCoreLib.Lux.Data.Services.Job;
-using EgwCoreLib.Lux.Data.Services.Production;
-using EgwCoreLib.Lux.Data.Services.Sales;
-using EgwCoreLib.Lux.Data.Services.Utils;
 using Lux.API.Services;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.OpenApi.Models;
@@ -169,86 +155,8 @@ builder.Services.AddDbContextFactory<DataLayerContext>(options =>
         .LogTo(_ => { }); // disabilita EF logging;
 });
 
-// servizi Scoped (usano DB)
-
-//builder.Services.AddSingleton<DataLayerServices>();
-builder.Services.AddScoped<DataLayerServices>();
-
-// Repository con interfaccia
-builder.Services.AddScoped<IConfGlassRepository, ConfGlassRepository>();
-builder.Services.AddScoped<IConfProfileRepository, ConfProfileRepository>();
-builder.Services.AddScoped<IConfWoodRepository, ConfWoodRepository>();
-builder.Services.AddScoped<ICostDriverRepository, CostDriverRepository>();
-builder.Services.AddScoped<ICustomerRepository, CustomerRepository>();
-builder.Services.AddScoped<IDealerRepository, DealerRepository>();
-builder.Services.AddScoped<IEnvirParamRepository, EnvirParamRepository>();
-builder.Services.AddScoped<IGenClassRepository, GenClassRepository>();
-builder.Services.AddScoped<IGenValRepository, GenValRepository>();
-builder.Services.AddScoped<IItemGroupRepository, ItemGroupRepository>();
-builder.Services.AddScoped<IItemRepository, ItemRepository>();
-builder.Services.AddScoped<IJobStepRepository, JobStepRepository>();
-builder.Services.AddScoped<IJobTaskRepository, JobTaskRepository>();
-builder.Services.AddScoped<IOfferRepository, OfferRepository>();
-builder.Services.AddScoped<IOfferRowRepository, OfferRowRepository>();
-builder.Services.AddScoped<IOrderRepository, OrderRepository>();
-builder.Services.AddScoped<IOrderRowRepository, OrderRowRepository>();
-builder.Services.AddScoped<IProductionBatchRepository, ProductionBatchRepository>();
-builder.Services.AddScoped<IProductionGroupRepository, ProductionGroupRepository>();
-builder.Services.AddScoped<IProductionItemRepository, ProductionItemRepository>();
-builder.Services.AddScoped<IProductionOdlRepository, ProductionOdlRepository>();
-builder.Services.AddScoped<IProductionPlantRepository, ProductionPlantRepository>();
-builder.Services.AddScoped<IResourceRepository, ResourceRepository>();
-builder.Services.AddScoped<ISellingItemRepository, SellingItemRepository>();
-builder.Services.AddScoped<ITagRepository, TagRepository>();
-builder.Services.AddScoped<ITemplateRepository, TemplateRepository>();
-builder.Services.AddScoped<ITemplateRowRepository, TemplateRowRepository>();
-
-// Servizi dati con interfaccia
-builder.Services.AddScoped<IConfGlassService, ConfGlassService>();
-builder.Services.AddScoped<IConfProfileService, ConfProfileService>();
-builder.Services.AddScoped<IConfWoodService, ConfWoodService>();
-builder.Services.AddScoped<ICostDriverService, CostDriverService>();
-builder.Services.AddScoped<ICustomerService, CustomerService>();
-builder.Services.AddScoped<IDealerService, DealerService>();
-builder.Services.AddScoped<IEnvirParamService, EnvirParamService>();
-builder.Services.AddScoped<IGenClassService, GenClassService>();
-builder.Services.AddScoped<IGenValService, GenValService>();
-builder.Services.AddScoped<IItemService, ItemService>();
-builder.Services.AddScoped<IItemGroupService, ItemGroupService>();
-builder.Services.AddScoped<IJobStepService, JobStepService>();
-builder.Services.AddScoped<IJobTaskService, JobTaskService>();
-builder.Services.AddScoped<IOfferService, OfferService>();
-builder.Services.AddScoped<IOfferRowService, OfferRowService>();
-builder.Services.AddScoped<IOrderService, OrderService>();
-builder.Services.AddScoped<IOrderRowService, OrderRowService>();
-builder.Services.AddScoped<IProductionGroupService, ProductionGroupService>();
-builder.Services.AddScoped<IProductionItemService, ProductionItemService>();
-builder.Services.AddScoped<IProductionBatchService, ProductionBatchService>();
-builder.Services.AddScoped<IProductionOdlService, ProductionOdlService>();
-builder.Services.AddScoped<IProductionPlantService, ProductionPlantService>();
-builder.Services.AddScoped<IResourceService, ResourceService>();
-builder.Services.AddScoped<ISellingItemService, SellingItemService>();
-builder.Services.AddScoped<ITagService, TagService>();
-builder.Services.AddScoped<ITemplateService, TemplateService>();
-builder.Services.AddScoped<ITemplateRowService, TemplateRowService>();
-
-// Da rivedere!!!
-#if false
-// registrazione automatica Repository e Servizi con Scrutor
-builder.Services.Scan(scan => scan
-    .FromAssemblyOf<BaseServ>() // Cerca nell'assembly dove si trova BaseServ
-
-    // Registra tutti i Repository
-    .AddClasses(classes => classes.Where(c => c.Name.EndsWith("Repository")))
-    .AsImplementedInterfaces() // Es: associa GenClassRepository a IGenClassRepository
-    .WithScopedLifetime()
-
-    // Registra tutti i Servizi
-    .AddClasses(classes => classes.Where(c => c.Name.EndsWith("Service")))
-    .AsImplementedInterfaces()
-    .WithScopedLifetime()
-); 
-#endif
+// registrazione in blocco servizi con metodo extension custom
+builder.Services.AddLuxData(connectionString);
 
 // servizi Singleton (no DB)
 builder.Services.AddSingleton<ImageCacheService>();
diff --git a/Lux.UI/Program.cs b/Lux.UI/Program.cs
index a87b0ad..f355edf 100644
--- a/Lux.UI/Program.cs
+++ b/Lux.UI/Program.cs
@@ -1,19 +1,5 @@
 using EgwCoreLib.Lux.Data;
-using EgwCoreLib.Lux.Data.Repository.Config;
-using EgwCoreLib.Lux.Data.Repository.Cost;
-using EgwCoreLib.Lux.Data.Repository.Items;
-using EgwCoreLib.Lux.Data.Repository.Job;
-using EgwCoreLib.Lux.Data.Repository.Production;
-using EgwCoreLib.Lux.Data.Repository.Sales;
-using EgwCoreLib.Lux.Data.Repository.Utils;
 using EgwCoreLib.Lux.Data.Services;
-using EgwCoreLib.Lux.Data.Services.Config;
-using EgwCoreLib.Lux.Data.Services.Cost;
-using EgwCoreLib.Lux.Data.Services.Items;
-using EgwCoreLib.Lux.Data.Services.Job;
-using EgwCoreLib.Lux.Data.Services.Production;
-using EgwCoreLib.Lux.Data.Services.Sales;
-using EgwCoreLib.Lux.Data.Services.Utils;
 using Lux.UI.Components;
 using Lux.UI.Components.Account;
 using Lux.UI.Data;
@@ -154,27 +140,23 @@ builder.Services.AddAuthentication(options =>
     })
     .AddIdentityCookies();
 
-var connectionString = builder.Configuration.GetConnectionString("DefaultConnection") ?? throw new InvalidOperationException("Connection string 'DefaultConnection' not found.");
-#if false
-builder.Services.AddDbContext<ApplicationDbContext>(options =>
-options.UseSqlServer(connectionString)); 
-#endif
+var conn = builder.Configuration.GetConnectionString("DefaultConnection") ?? throw new InvalidOperationException("Connection string 'DefaultConnection' not found.");
 
 // ApplicationDbContext (già presente)
 builder.Services.AddDbContext<ApplicationDbContext>(options =>
 {
     options.UseMySql(
-        builder.Configuration.GetConnectionString("DefaultConnection"),
-        ServerVersion.AutoDetect(builder.Configuration.GetConnectionString("DefaultConnection")))
+        conn,
+        ServerVersion.AutoDetect(conn))
         .EnableSensitiveDataLogging(false)
         .EnableDetailedErrors(true);
     //.LogTo(_ => { }); // disabilita EF logging
 });
 
 // DataLayerContext (manca!)
+conn = builder.Configuration.GetConnectionString("Lux.All") ?? "";
 builder.Services.AddDbContextFactory<DataLayerContext>(options =>
 {
-    var conn = builder.Configuration.GetConnectionString("Lux.All");
     options.UseMySql(conn, ServerVersion.AutoDetect(conn), mySqlOptions =>
     {
         mySqlOptions.EnableStringComparisonTranslations();
@@ -184,9 +166,6 @@ builder.Services.AddDbContextFactory<DataLayerContext>(options =>
     //.LogTo(_ => { }, Microsoft.Extensions.Logging.LogLevel.None); // disabilita EF logging
 });
 
-
-
-
 builder.Services.AddDatabaseDeveloperPageExceptionFilter();
 
 builder.Services.AddIdentityCore<ApplicationUser>(options => options.SignIn.RequireConfirmedAccount = true)
@@ -209,84 +188,9 @@ builder.Services.AddSingleton<CalcRequestService>();
 builder.Services.AddSingleton<ConfigDataService>();
 builder.Services.AddSingleton<ProdService>();
 
-//builder.Services.AddSingleton<DataLayerServices>();
-builder.Services.AddScoped<DataLayerServices>();
 
-// Repository con interfaccia
-builder.Services.AddScoped<IConfGlassRepository, ConfGlassRepository>();
-builder.Services.AddScoped<IConfProfileRepository, ConfProfileRepository>();
-builder.Services.AddScoped<IConfWoodRepository, ConfWoodRepository>();
-builder.Services.AddScoped<ICostDriverRepository, CostDriverRepository>();
-builder.Services.AddScoped<ICustomerRepository, CustomerRepository>();
-builder.Services.AddScoped<IDealerRepository, DealerRepository>();
-builder.Services.AddScoped<IEnvirParamRepository, EnvirParamRepository>();
-builder.Services.AddScoped<IGenClassRepository, GenClassRepository>();
-builder.Services.AddScoped<IGenValRepository, GenValRepository>();
-builder.Services.AddScoped<IItemGroupRepository, ItemGroupRepository>();
-builder.Services.AddScoped<IItemRepository, ItemRepository>();
-builder.Services.AddScoped<IJobStepRepository, JobStepRepository>();
-builder.Services.AddScoped<IJobTaskRepository, JobTaskRepository>();
-builder.Services.AddScoped<IOfferRepository, OfferRepository>();
-builder.Services.AddScoped<IOfferRowRepository, OfferRowRepository>();
-builder.Services.AddScoped<IOrderRepository, OrderRepository>();
-builder.Services.AddScoped<IOrderRowRepository, OrderRowRepository>();
-builder.Services.AddScoped<IProductionBatchRepository, ProductionBatchRepository>();
-builder.Services.AddScoped<IProductionGroupRepository, ProductionGroupRepository>();
-builder.Services.AddScoped<IProductionItemRepository, ProductionItemRepository>();
-builder.Services.AddScoped<IProductionOdlRepository, ProductionOdlRepository>();
-builder.Services.AddScoped<IProductionPlantRepository, ProductionPlantRepository>();
-builder.Services.AddScoped<IResourceRepository, ResourceRepository>();
-builder.Services.AddScoped<ISellingItemRepository, SellingItemRepository>();
-builder.Services.AddScoped<ITagRepository, TagRepository>();
-builder.Services.AddScoped<ITemplateRepository, TemplateRepository>();
-builder.Services.AddScoped<ITemplateRowRepository, TemplateRowRepository>();
-
-// Servizi dati con interfaccia
-builder.Services.AddScoped<IConfGlassService, ConfGlassService>();
-builder.Services.AddScoped<IConfProfileService, ConfProfileService>();
-builder.Services.AddScoped<IConfWoodService, ConfWoodService>();
-builder.Services.AddScoped<ICostDriverService, CostDriverService>();
-builder.Services.AddScoped<ICustomerService, CustomerService>();
-builder.Services.AddScoped<IDealerService, DealerService>();
-builder.Services.AddScoped<IEnvirParamService, EnvirParamService>();
-builder.Services.AddScoped<IGenClassService, GenClassService>();
-builder.Services.AddScoped<IGenValService, GenValService>();
-builder.Services.AddScoped<IItemService, ItemService>();
-builder.Services.AddScoped<IItemGroupService, ItemGroupService>();
-builder.Services.AddScoped<IJobStepService, JobStepService>();
-builder.Services.AddScoped<IJobTaskService, JobTaskService>();
-builder.Services.AddScoped<IOfferService, OfferService>();
-builder.Services.AddScoped<IOfferRowService, OfferRowService>();
-builder.Services.AddScoped<IOrderService, OrderService>();
-builder.Services.AddScoped<IOrderRowService, OrderRowService>();
-builder.Services.AddScoped<IProductionGroupService, ProductionGroupService>();
-builder.Services.AddScoped<IProductionItemService, ProductionItemService>();
-builder.Services.AddScoped<IProductionBatchService, ProductionBatchService>();
-builder.Services.AddScoped<IProductionOdlService, ProductionOdlService>();
-builder.Services.AddScoped<IProductionPlantService, ProductionPlantService>();
-builder.Services.AddScoped<IResourceService, ResourceService>();
-builder.Services.AddScoped<ISellingItemService, SellingItemService>();
-builder.Services.AddScoped<ITagService, TagService>();
-builder.Services.AddScoped<ITemplateService, TemplateService>();
-builder.Services.AddScoped<ITemplateRowService, TemplateRowService>();
-
-// Da rivedere!!!
-#if false
-// registrazione automatica Service e Servizi con Scrutor
-builder.Services.Scan(scan => scan
-    .FromAssemblyOf<BaseServ>() // Cerca nell'assembly dove si trova BaseServ
-
-    // Registra tutti i Repository
-    .AddClasses(classes => classes.Where(c => c.Name.EndsWith("Repository")))
-    .AsImplementedInterfaces() // Es: associa GenClassRepository a IGenClassRepository
-    .WithScopedLifetime()
-
-    // Registra tutti i Servizi
-    .AddClasses(classes => classes.Where(c => c.Name.EndsWith("Service")))
-    .AsImplementedInterfaces()
-    .WithScopedLifetime()
-); 
-#endif
+// registrazione in blocco servizi con metodo extension custom
+builder.Services.AddLuxData(conn);
 
 // init servizio gestone ReqIndex
 string cleanupDayTTL = configuration.GetValue<string>("ServerConf:CleanupDayTTL") ?? "360";