kocoded/oqtane.framework
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #4006 from zyhfish/task/fix-issue-3977-new

Browse Source
This commit is contained in:
Shaun Walker 2024-03-17 14:08:23 -04:00 committed by GitHub
commit cd9f2ab232
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 69 additions and 50 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

47
Oqtane.Server/Repository/PageModuleRepository.cs
View File

@ -10,15 +10,15 @@ namespace Oqtane.Repository
{ {
public class PageModuleRepository : IPageModuleRepository public class PageModuleRepository : IPageModuleRepository
{ {
private TenantDBContext _db; private readonly IDbContextFactory<TenantDBContext> _dbContextFactory;
private readonly IModuleDefinitionRepository _moduleDefinitions; private readonly IModuleDefinitionRepository _moduleDefinitions;
private readonly IModuleRepository _modules; private readonly IModuleRepository _modules;
private readonly IPermissionRepository _permissions; private readonly IPermissionRepository _permissions;
private readonly ISettingRepository _settings; private readonly ISettingRepository _settings;
public PageModuleRepository(TenantDBContext context, IModuleDefinitionRepository moduleDefinitions, IModuleRepository modules, IPermissionRepository permissions, ISettingRepository settings) public PageModuleRepository(IDbContextFactory<TenantDBContext> dbContextFactory, IModuleDefinitionRepository moduleDefinitions, IModuleRepository modules, IPermissionRepository permissions, ISettingRepository settings)
{ {
_db = context; _dbContextFactory = dbContextFactory;
_moduleDefinitions = moduleDefinitions; _moduleDefinitions = moduleDefinitions;
_modules = modules; _modules = modules;
_permissions = permissions; _permissions = permissions;
@ -27,7 +27,9 @@ namespace Oqtane.Repository
public IEnumerable<PageModule> GetPageModules(int siteId) public IEnumerable<PageModule> GetPageModules(int siteId)
{ {
var pagemodules = _db.PageModule using(var db = _dbContextFactory.CreateDbContext())
{
var pagemodules = db.PageModule
.Include(item => item.Module) // eager load modules .Include(item => item.Module) // eager load modules
.Where(item => item.Module.SiteId == siteId).ToList(); .Where(item => item.Module.SiteId == siteId).ToList();
if (pagemodules.Any()) if (pagemodules.Any())
@ -42,19 +44,27 @@ namespace Oqtane.Repository
return pagemodules; return pagemodules;
} }
}
public PageModule AddPageModule(PageModule pageModule) public PageModule AddPageModule(PageModule pageModule)
{ {
_db.PageModule.Add(pageModule); using (var db = _dbContextFactory.CreateDbContext())
_db.SaveChanges(); {
db.PageModule.Add(pageModule);
db.SaveChanges();
return pageModule; return pageModule;
} }
}
public PageModule UpdatePageModule(PageModule pageModule) public PageModule UpdatePageModule(PageModule pageModule)
{ {
_db.Entry(pageModule).State = EntityState.Modified; using (var db = _dbContextFactory.CreateDbContext())
_db.SaveChanges(); {
db.Entry(pageModule).State = EntityState.Modified;
db.SaveChanges();
return pageModule; return pageModule;
} }
}
public PageModule GetPageModule(int pageModuleId) public PageModule GetPageModule(int pageModuleId)
{ {
@ -62,16 +72,18 @@ namespace Oqtane.Repository
} }
public PageModule GetPageModule(int pageModuleId, bool tracking) public PageModule GetPageModule(int pageModuleId, bool tracking)
{
using (var db = _dbContextFactory.CreateDbContext())
{ {
PageModule pagemodule; PageModule pagemodule;
if (tracking) if (tracking)
{ {
pagemodule = _db.PageModule.Include(item => item.Module) // eager load modules pagemodule = db.PageModule.Include(item => item.Module) // eager load modules
.FirstOrDefault(item => item.PageModuleId == pageModuleId); .FirstOrDefault(item => item.PageModuleId == pageModuleId);
} }
else else
{ {
pagemodule = _db.PageModule.AsNoTracking().Include(item => item.Module) // eager load modules pagemodule = db.PageModule.AsNoTracking().Include(item => item.Module) // eager load modules
.FirstOrDefault(item => item.PageModuleId == pageModuleId); .FirstOrDefault(item => item.PageModuleId == pageModuleId);
} }
if (pagemodule != null) if (pagemodule != null)
@ -82,10 +94,13 @@ namespace Oqtane.Repository
} }
return pagemodule; return pagemodule;
} }
}
public PageModule GetPageModule(int pageId, int moduleId) public PageModule GetPageModule(int pageId, int moduleId)
{ {
PageModule pagemodule = _db.PageModule.Include(item => item.Module) // eager load modules using (var db = _dbContextFactory.CreateDbContext())
{
PageModule pagemodule = db.PageModule.Include(item => item.Module) // eager load modules
.SingleOrDefault(item => item.PageId == pageId && item.ModuleId == moduleId); .SingleOrDefault(item => item.PageId == pageId && item.ModuleId == moduleId);
if (pagemodule != null) if (pagemodule != null)
{ {
@ -95,14 +110,17 @@ namespace Oqtane.Repository
} }
return pagemodule; return pagemodule;
} }
}
public void DeletePageModule(int pageModuleId) public void DeletePageModule(int pageModuleId)
{ {
PageModule pageModule = _db.PageModule.Include(item => item.Module) // eager load modules using (var db = _dbContextFactory.CreateDbContext())
{
PageModule pageModule = db.PageModule.Include(item => item.Module) // eager load modules
.SingleOrDefault(item => item.PageModuleId == pageModuleId); .SingleOrDefault(item => item.PageModuleId == pageModuleId);
_settings.DeleteSettings(EntityNames.PageModule, pageModuleId); _settings.DeleteSettings(EntityNames.PageModule, pageModuleId);
_db.PageModule.Remove(pageModule); db.PageModule.Remove(pageModule);
_db.SaveChanges(); db.SaveChanges();
// check if there are any remaining module instances in the site // check if there are any remaining module instances in the site
var pageModules = GetPageModules(pageModule.Module.SiteId); var pageModules = GetPageModules(pageModule.Module.SiteId);
@ -111,6 +129,7 @@ namespace Oqtane.Repository
_modules.DeleteModule(pageModule.ModuleId); _modules.DeleteModule(pageModule.ModuleId);
} }
} }
}
private PageModule GetPageModule(PageModule pageModule, List<ModuleDefinition> moduleDefinitions, List<Permission> modulePermissions) private PageModule GetPageModule(PageModule pageModule, List<ModuleDefinition> moduleDefinitions, List<Permission> modulePermissions)
{ {