Skip to content

Commit

Permalink
Merge pull request #2601 from leigh-pointer/ModDefSettingsDelete
Browse files Browse the repository at this point in the history
Fix for Missing Delete ModuleDefinition settings #1966
  • Loading branch information
sbwalker authored Feb 17, 2023
2 parents b3f7353 + 52300e6 commit d1d00e6
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion Oqtane.Server/Repository/ModuleDefinitionRepository.cs
Original file line number Diff line number Diff line change
Expand Up @@ -18,12 +18,14 @@ public class ModuleDefinitionRepository : IModuleDefinitionRepository
private MasterDBContext _db;
private readonly IMemoryCache _cache;
private readonly IPermissionRepository _permissions;
private readonly ISettingRepository _settings;

public ModuleDefinitionRepository(MasterDBContext context, IMemoryCache cache, IPermissionRepository permissions)
public ModuleDefinitionRepository(MasterDBContext context, IMemoryCache cache, IPermissionRepository permissions, ISettingRepository settings)
{
_db = context;
_cache = cache;
_permissions = permissions;
_settings = settings;
}

public IEnumerable<ModuleDefinition> GetModuleDefinitions()
Expand Down Expand Up @@ -52,6 +54,7 @@ public void UpdateModuleDefinition(ModuleDefinition moduleDefinition)
public void DeleteModuleDefinition(int moduleDefinitionId)
{
ModuleDefinition moduleDefinition = _db.ModuleDefinition.Find(moduleDefinitionId);
_settings.DeleteSettings(EntityNames.ModuleDefinition, moduleDefinitionId);
_db.ModuleDefinition.Remove(moduleDefinition);
_db.SaveChanges();
_cache.Remove("moduledefinitions");
Expand Down

0 comments on commit d1d00e6

Please sign in to comment.