Merge pull request #2582 from sbwalker/dev
fix #2574 - check for null ModuleDefinition reference when loding permissions in PageModuleRepository (credit @beolafsen)
This commit is contained in:
commit
af55c11aa0
@ -104,7 +104,7 @@ namespace Oqtane.Repository
|
|||||||
|
|
||||||
// moduledefinition permissionnames can specify permissions for other entities (ie. API permissions)
|
// moduledefinition permissionnames can specify permissions for other entities (ie. API permissions)
|
||||||
pageModule.Module.ModuleDefinition = moduleDefinitions.Find(item => item.ModuleDefinitionName == pageModule.Module.ModuleDefinitionName);
|
pageModule.Module.ModuleDefinition = moduleDefinitions.Find(item => item.ModuleDefinitionName == pageModule.Module.ModuleDefinitionName);
|
||||||
if (!string.IsNullOrEmpty(pageModule.Module.ModuleDefinition.PermissionNames) && pageModule.Module.ModuleDefinition.PermissionNames.Contains(":"))
|
if (pageModule.Module.ModuleDefinition != null && !string.IsNullOrEmpty(pageModule.Module.ModuleDefinition.PermissionNames) && pageModule.Module.ModuleDefinition.PermissionNames.Contains(":"))
|
||||||
{
|
{
|
||||||
foreach (var permissionname in pageModule.Module.ModuleDefinition.PermissionNames.Split(",", System.StringSplitOptions.RemoveEmptyEntries))
|
foreach (var permissionname in pageModule.Module.ModuleDefinition.PermissionNames.Split(",", System.StringSplitOptions.RemoveEmptyEntries))
|
||||||
{
|
{
|
||||||
|
Reference in New Issue
Block a user