diff --git a/Oqtane.Client/Modules/Admin/Error/Index.razor b/Oqtane.Client/Modules/Admin/Error/Index.razor index 9e0df574..7ddb0ce3 100644 --- a/Oqtane.Client/Modules/Admin/Error/Index.razor +++ b/Oqtane.Client/Modules/Admin/Error/Index.razor @@ -8,7 +8,7 @@ protected override async Task OnInitializedAsync() { Module module = await ModuleService.GetModuleAsync(ModuleState.ModuleId); - if (UserSecurity.IsAuthorized(PageState.User, Constants.HostRole)) + if (UserSecurity.IsAuthorized(PageState.User, RoleNames.Host)) { string message = "A Problem Was Encountered Loading Module " + module.ModuleDefinitionName; AddModuleMessage(message, MessageType.Error); diff --git a/Oqtane.Client/Modules/Admin/Roles/Users.razor b/Oqtane.Client/Modules/Admin/Roles/Users.razor index c292d3d5..a583aedc 100644 --- a/Oqtane.Client/Modules/Admin/Roles/Users.razor +++ b/Oqtane.Client/Modules/Admin/Roles/Users.razor @@ -88,7 +88,7 @@ else Role role = await RoleService.GetRoleAsync(roleid); name = role.Name; users = await UserRoleService.GetUserRolesAsync(PageState.Site.SiteId); - users = users.Where(item => item.Role.Name == Constants.RegisteredRole).ToList(); + users = users.Where(item => item.Role.Name == RoleNames.Registered).ToList(); await GetUserRoles(); } catch (Exception ex) diff --git a/Oqtane.Client/Modules/Admin/UserProfile/Index.razor b/Oqtane.Client/Modules/Admin/UserProfile/Index.razor index 9c718ef1..30e6b075 100644 --- a/Oqtane.Client/Modules/Admin/UserProfile/Index.razor +++ b/Oqtane.Client/Modules/Admin/UserProfile/Index.razor @@ -79,7 +79,7 @@ else @foreach (Profile profile in profiles) { var p = profile; - if (!p.IsPrivate || UserSecurity.IsAuthorized(PageState.User, Constants.AdminRole)) + if (!p.IsPrivate || UserSecurity.IsAuthorized(PageState.User, RoleNames.Admin)) { if (p.Category != category) { @@ -299,7 +299,7 @@ else { settings = SettingService.SetSetting(settings, profile.Name, profile.DefaultValue); } - if (!profile.IsPrivate || UserSecurity.IsAuthorized(PageState.User, Constants.AdminRole)) + if (!profile.IsPrivate || UserSecurity.IsAuthorized(PageState.User, RoleNames.Admin)) { if (profile.IsRequired && string.IsNullOrEmpty(SettingService.GetSetting(settings, profile.Name, string.Empty))) { diff --git a/Oqtane.Client/Modules/Admin/Users/Index.razor b/Oqtane.Client/Modules/Admin/Users/Index.razor index 682e2d3b..efcd89a2 100644 --- a/Oqtane.Client/Modules/Admin/Users/Index.razor +++ b/Oqtane.Client/Modules/Admin/Users/Index.razor @@ -58,10 +58,10 @@ else { if (string.IsNullOrEmpty(_search)) { - return allroles.Where(item => item.Role.Name == Constants.RegisteredRole).ToList(); + return allroles.Where(item => item.Role.Name == RoleNames.Registered).ToList(); } return allroles - .Where(item => item.Role.Name == Constants.RegisteredRole && + .Where(item => item.Role.Name == RoleNames.Registered && ( item.User.Username.Contains(search, StringComparison.OrdinalIgnoreCase) || item.User.Email.Contains(search, StringComparison.OrdinalIgnoreCase) || diff --git a/Oqtane.Client/Modules/Admin/Users/Roles.razor b/Oqtane.Client/Modules/Admin/Users/Roles.razor index a4e946df..9a056a43 100644 --- a/Oqtane.Client/Modules/Admin/Users/Roles.razor +++ b/Oqtane.Client/Modules/Admin/Users/Roles.razor @@ -63,7 +63,7 @@ else @context.Role.Name - @if (context.Role.Name != Constants.RegisteredRole) + @if (context.Role.Name != RoleNames.Registered) { } diff --git a/Oqtane.Client/Modules/Controls/ActionDialog.razor b/Oqtane.Client/Modules/Controls/ActionDialog.razor index fc22605c..94254ab9 100644 --- a/Oqtane.Client/Modules/Controls/ActionDialog.razor +++ b/Oqtane.Client/Modules/Controls/ActionDialog.razor @@ -134,10 +134,10 @@ authorized = UserSecurity.IsAuthorized(PageState.User,PermissionNames.Edit, ModuleState.Permissions); break; case SecurityAccessLevel.Admin: - authorized = UserSecurity.IsAuthorized(PageState.User, Constants.AdminRole); + authorized = UserSecurity.IsAuthorized(PageState.User, RoleNames.Admin); break; case SecurityAccessLevel.Host: - authorized = UserSecurity.IsAuthorized(PageState.User, Constants.HostRole); + authorized = UserSecurity.IsAuthorized(PageState.User, RoleNames.Host); break; } } diff --git a/Oqtane.Client/Modules/Controls/ActionLink.razor b/Oqtane.Client/Modules/Controls/ActionLink.razor index 7de2331a..a4bcba0f 100644 --- a/Oqtane.Client/Modules/Controls/ActionLink.razor +++ b/Oqtane.Client/Modules/Controls/ActionLink.razor @@ -135,10 +135,10 @@ authorized = UserSecurity.IsAuthorized(PageState.User, PermissionNames.Edit, ModuleState.Permissions); break; case SecurityAccessLevel.Admin: - authorized = UserSecurity.IsAuthorized(PageState.User, Constants.AdminRole); + authorized = UserSecurity.IsAuthorized(PageState.User, RoleNames.Admin); break; case SecurityAccessLevel.Host: - authorized = UserSecurity.IsAuthorized(PageState.User, Constants.HostRole); + authorized = UserSecurity.IsAuthorized(PageState.User, RoleNames.Host); break; } } diff --git a/Oqtane.Client/Modules/Controls/FileManager.razor b/Oqtane.Client/Modules/Controls/FileManager.razor index 7872aba0..291b2acb 100644 --- a/Oqtane.Client/Modules/Controls/FileManager.razor +++ b/Oqtane.Client/Modules/Controls/FileManager.razor @@ -173,7 +173,7 @@ _haseditpermission = false; if (!string.IsNullOrEmpty(Folder)) { - _haseditpermission = UserSecurity.IsAuthorized(PageState.User, Constants.HostRole); + _haseditpermission = UserSecurity.IsAuthorized(PageState.User, RoleNames.Host); _files = await FileService.GetFilesAsync(Folder); } else diff --git a/Oqtane.Client/Modules/Controls/ModuleMessage.razor b/Oqtane.Client/Modules/Controls/ModuleMessage.razor index 171a8926..680f45c0 100644 --- a/Oqtane.Client/Modules/Controls/ModuleMessage.razor +++ b/Oqtane.Client/Modules/Controls/ModuleMessage.razor @@ -6,7 +6,7 @@ {
- @if (UserSecurity.IsAuthorized(PageState.User, Constants.AdminRole)) + @if (UserSecurity.IsAuthorized(PageState.User, RoleNames.Admin)) {
@@ -51,7 +51,7 @@

- @if (UserSecurity.GetPermissionStrings(PageState.Page.Permissions).FirstOrDefault(item => item.PermissionName == PermissionNames.View).Permissions.Split(';').Contains(Constants.AllUsersRole)) + @if (UserSecurity.GetPermissionStrings(PageState.Page.Permissions).FirstOrDefault(item => item.PermissionName == PermissionNames.View).Permissions.Split(';').Contains(RoleNames.Everyone)) {
@@ -510,8 +510,8 @@ if (permissionstring.PermissionName == PermissionNames.View) { List ids = permissionstring.Permissions.Split(';').ToList(); - if (!ids.Contains(Constants.AllUsersRole)) ids.Add(Constants.AllUsersRole); - if (!ids.Contains(Constants.RegisteredRole)) ids.Add(Constants.RegisteredRole); + if (!ids.Contains(RoleNames.Everyone)) ids.Add(RoleNames.Everyone); + if (!ids.Contains(RoleNames.Registered)) ids.Add(RoleNames.Registered); permissionstring.Permissions = string.Join(";", ids.ToArray()); } } @@ -531,12 +531,12 @@ switch (action) { case "publish": - if (!ids.Contains(Constants.AllUsersRole)) ids.Add(Constants.AllUsersRole); - if (!ids.Contains(Constants.RegisteredRole)) ids.Add(Constants.RegisteredRole); + if (!ids.Contains(RoleNames.Everyone)) ids.Add(RoleNames.Everyone); + if (!ids.Contains(RoleNames.Registered)) ids.Add(RoleNames.Registered); break; case "unpublish": - ids.Remove(Constants.AllUsersRole); - ids.Remove(Constants.RegisteredRole); + ids.Remove(RoleNames.Everyone); + ids.Remove(RoleNames.Registered); break; } permissionstring.Permissions = string.Join(";", ids.ToArray()); diff --git a/Oqtane.Client/Themes/Controls/ModuleActionsBase.cs b/Oqtane.Client/Themes/Controls/ModuleActionsBase.cs index a6a3f2ba..b0e95901 100644 --- a/Oqtane.Client/Themes/Controls/ModuleActionsBase.cs +++ b/Oqtane.Client/Themes/Controls/ModuleActionsBase.cs @@ -33,7 +33,7 @@ namespace Oqtane.Themes.Controls { actionList.Add(new ActionViewModel {Icon = Icons.Cog, Name = "Manage Settings", Action = async (u, m) => await Settings(u, m)}); - if (UserSecurity.GetPermissionStrings(ModuleState.Permissions).FirstOrDefault(item => item.PermissionName == PermissionNames.View).Permissions.Split(';').Contains(Constants.AllUsersRole)) + if (UserSecurity.GetPermissionStrings(ModuleState.Permissions).FirstOrDefault(item => item.PermissionName == PermissionNames.View).Permissions.Split(';').Contains(RoleNames.Everyone)) { actionList.Add(new ActionViewModel {Icon=Icons.CircleX, Name = "Unpublish Module", Action = async (s, m) => await Unpublish(s, m) }); } @@ -141,8 +141,8 @@ namespace Oqtane.Themes.Controls if (permissionstring.PermissionName == PermissionNames.View) { List ids = permissionstring.Permissions.Split(';').ToList(); - if (!ids.Contains(Constants.AllUsersRole)) ids.Add(Constants.AllUsersRole); - if (!ids.Contains(Constants.RegisteredRole)) ids.Add(Constants.RegisteredRole); + if (!ids.Contains(RoleNames.Everyone)) ids.Add(RoleNames.Everyone); + if (!ids.Contains(RoleNames.Registered)) ids.Add(RoleNames.Registered); permissionstring.Permissions = string.Join(";", ids.ToArray()); } } @@ -159,8 +159,8 @@ namespace Oqtane.Themes.Controls if (permissionstring.PermissionName == PermissionNames.View) { List ids = permissionstring.Permissions.Split(';').ToList(); - ids.Remove(Constants.AllUsersRole); - ids.Remove(Constants.RegisteredRole); + ids.Remove(RoleNames.Everyone); + ids.Remove(RoleNames.Registered); permissionstring.Permissions = string.Join(";", ids.ToArray()); } } diff --git a/Oqtane.Client/UI/Pane.razor b/Oqtane.Client/UI/Pane.razor index 0c2c0f69..5ddd4d70 100644 --- a/Oqtane.Client/UI/Pane.razor +++ b/Oqtane.Client/UI/Pane.razor @@ -81,10 +81,10 @@ else authorized = UserSecurity.IsAuthorized(PageState.User, PermissionNames.Edit, module.Permissions); break; case SecurityAccessLevel.Admin: - authorized = UserSecurity.IsAuthorized(PageState.User, Constants.AdminRole); + authorized = UserSecurity.IsAuthorized(PageState.User, RoleNames.Admin); break; case SecurityAccessLevel.Host: - authorized = UserSecurity.IsAuthorized(PageState.User, Constants.HostRole); + authorized = UserSecurity.IsAuthorized(PageState.User, RoleNames.Host); break; } } diff --git a/Oqtane.Server/Controllers/AliasController.cs b/Oqtane.Server/Controllers/AliasController.cs index c4422587..7b202b46 100644 --- a/Oqtane.Server/Controllers/AliasController.cs +++ b/Oqtane.Server/Controllers/AliasController.cs @@ -32,7 +32,7 @@ namespace Oqtane.Controllers // GET: api/ [HttpGet] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public IEnumerable Get() { return _aliases.GetAliases(); @@ -40,7 +40,7 @@ namespace Oqtane.Controllers // GET api//5 [HttpGet("{id}")] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public Alias Get(int id) { return _aliases.GetAlias(id); @@ -86,7 +86,7 @@ namespace Oqtane.Controllers // POST api/ [HttpPost] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public Alias Post([FromBody] Alias alias) { if (ModelState.IsValid) @@ -99,7 +99,7 @@ namespace Oqtane.Controllers // PUT api//5 [HttpPut("{id}")] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public Alias Put(int id, [FromBody] Alias alias) { if (ModelState.IsValid) @@ -112,7 +112,7 @@ namespace Oqtane.Controllers // DELETE api//5 [HttpDelete("{id}")] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public void Delete(int id) { _aliases.DeleteAlias(id); diff --git a/Oqtane.Server/Controllers/FileController.cs b/Oqtane.Server/Controllers/FileController.cs index 19281dff..73fad56e 100644 --- a/Oqtane.Server/Controllers/FileController.cs +++ b/Oqtane.Server/Controllers/FileController.cs @@ -58,7 +58,7 @@ namespace Oqtane.Controllers } else { - if (User.IsInRole(Constants.HostRole)) + if (User.IsInRole(RoleNames.Host)) { folder = GetFolderPath(folder); if (Directory.Exists(folder)) @@ -132,7 +132,7 @@ namespace Oqtane.Controllers // PUT api//5 [HttpPut("{id}")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public Models.File Put(int id, [FromBody] Models.File file) { if (ModelState.IsValid && _userPermissions.IsAuthorized(User, EntityNames.Folder, file.FolderId, PermissionNames.Edit)) @@ -164,7 +164,7 @@ namespace Oqtane.Controllers // DELETE api//5 [HttpDelete("{id}")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public void Delete(int id) { Models.File file = _files.GetFile(id); @@ -282,7 +282,7 @@ namespace Oqtane.Controllers } else { - if (User.IsInRole(Constants.HostRole)) + if (User.IsInRole(RoleNames.Host)) { folderPath = GetFolderPath(folder); } diff --git a/Oqtane.Server/Controllers/FolderController.cs b/Oqtane.Server/Controllers/FolderController.cs index 52a0c9fb..b4ad16c4 100644 --- a/Oqtane.Server/Controllers/FolderController.cs +++ b/Oqtane.Server/Controllers/FolderController.cs @@ -93,7 +93,7 @@ namespace Oqtane.Controllers // POST api/ [HttpPost] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public Folder Post([FromBody] Folder folder) { if (ModelState.IsValid) @@ -106,7 +106,7 @@ namespace Oqtane.Controllers else { permissions = new List { - new Permission(PermissionNames.Edit, Constants.AdminRole, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true), }.EncodePermissions(); } if (_userPermissions.IsAuthorized(User, PermissionNames.Edit, permissions)) @@ -141,7 +141,7 @@ namespace Oqtane.Controllers // PUT api//5 [HttpPut("{id}")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public Folder Put(int id, [FromBody] Folder folder) { if (ModelState.IsValid && _userPermissions.IsAuthorized(User, EntityNames.Folder, folder.FolderId, PermissionNames.Edit)) @@ -182,7 +182,7 @@ namespace Oqtane.Controllers // PUT api//?siteid=x&folderid=y&parentid=z [HttpPut] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public void Put(int siteid, int folderid, int? parentid) { if (_userPermissions.IsAuthorized(User, EntityNames.Folder, folderid, PermissionNames.Edit)) @@ -209,7 +209,7 @@ namespace Oqtane.Controllers // DELETE api//5 [HttpDelete("{id}")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public void Delete(int id) { if (_userPermissions.IsAuthorized(User, EntityNames.Folder, id, PermissionNames.Edit)) diff --git a/Oqtane.Server/Controllers/InstallationController.cs b/Oqtane.Server/Controllers/InstallationController.cs index 29583e8a..91a9fd4e 100644 --- a/Oqtane.Server/Controllers/InstallationController.cs +++ b/Oqtane.Server/Controllers/InstallationController.cs @@ -36,7 +36,7 @@ namespace Oqtane.Controllers { var installation = new Installation {Success = false, Message = ""}; - if (ModelState.IsValid && (User.IsInRole(Constants.HostRole) || string.IsNullOrEmpty(_config.GetConnectionString(SettingKeys.ConnectionStringKey)))) + if (ModelState.IsValid && (User.IsInRole(RoleNames.Host) || string.IsNullOrEmpty(_config.GetConnectionString(SettingKeys.ConnectionStringKey)))) { installation = _databaseManager.Install(config); } @@ -57,7 +57,7 @@ namespace Oqtane.Controllers } [HttpGet("upgrade")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public Installation Upgrade() { var installation = new Installation {Success = true, Message = ""}; diff --git a/Oqtane.Server/Controllers/JobController.cs b/Oqtane.Server/Controllers/JobController.cs index 8e1993aa..d578c357 100644 --- a/Oqtane.Server/Controllers/JobController.cs +++ b/Oqtane.Server/Controllers/JobController.cs @@ -28,7 +28,7 @@ namespace Oqtane.Controllers // GET: api/ [HttpGet] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public IEnumerable Get() { return _jobs.GetJobs(); @@ -36,7 +36,7 @@ namespace Oqtane.Controllers // GET api//5 [HttpGet("{id}")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public Job Get(int id) { return _jobs.GetJob(id); @@ -44,7 +44,7 @@ namespace Oqtane.Controllers // POST api/ [HttpPost] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public Job Post([FromBody] Job job) { if (ModelState.IsValid) @@ -57,7 +57,7 @@ namespace Oqtane.Controllers // PUT api//5 [HttpPut("{id}")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public Job Put(int id, [FromBody] Job job) { if (ModelState.IsValid) @@ -70,7 +70,7 @@ namespace Oqtane.Controllers // DELETE api//5 [HttpDelete("{id}")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public void Delete(int id) { _jobs.DeleteJob(id); @@ -79,7 +79,7 @@ namespace Oqtane.Controllers // GET api//start [HttpGet("start/{id}")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public void Start(int id) { Job job = _jobs.GetJob(id); @@ -93,7 +93,7 @@ namespace Oqtane.Controllers // GET api//stop [HttpGet("stop/{id}")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public void Stop(int id) { Job job = _jobs.GetJob(id); diff --git a/Oqtane.Server/Controllers/JobLogController.cs b/Oqtane.Server/Controllers/JobLogController.cs index 0c3f8bc5..35637cb4 100644 --- a/Oqtane.Server/Controllers/JobLogController.cs +++ b/Oqtane.Server/Controllers/JobLogController.cs @@ -23,7 +23,7 @@ namespace Oqtane.Controllers // GET: api/ [HttpGet] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public IEnumerable Get() { return _jobLogs.GetJobLogs(); @@ -31,7 +31,7 @@ namespace Oqtane.Controllers // GET api//5 [HttpGet("{id}")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public JobLog Get(int id) { return _jobLogs.GetJobLog(id); @@ -39,7 +39,7 @@ namespace Oqtane.Controllers // POST api/ [HttpPost] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public JobLog Post([FromBody] JobLog jobLog) { if (ModelState.IsValid) @@ -52,7 +52,7 @@ namespace Oqtane.Controllers // PUT api//5 [HttpPut("{id}")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public JobLog Put(int id, [FromBody] JobLog jobLog) { if (ModelState.IsValid) @@ -65,7 +65,7 @@ namespace Oqtane.Controllers // DELETE api//5 [HttpDelete("{id}")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public void Delete(int id) { _jobLogs.DeleteJobLog(id); diff --git a/Oqtane.Server/Controllers/LogController.cs b/Oqtane.Server/Controllers/LogController.cs index 8cccfd12..0afb9ad2 100644 --- a/Oqtane.Server/Controllers/LogController.cs +++ b/Oqtane.Server/Controllers/LogController.cs @@ -23,7 +23,7 @@ namespace Oqtane.Controllers // GET: api/?siteid=x&level=y&function=z&rows=50 [HttpGet] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public IEnumerable Get(string siteid, string level, string function, string rows) { return _logs.GetLogs(int.Parse(siteid), level, function, int.Parse(rows)); @@ -31,7 +31,7 @@ namespace Oqtane.Controllers // GET api//5 [HttpGet("{id}")] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public Log Get(int id) { return _logs.GetLog(id); diff --git a/Oqtane.Server/Controllers/ModuleController.cs b/Oqtane.Server/Controllers/ModuleController.cs index b90db0b8..907b4892 100644 --- a/Oqtane.Server/Controllers/ModuleController.cs +++ b/Oqtane.Server/Controllers/ModuleController.cs @@ -89,7 +89,7 @@ namespace Oqtane.Controllers // POST api/ [HttpPost] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public Module Post([FromBody] Module module) { if (ModelState.IsValid && _userPermissions.IsAuthorized(User, EntityNames.Page, module.PageId, PermissionNames.Edit)) @@ -108,7 +108,7 @@ namespace Oqtane.Controllers // PUT api//5 [HttpPut("{id}")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public Module Put(int id, [FromBody] Module module) { if (ModelState.IsValid && _userPermissions.IsAuthorized(User, EntityNames.Module, module.ModuleId, PermissionNames.Edit)) @@ -140,7 +140,7 @@ namespace Oqtane.Controllers // DELETE api//5 [HttpDelete("{id}")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public void Delete(int id) { if (_userPermissions.IsAuthorized(User, EntityNames.Module, id, PermissionNames.Edit)) @@ -157,7 +157,7 @@ namespace Oqtane.Controllers // GET api//export?moduleid=x [HttpGet("export")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public string Export(int moduleid) { string content = ""; @@ -175,7 +175,7 @@ namespace Oqtane.Controllers // POST api//import?moduleid=x [HttpPost("import")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public bool Import(int moduleid, [FromBody] string content) { bool success = false; diff --git a/Oqtane.Server/Controllers/ModuleDefinitionController.cs b/Oqtane.Server/Controllers/ModuleDefinitionController.cs index 9c6377d0..59265773 100644 --- a/Oqtane.Server/Controllers/ModuleDefinitionController.cs +++ b/Oqtane.Server/Controllers/ModuleDefinitionController.cs @@ -81,7 +81,7 @@ namespace Oqtane.Controllers // PUT api//5 [HttpPut("{id}")] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public void Put(int id, [FromBody] ModuleDefinition moduleDefinition) { if (ModelState.IsValid) @@ -92,7 +92,7 @@ namespace Oqtane.Controllers } [HttpGet("install")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public void InstallModules() { _logger.Log(LogLevel.Information, this, LogFunction.Create, "Modules Installed"); @@ -101,7 +101,7 @@ namespace Oqtane.Controllers // DELETE api//5?siteid=x [HttpDelete("{id}")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public void Delete(int id, int siteid) { ModuleDefinition moduledefinition = _moduleDefinitions.GetModuleDefinition(id, siteid); @@ -168,7 +168,7 @@ namespace Oqtane.Controllers // POST api/?moduleid=x [HttpPost] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public void Post([FromBody] ModuleDefinition moduleDefinition, string moduleid) { if (ModelState.IsValid) diff --git a/Oqtane.Server/Controllers/NotificationController.cs b/Oqtane.Server/Controllers/NotificationController.cs index 5310f5c7..ec020e79 100644 --- a/Oqtane.Server/Controllers/NotificationController.cs +++ b/Oqtane.Server/Controllers/NotificationController.cs @@ -26,7 +26,7 @@ namespace Oqtane.Controllers // GET: api/?siteid=x&type=y&userid=z [HttpGet] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public IEnumerable Get(string siteid, string direction, string userid) { IEnumerable notifications = null; @@ -46,7 +46,7 @@ namespace Oqtane.Controllers // GET api//5 [HttpGet("{id}")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public Notification Get(int id) { Notification notification = _notifications.GetNotification(id); @@ -59,7 +59,7 @@ namespace Oqtane.Controllers // POST api/ [HttpPost] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public Notification Post([FromBody] Notification notification) { if (IsAuthorized(notification.FromUserId)) @@ -72,7 +72,7 @@ namespace Oqtane.Controllers // PUT api//5 [HttpPut("{id}")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public Notification Put(int id, [FromBody] Notification notification) { if (IsAuthorized(notification.FromUserId)) @@ -85,7 +85,7 @@ namespace Oqtane.Controllers // DELETE api//5 [HttpDelete("{id}")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public void Delete(int id) { Notification notification = _notifications.GetNotification(id); diff --git a/Oqtane.Server/Controllers/PackageController.cs b/Oqtane.Server/Controllers/PackageController.cs index 51194245..95b22c33 100644 --- a/Oqtane.Server/Controllers/PackageController.cs +++ b/Oqtane.Server/Controllers/PackageController.cs @@ -27,7 +27,7 @@ namespace Oqtane.Controllers // GET: api/?tag=x [HttpGet] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public async Task> Get(string tag) { List packages = new List(); @@ -56,7 +56,7 @@ namespace Oqtane.Controllers } [HttpPost] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public async Task Post(string packageid, string version, string folder) { using (var httpClient = new HttpClient()) diff --git a/Oqtane.Server/Controllers/PageController.cs b/Oqtane.Server/Controllers/PageController.cs index 01645921..5a6cff8b 100644 --- a/Oqtane.Server/Controllers/PageController.cs +++ b/Oqtane.Server/Controllers/PageController.cs @@ -102,7 +102,7 @@ namespace Oqtane.Controllers // POST api/ [HttpPost] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public Page Post([FromBody] Page page) { if (ModelState.IsValid) @@ -115,7 +115,7 @@ namespace Oqtane.Controllers else { permissions = new List { - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(); } @@ -147,7 +147,7 @@ namespace Oqtane.Controllers // POST api//5?userid=x [HttpPost("{id}")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public Page Post(int id, string userid) { Page page = null; @@ -213,7 +213,7 @@ namespace Oqtane.Controllers // PUT api//5 [HttpPut("{id}")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public Page Put(int id, [FromBody] Page page) { if (ModelState.IsValid && _userPermissions.IsAuthorized(User, EntityNames.Page, page.PageId, PermissionNames.Edit)) @@ -233,7 +233,7 @@ namespace Oqtane.Controllers // PUT api//?siteid=x&pageid=y&parentid=z [HttpPut] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public void Put(int siteid, int pageid, int? parentid) { if (_userPermissions.IsAuthorized(User, EntityNames.Page, pageid, PermissionNames.Edit)) @@ -261,7 +261,7 @@ namespace Oqtane.Controllers // DELETE api//5 [HttpDelete("{id}")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public void Delete(int id) { Page page = _pages.GetPage(id); diff --git a/Oqtane.Server/Controllers/PageModuleController.cs b/Oqtane.Server/Controllers/PageModuleController.cs index f2c0d489..093816d8 100644 --- a/Oqtane.Server/Controllers/PageModuleController.cs +++ b/Oqtane.Server/Controllers/PageModuleController.cs @@ -65,7 +65,7 @@ namespace Oqtane.Controllers // POST api/ [HttpPost] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public PageModule Post([FromBody] PageModule pageModule) { if (ModelState.IsValid && _userPermissions.IsAuthorized(User, EntityNames.Page, pageModule.PageId, PermissionNames.Edit)) @@ -85,7 +85,7 @@ namespace Oqtane.Controllers // PUT api//5 [HttpPut("{id}")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public PageModule Put(int id, [FromBody] PageModule pageModule) { if (ModelState.IsValid && _userPermissions.IsAuthorized(User, EntityNames.Module, pageModule.ModuleId, PermissionNames.Edit)) @@ -105,7 +105,7 @@ namespace Oqtane.Controllers // PUT api//?pageid=x&pane=y [HttpPut] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public void Put(int pageid, string pane) { if (_userPermissions.IsAuthorized(User, EntityNames.Page, pageid, PermissionNames.Edit)) @@ -133,7 +133,7 @@ namespace Oqtane.Controllers // DELETE api//5 [HttpDelete("{id}")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public void Delete(int id) { PageModule pagemodule = _pageModules.GetPageModule(id); diff --git a/Oqtane.Server/Controllers/ProfileController.cs b/Oqtane.Server/Controllers/ProfileController.cs index aa1158ac..91c0bd05 100644 --- a/Oqtane.Server/Controllers/ProfileController.cs +++ b/Oqtane.Server/Controllers/ProfileController.cs @@ -37,7 +37,7 @@ namespace Oqtane.Controllers // POST api/ [HttpPost] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public Profile Post([FromBody] Profile profile) { if (ModelState.IsValid) @@ -50,7 +50,7 @@ namespace Oqtane.Controllers // PUT api//5 [HttpPut("{id}")] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public Profile Put(int id, [FromBody] Profile profile) { if (ModelState.IsValid) @@ -63,7 +63,7 @@ namespace Oqtane.Controllers // DELETE api//5 [HttpDelete("{id}")] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public void Delete(int id) { _profiles.DeleteProfile(id); diff --git a/Oqtane.Server/Controllers/RoleController.cs b/Oqtane.Server/Controllers/RoleController.cs index 418382c5..a0ce556e 100644 --- a/Oqtane.Server/Controllers/RoleController.cs +++ b/Oqtane.Server/Controllers/RoleController.cs @@ -23,7 +23,7 @@ namespace Oqtane.Controllers // GET: api/?siteid=x [HttpGet] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public IEnumerable Get(string siteid) { return _roles.GetRoles(int.Parse(siteid)); @@ -31,7 +31,7 @@ namespace Oqtane.Controllers // GET api//5 [HttpGet("{id}")] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public Role Get(int id) { return _roles.GetRole(id); @@ -39,7 +39,7 @@ namespace Oqtane.Controllers // POST api/ [HttpPost] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public Role Post([FromBody] Role role) { if (ModelState.IsValid) @@ -52,7 +52,7 @@ namespace Oqtane.Controllers // PUT api//5 [HttpPut("{id}")] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public Role Put(int id, [FromBody] Role role) { if (ModelState.IsValid) @@ -65,7 +65,7 @@ namespace Oqtane.Controllers // DELETE api//5 [HttpDelete("{id}")] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public void Delete(int id) { _roles.DeleteRole(id); diff --git a/Oqtane.Server/Controllers/SettingController.cs b/Oqtane.Server/Controllers/SettingController.cs index e9be3798..2ffaea12 100644 --- a/Oqtane.Server/Controllers/SettingController.cs +++ b/Oqtane.Server/Controllers/SettingController.cs @@ -124,10 +124,10 @@ namespace Oqtane.Controllers switch (entityName) { case EntityNames.Tenant: - authorized = User.IsInRole(Constants.HostRole); + authorized = User.IsInRole(RoleNames.Host); break; case EntityNames.Site: - authorized = User.IsInRole(Constants.AdminRole); + authorized = User.IsInRole(RoleNames.Admin); break; case EntityNames.Page: case EntityNames.Module: @@ -138,7 +138,7 @@ namespace Oqtane.Controllers authorized = true; if (permissionName == PermissionNames.Edit) { - authorized = User.IsInRole(Constants.AdminRole) || (_userPermissions.GetUser(User).UserId == entityId); + authorized = User.IsInRole(RoleNames.Admin) || (_userPermissions.GetUser(User).UserId == entityId); } break; } diff --git a/Oqtane.Server/Controllers/SiteController.cs b/Oqtane.Server/Controllers/SiteController.cs index c358ea3c..d4a50e27 100644 --- a/Oqtane.Server/Controllers/SiteController.cs +++ b/Oqtane.Server/Controllers/SiteController.cs @@ -28,7 +28,7 @@ namespace Oqtane.Controllers // GET: api/ [HttpGet] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public IEnumerable Get() { return _sites.GetSites(); @@ -57,7 +57,7 @@ namespace Oqtane.Controllers } else { - authorized = User.IsInRole(Constants.HostRole); + authorized = User.IsInRole(RoleNames.Host); } if (authorized) { @@ -70,7 +70,7 @@ namespace Oqtane.Controllers // PUT api//5 [HttpPut("{id}")] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public Site Put(int id, [FromBody] Site site) { if (ModelState.IsValid) @@ -84,7 +84,7 @@ namespace Oqtane.Controllers // DELETE api//5 [HttpDelete("{id}")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public void Delete(int id) { _sites.DeleteSite(id); diff --git a/Oqtane.Server/Controllers/SiteTemplateController.cs b/Oqtane.Server/Controllers/SiteTemplateController.cs index c63170c1..ca98eadd 100644 --- a/Oqtane.Server/Controllers/SiteTemplateController.cs +++ b/Oqtane.Server/Controllers/SiteTemplateController.cs @@ -19,7 +19,7 @@ namespace Oqtane.Controllers // GET: api/ [HttpGet] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public IEnumerable Get() { return _siteTemplates.GetSiteTemplates(); diff --git a/Oqtane.Server/Controllers/SqlController.cs b/Oqtane.Server/Controllers/SqlController.cs index 5bce5a92..189002e8 100644 --- a/Oqtane.Server/Controllers/SqlController.cs +++ b/Oqtane.Server/Controllers/SqlController.cs @@ -30,7 +30,7 @@ namespace Oqtane.Controllers // POST: api/ [HttpPost] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public SqlQuery Post([FromBody] SqlQuery sqlquery) { var results = new List>(); diff --git a/Oqtane.Server/Controllers/SystemController.cs b/Oqtane.Server/Controllers/SystemController.cs index 2f5d7339..f45927a1 100644 --- a/Oqtane.Server/Controllers/SystemController.cs +++ b/Oqtane.Server/Controllers/SystemController.cs @@ -19,7 +19,7 @@ namespace Oqtane.Controllers // GET: api/ [HttpGet] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public Dictionary Get() { Dictionary systeminfo = new Dictionary(); diff --git a/Oqtane.Server/Controllers/TenantController.cs b/Oqtane.Server/Controllers/TenantController.cs index 0f76fc78..cf251468 100644 --- a/Oqtane.Server/Controllers/TenantController.cs +++ b/Oqtane.Server/Controllers/TenantController.cs @@ -23,7 +23,7 @@ namespace Oqtane.Controllers // GET: api/ [HttpGet] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public IEnumerable Get() { return _tenants.GetTenants(); @@ -31,7 +31,7 @@ namespace Oqtane.Controllers // GET api//5 [HttpGet("{id}")] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public Tenant Get(int id) { return _tenants.GetTenant(id); @@ -39,7 +39,7 @@ namespace Oqtane.Controllers // POST api/ [HttpPost] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public Tenant Post([FromBody] Tenant tenant) { if (ModelState.IsValid) @@ -52,7 +52,7 @@ namespace Oqtane.Controllers // PUT api//5 [HttpPut("{id}")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public Tenant Put(int id, [FromBody] Tenant tenant) { if (ModelState.IsValid) @@ -65,7 +65,7 @@ namespace Oqtane.Controllers // DELETE api//5 [HttpDelete("{id}")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public void Delete(int id) { _tenants.DeleteTenant(id); diff --git a/Oqtane.Server/Controllers/ThemeController.cs b/Oqtane.Server/Controllers/ThemeController.cs index 3acb66aa..f99fb446 100644 --- a/Oqtane.Server/Controllers/ThemeController.cs +++ b/Oqtane.Server/Controllers/ThemeController.cs @@ -34,14 +34,14 @@ namespace Oqtane.Controllers // GET: api/ [HttpGet] - [Authorize(Roles = Constants.RegisteredRole)] + [Authorize(Roles = RoleNames.Registered)] public IEnumerable Get() { return _themes.GetThemes(); } [HttpGet("install")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public void InstallThemes() { _logger.Log(LogLevel.Information, this, LogFunction.Create, "Themes Installed"); @@ -50,7 +50,7 @@ namespace Oqtane.Controllers // DELETE api//xxx [HttpDelete("{themename}")] - [Authorize(Roles = Constants.HostRole)] + [Authorize(Roles = RoleNames.Host)] public void Delete(string themename) { List themes = _themes.GetThemes().ToList(); diff --git a/Oqtane.Server/Controllers/UserController.cs b/Oqtane.Server/Controllers/UserController.cs index fff15612..e03f1022 100644 --- a/Oqtane.Server/Controllers/UserController.cs +++ b/Oqtane.Server/Controllers/UserController.cs @@ -77,7 +77,7 @@ namespace Oqtane.Controllers private User Filter(User user) { - if (user != null && !User.IsInRole(Constants.AdminRole) && User.Identity.Name?.ToLower() != user.Username.ToLower()) + if (user != null && !User.IsInRole(RoleNames.Admin) && User.Identity.Name?.ToLower() != user.Username.ToLower()) { user.DisplayName = ""; user.Email = ""; @@ -118,7 +118,7 @@ namespace Oqtane.Controllers bool verified; bool allowregistration; - if (user.Username == Constants.HostUser || User.IsInRole(Constants.AdminRole)) + if (user.Username == Constants.HostUser || User.IsInRole(RoleNames.Admin)) { verified = true; allowregistration = true; @@ -166,7 +166,7 @@ namespace Oqtane.Controllers // assign to host role if this is the host user ( initial installation ) if (user.Username == Constants.HostUser) { - int hostroleid = _roles.GetRoles(user.SiteId, true).Where(item => item.Name == Constants.HostRole).FirstOrDefault().RoleId; + int hostroleid = _roles.GetRoles(user.SiteId, true).Where(item => item.Name == RoleNames.Host).FirstOrDefault().RoleId; UserRole userrole = new UserRole(); userrole.UserId = newUser.UserId; userrole.RoleId = hostroleid; @@ -190,7 +190,7 @@ namespace Oqtane.Controllers Permissions = new List { new Permission(PermissionNames.Browse, newUser.UserId, true), - new Permission(PermissionNames.View, Constants.AllUsersRole, true), + new Permission(PermissionNames.View, RoleNames.Everyone, true), new Permission(PermissionNames.Edit, newUser.UserId, true) }.EncodePermissions() }); @@ -242,7 +242,7 @@ namespace Oqtane.Controllers { if (ModelState.IsValid) { - if (User.IsInRole(Constants.AdminRole) || User.Identity.Name == user.Username) + if (User.IsInRole(RoleNames.Admin) || User.Identity.Name == user.Username) { if (user.Password != "") { @@ -270,7 +270,7 @@ namespace Oqtane.Controllers // DELETE api//5?siteid=x [HttpDelete("{id}")] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public async Task Delete(int id) { IdentityUser identityuser = await _identityUserManager.FindByNameAsync(_users.GetUser(id).Username); @@ -460,9 +460,9 @@ namespace Oqtane.Controllers foreach (UserRole userrole in userroles) { roles += userrole.Role.Name + ";"; - if (userrole.Role.Name == Constants.HostRole && userroles.Where(item => item.Role.Name == Constants.AdminRole).FirstOrDefault() == null) + if (userrole.Role.Name == RoleNames.Host && userroles.Where(item => item.Role.Name == RoleNames.Admin).FirstOrDefault() == null) { - roles += Constants.AdminRole + ";"; + roles += RoleNames.Admin + ";"; } } if (roles != "") roles = ";" + roles; diff --git a/Oqtane.Server/Controllers/UserRoleController.cs b/Oqtane.Server/Controllers/UserRoleController.cs index 50b6d957..6eeb3de1 100644 --- a/Oqtane.Server/Controllers/UserRoleController.cs +++ b/Oqtane.Server/Controllers/UserRoleController.cs @@ -27,7 +27,7 @@ namespace Oqtane.Controllers // GET: api/?siteid=x [HttpGet] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public IEnumerable Get(string siteid) { return _userRoles.GetUserRoles(int.Parse(siteid)); @@ -35,7 +35,7 @@ namespace Oqtane.Controllers // GET api//5 [HttpGet("{id}")] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public UserRole Get(int id) { return _userRoles.GetUserRole(id); @@ -43,7 +43,7 @@ namespace Oqtane.Controllers // POST api/ [HttpPost] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public UserRole Post([FromBody] UserRole userRole) { if (ModelState.IsValid) @@ -57,7 +57,7 @@ namespace Oqtane.Controllers // PUT api//5 [HttpPut("{id}")] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public UserRole Put(int id, [FromBody] UserRole userRole) { if (ModelState.IsValid) @@ -71,7 +71,7 @@ namespace Oqtane.Controllers // DELETE api//5 [HttpDelete("{id}")] - [Authorize(Roles = Constants.AdminRole)] + [Authorize(Roles = RoleNames.Admin)] public void Delete(int id) { UserRole userRole = _userRoles.GetUserRole(id); diff --git a/Oqtane.Server/Infrastructure/DatabaseManager.cs b/Oqtane.Server/Infrastructure/DatabaseManager.cs index 6aa2369a..79c72c58 100644 --- a/Oqtane.Server/Infrastructure/DatabaseManager.cs +++ b/Oqtane.Server/Infrastructure/DatabaseManager.cs @@ -458,7 +458,7 @@ namespace Oqtane.Infrastructure }; user = users.AddUser(user); - var hostRoleId = roles.GetRoles(user.SiteId, true).FirstOrDefault(item => item.Name == Constants.HostRole)?.RoleId ?? 0; + var hostRoleId = roles.GetRoles(user.SiteId, true).FirstOrDefault(item => item.Name == RoleNames.Host)?.RoleId ?? 0; var userRole = new UserRole { UserId = user.UserId, RoleId = hostRoleId, EffectiveDate = null, ExpiryDate = null }; userroles.AddUserRole(userRole); @@ -477,7 +477,7 @@ namespace Oqtane.Infrastructure Permissions = new List { new Permission(PermissionNames.Browse, user.UserId, true), - new Permission(PermissionNames.View, Constants.AllUsersRole, true), + new Permission(PermissionNames.View, RoleNames.Everyone, true), new Permission(PermissionNames.Edit, user.UserId, true), }.EncodePermissions(), }); diff --git a/Oqtane.Server/Infrastructure/SiteTemplates/DefaultSiteTemplate.cs b/Oqtane.Server/Infrastructure/SiteTemplates/DefaultSiteTemplate.cs index 8b71c33b..449b7df9 100644 --- a/Oqtane.Server/Infrastructure/SiteTemplates/DefaultSiteTemplate.cs +++ b/Oqtane.Server/Infrastructure/SiteTemplates/DefaultSiteTemplate.cs @@ -43,16 +43,16 @@ namespace Oqtane.SiteTemplates IsNavigation = true, IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.AllUsersRole, true), - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Everyone, true), + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions() , PageTemplateModules = new List { new PageTemplateModule { ModuleDefinitionName = "Oqtane.Modules.HtmlText, Oqtane.Client", Title = "Welcome To Oqtane...", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.AllUsersRole, true), - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Everyone, true), + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "

Oqtane is an open source modular application framework that provides advanced functionality for developing web and mobile applications on ASP.NET Core. It leverages the revolutionary new Blazor component model to compose a fully dynamic web development experience which can be hosted either client-side or server-side. Whether you are looking for a platform to accelerate your web development efforts, or simply interested in exploring the anatomy of a large-scale Blazor application, Oqtane provides a solid foundation based on proven enterprise architectural principles.

" + "

Join Our Community  Clone Our Repo

" + @@ -61,9 +61,9 @@ namespace Oqtane.SiteTemplates }, new PageTemplateModule { ModuleDefinitionName = "Oqtane.Modules.HtmlText, Oqtane.Client", Title = "MIT License", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.AllUsersRole, true), - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Everyone, true), + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "

Copyright (c) 2019-2020 .NET Foundation

" + "

Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the \"Software\"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:

" + @@ -72,9 +72,9 @@ namespace Oqtane.SiteTemplates }, new PageTemplateModule { ModuleDefinitionName = "Oqtane.Modules.HtmlText, Oqtane.Client", Title = "Secure Content", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.RegisteredRole, true), - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Registered, true), + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "

Oqtane allows you to control access to your content using security roles. This module is only visible to Registered Users of the site.

" } @@ -89,16 +89,16 @@ namespace Oqtane.SiteTemplates IsNavigation = true, IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.RegisteredRole, true), - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Registered, true), + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), PageTemplateModules = new List { new PageTemplateModule { ModuleDefinitionName = "Oqtane.Modules.HtmlText, Oqtane.Client", Title = "Secure Content", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.RegisteredRole, true), - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Registered, true), + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "

Oqtane allows you to control access to your content using security roles. This page is only visible to Registered Users of the site.

" } @@ -113,16 +113,16 @@ namespace Oqtane.SiteTemplates IsNavigation = true, IsPersonalizable = true, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.AllUsersRole, true), - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Everyone, true), + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), PageTemplateModules = new List { new PageTemplateModule { ModuleDefinitionName = "Oqtane.Modules.HtmlText, Oqtane.Client", Title = "My Page", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.AllUsersRole, true), - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Everyone, true), + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "

Oqtane offers native support for user personalized pages. If a page is identified as personalizable by the site administrator in the page settings, when an authenticated user visits the page they will see an edit button at the top right corner of the page next to their username. When they click this button the sytem will create a new version of the page and allow them to edit the page content.

" } diff --git a/Oqtane.Server/Infrastructure/SiteTemplates/EmptySiteTemplate.cs b/Oqtane.Server/Infrastructure/SiteTemplates/EmptySiteTemplate.cs index 33ccaf32..408fa3fe 100644 --- a/Oqtane.Server/Infrastructure/SiteTemplates/EmptySiteTemplate.cs +++ b/Oqtane.Server/Infrastructure/SiteTemplates/EmptySiteTemplate.cs @@ -31,9 +31,9 @@ namespace Oqtane.SiteTemplates IsNavigation = true, IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.AllUsersRole, true), - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Everyone, true), + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), PageTemplateModules = new List() }); diff --git a/Oqtane.Server/Infrastructure/UpgradeManager.cs b/Oqtane.Server/Infrastructure/UpgradeManager.cs index 2c538d97..0344de6e 100644 --- a/Oqtane.Server/Infrastructure/UpgradeManager.cs +++ b/Oqtane.Server/Infrastructure/UpgradeManager.cs @@ -40,9 +40,9 @@ namespace Oqtane.Infrastructure // EditMode = false, // PagePermissions = new List // { - // new Permission(PermissionNames.View, Constants.AdminRole, true), - // new Permission(PermissionNames.View, Constants.AllUsersRole, true), - // new Permission(PermissionNames.Edit, Constants.AdminRole, true) + // new Permission(PermissionNames.View, RoleNames.Admin, true), + // new Permission(PermissionNames.View, RoleNames.Everyone, true), + // new Permission(PermissionNames.Edit, RoleNames.Admin, true) // }.EncodePermissions(), // PageTemplateModules = new List // { @@ -51,9 +51,9 @@ namespace Oqtane.Infrastructure // ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Login.Index).ToModuleDefinitionName(), Title = "Test", Pane = "Content", // ModulePermissions = new List // { - // new Permission(PermissionNames.View, Constants.AdminRole, true), - // new Permission(PermissionNames.View, Constants.AllUsersRole, true), - // new Permission(PermissionNames.Edit, Constants.AdminRole, true) + // new Permission(PermissionNames.View, RoleNames.Admin, true), + // new Permission(PermissionNames.View, RoleNames.Everyone, true), + // new Permission(PermissionNames.Edit, RoleNames.Admin, true) // }.EncodePermissions(), // Content = "" // } diff --git a/Oqtane.Server/Repository/ModuleDefinitionRepository.cs b/Oqtane.Server/Repository/ModuleDefinitionRepository.cs index b2bfc71d..b0d21ff2 100644 --- a/Oqtane.Server/Repository/ModuleDefinitionRepository.cs +++ b/Oqtane.Server/Repository/ModuleDefinitionRepository.cs @@ -239,15 +239,15 @@ namespace Oqtane.Repository { moduledefinition.Permissions = new List { - new Permission(PermissionNames.Utilize, Constants.AdminRole, true) + new Permission(PermissionNames.Utilize, RoleNames.Admin, true) }.EncodePermissions(); } else { moduledefinition.Permissions = new List { - new Permission(PermissionNames.Utilize, Constants.AdminRole, true), - new Permission(PermissionNames.Utilize, Constants.RegisteredRole, true) + new Permission(PermissionNames.Utilize, RoleNames.Admin, true), + new Permission(PermissionNames.Utilize, RoleNames.Registered, true) }.EncodePermissions(); } diff --git a/Oqtane.Server/Repository/SiteRepository.cs b/Oqtane.Server/Repository/SiteRepository.cs index 530360da..0130e5e3 100644 --- a/Oqtane.Server/Repository/SiteRepository.cs +++ b/Oqtane.Server/Repository/SiteRepository.cs @@ -60,9 +60,9 @@ namespace Oqtane.Repository IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.View, Constants.AllUsersRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.View, RoleNames.Everyone, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -71,9 +71,9 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Login.Index).ToModuleDefinitionName(), Title = "User Login", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.View, Constants.AllUsersRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.View, RoleNames.Everyone, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "" } @@ -89,9 +89,9 @@ namespace Oqtane.Repository IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.View, Constants.AllUsersRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.View, RoleNames.Everyone, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -100,9 +100,9 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Register.Index).ToModuleDefinitionName(), Title = "User Registration", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.View, Constants.AllUsersRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.View, RoleNames.Everyone, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "" } @@ -119,9 +119,9 @@ namespace Oqtane.Repository IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.View, Constants.AllUsersRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.View, RoleNames.Everyone, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -130,9 +130,9 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Reset.Index).ToModuleDefinitionName(), Title = "Password Reset", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.View, Constants.AllUsersRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.View, RoleNames.Everyone, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "" } @@ -148,9 +148,9 @@ namespace Oqtane.Repository IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.View, Constants.RegisteredRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.View, RoleNames.Registered, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -159,9 +159,9 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.UserProfile.Index).ToModuleDefinitionName(), Title = "User Profile", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.View, Constants.RegisteredRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.View, RoleNames.Registered, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "" } @@ -174,8 +174,8 @@ namespace Oqtane.Repository Name = "Admin", Parent = "", Path = "admin", Icon = "", IsNavigation = false, IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -184,8 +184,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Dashboard.Index).ToModuleDefinitionName(), Title = "Admin Dashboard", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "" } @@ -201,8 +201,8 @@ namespace Oqtane.Repository IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -211,8 +211,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Site.Index).ToModuleDefinitionName(), Title = "Site Settings", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "" } @@ -228,8 +228,8 @@ namespace Oqtane.Repository IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -238,8 +238,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Pages.Index).ToModuleDefinitionName(), Title = "Page Management", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "" } @@ -255,8 +255,8 @@ namespace Oqtane.Repository IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -265,8 +265,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Users.Index).ToModuleDefinitionName(), Title = "User Management", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "" } @@ -282,8 +282,8 @@ namespace Oqtane.Repository IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -292,8 +292,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Profiles.Index).ToModuleDefinitionName(), Title = "Profile Management", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "" } @@ -309,8 +309,8 @@ namespace Oqtane.Repository IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -319,8 +319,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Roles.Index).ToModuleDefinitionName(), Title = "Role Management", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "" } @@ -336,8 +336,8 @@ namespace Oqtane.Repository IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -346,8 +346,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Files.Index).ToModuleDefinitionName(), Title = "File Management", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "" } @@ -363,8 +363,8 @@ namespace Oqtane.Repository IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -373,8 +373,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.RecycleBin.Index).ToModuleDefinitionName(), Title = "Recycle Bin", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions(), Content = "" } @@ -392,8 +392,8 @@ namespace Oqtane.Repository IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -402,8 +402,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Logs.Index).ToModuleDefinitionName(), Title = "Event Log", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), Content = "" } @@ -418,8 +418,8 @@ namespace Oqtane.Repository IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -428,8 +428,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Tenants.Index).ToModuleDefinitionName(), Title = "Tenant Management", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), Content = "" } @@ -440,8 +440,8 @@ namespace Oqtane.Repository Name = "Site Management", Parent = "Admin", Path = "admin/sites", Icon = Icons.Globe, IsNavigation = false, IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -450,8 +450,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Sites.Index).ToModuleDefinitionName(), Title = "Site Management", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), Content = "" } @@ -462,8 +462,8 @@ namespace Oqtane.Repository Name = "Module Management", Parent = "Admin", Path = "admin/modules", Icon = Icons.Browser, IsNavigation = false, IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -472,8 +472,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.ModuleDefinitions.Index).ToModuleDefinitionName(), Title = "Module Management", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), Content = "" } @@ -484,8 +484,8 @@ namespace Oqtane.Repository Name = "Theme Management", Parent = "Admin", Path = "admin/themes", Icon = Icons.Brush, IsNavigation = false, IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -494,8 +494,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Themes.Index).ToModuleDefinitionName(), Title = "Theme Management", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), Content = "" } @@ -506,8 +506,8 @@ namespace Oqtane.Repository Name = "Scheduled Jobs", Parent = "Admin", Path = "admin/jobs", Icon = Icons.Timer, IsNavigation = false, IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -516,8 +516,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Jobs.Index).ToModuleDefinitionName(), Title = "Scheduled Jobs", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), Content = "" } @@ -533,8 +533,8 @@ namespace Oqtane.Repository IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -543,8 +543,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Sql.Index).ToModuleDefinitionName(), Title = "Sql Management", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), Content = "" } @@ -560,8 +560,8 @@ namespace Oqtane.Repository IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -570,8 +570,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.SystemInfo.Index).ToModuleDefinitionName(), Title = "System Info", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), Content = "" } @@ -582,8 +582,8 @@ namespace Oqtane.Repository Name = "System Update", Parent = "Admin", Path = "admin/update", Icon = Icons.Aperture, IsNavigation = false, IsPersonalizable = false, PagePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), PageTemplateModules = new List { @@ -592,8 +592,8 @@ namespace Oqtane.Repository ModuleDefinitionName = typeof(Oqtane.Modules.Admin.Upgrade.Index).ToModuleDefinitionName(), Title = "System Update", Pane = "Content", ModulePermissions = new List { - new Permission(PermissionNames.View, Constants.HostRole, true), - new Permission(PermissionNames.Edit, Constants.HostRole, true) + new Permission(PermissionNames.View, RoleNames.Host, true), + new Permission(PermissionNames.Edit, RoleNames.Host, true) }.EncodePermissions(), Content = "" } @@ -640,18 +640,18 @@ namespace Oqtane.Repository { // create default entities for site List roles = _roleRepository.GetRoles(site.SiteId, true).ToList(); - if (!roles.Where(item => item.Name == Constants.AllUsersRole).Any()) + if (!roles.Where(item => item.Name == RoleNames.Everyone).Any()) { - _roleRepository.AddRole(new Role {SiteId = null, Name = Constants.AllUsersRole, Description = "All Users", IsAutoAssigned = false, IsSystem = true}); + _roleRepository.AddRole(new Role {SiteId = null, Name = RoleNames.Everyone, Description = "All Users", IsAutoAssigned = false, IsSystem = true}); } - if (!roles.Where(item => item.Name == Constants.HostRole).Any()) + if (!roles.Where(item => item.Name == RoleNames.Host).Any()) { - _roleRepository.AddRole(new Role {SiteId = null, Name = Constants.HostRole, Description = "Application Administrators", IsAutoAssigned = false, IsSystem = true}); + _roleRepository.AddRole(new Role {SiteId = null, Name = RoleNames.Host, Description = "Application Administrators", IsAutoAssigned = false, IsSystem = true}); } - _roleRepository.AddRole(new Role {SiteId = site.SiteId, Name = Constants.RegisteredRole, Description = "Registered Users", IsAutoAssigned = true, IsSystem = true}); - _roleRepository.AddRole(new Role {SiteId = site.SiteId, Name = Constants.AdminRole, Description = "Site Administrators", IsAutoAssigned = false, IsSystem = true}); + _roleRepository.AddRole(new Role {SiteId = site.SiteId, Name = RoleNames.Registered, Description = "Registered Users", IsAutoAssigned = true, IsSystem = true}); + _roleRepository.AddRole(new Role {SiteId = site.SiteId, Name = RoleNames.Admin, Description = "Site Administrators", IsAutoAssigned = false, IsSystem = true}); _profileRepository.AddProfile(new Profile {SiteId = site.SiteId, Name = "FirstName", Title = "First Name", Description = "Your First Or Given Name", Category = "Name", ViewOrder = 1, MaxLength = 50, DefaultValue = "", IsRequired = true, IsPrivate = false}); @@ -675,9 +675,9 @@ namespace Oqtane.Repository SiteId = site.SiteId, ParentId = null, Name = "Root", Path = "", Order = 1, IsSystem = true, Permissions = new List { - new Permission(PermissionNames.Browse, Constants.AdminRole, true), - new Permission(PermissionNames.View, Constants.AllUsersRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.Browse, RoleNames.Admin, true), + new Permission(PermissionNames.View, RoleNames.Everyone, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions() }); _folderRepository.AddFolder(new Folder @@ -685,9 +685,9 @@ namespace Oqtane.Repository SiteId = site.SiteId, ParentId = folder.FolderId, Name = "Users", Path = Utilities.PathCombine("Users",Path.DirectorySeparatorChar.ToString()), Order = 1, IsSystem = true, Permissions = new List { - new Permission(PermissionNames.Browse, Constants.AdminRole, true), - new Permission(PermissionNames.View, Constants.AdminRole, true), - new Permission(PermissionNames.Edit, Constants.AdminRole, true) + new Permission(PermissionNames.Browse, RoleNames.Admin, true), + new Permission(PermissionNames.View, RoleNames.Admin, true), + new Permission(PermissionNames.Edit, RoleNames.Admin, true) }.EncodePermissions() }); diff --git a/Oqtane.Server/Security/ClaimsPrincipalFactory.cs b/Oqtane.Server/Security/ClaimsPrincipalFactory.cs index 1e945938..6ac81731 100644 --- a/Oqtane.Server/Security/ClaimsPrincipalFactory.cs +++ b/Oqtane.Server/Security/ClaimsPrincipalFactory.cs @@ -39,15 +39,15 @@ namespace Oqtane.Security { id.AddClaim(new Claim(_options.ClaimsIdentity.RoleClaimType, userrole.Role.Name)); // host users are members of every site - if (userrole.Role.Name == Constants.HostRole) + if (userrole.Role.Name == RoleNames.Host) { - if (userroles.Where(item => item.Role.Name == Constants.RegisteredRole).FirstOrDefault() == null) + if (userroles.Where(item => item.Role.Name == RoleNames.Registered).FirstOrDefault() == null) { - id.AddClaim(new Claim(_options.ClaimsIdentity.RoleClaimType, Constants.RegisteredRole)); + id.AddClaim(new Claim(_options.ClaimsIdentity.RoleClaimType, RoleNames.Registered)); } - if (userroles.Where(item => item.Role.Name == Constants.AdminRole).FirstOrDefault() == null) + if (userroles.Where(item => item.Role.Name == RoleNames.Admin).FirstOrDefault() == null) { - id.AddClaim(new Claim(_options.ClaimsIdentity.RoleClaimType, Constants.AdminRole)); + id.AddClaim(new Claim(_options.ClaimsIdentity.RoleClaimType, RoleNames.Admin)); } } } diff --git a/Oqtane.Shared/Security/UserSecurity.cs b/Oqtane.Shared/Security/UserSecurity.cs index a8e85304..ec2d0db0 100644 --- a/Oqtane.Shared/Security/UserSecurity.cs +++ b/Oqtane.Shared/Security/UserSecurity.cs @@ -85,14 +85,14 @@ namespace Oqtane.Security if (permission.StartsWith("!")) { string denyRole = permission.Replace("!", ""); - if (denyRole == Constants.AllUsersRole || IsAllowed(userId, roles, denyRole)) + if (denyRole == RoleNames.Everyone || IsAllowed(userId, roles, denyRole)) { allowed = false; } } else // grant permission { - if (permission == Constants.AllUsersRole || IsAllowed(userId, roles, permission)) + if (permission == RoleNames.Everyone || IsAllowed(userId, roles, permission)) { allowed = true; } diff --git a/Oqtane.Shared/Shared/Constants.cs b/Oqtane.Shared/Shared/Constants.cs index db62f416..ec6d7df7 100644 --- a/Oqtane.Shared/Shared/Constants.cs +++ b/Oqtane.Shared/Shared/Constants.cs @@ -1,10 +1,9 @@ using System; using System.Globalization; -namespace Oqtane.Shared -{ - public class Constants - { +namespace Oqtane.Shared { + + public class Constants { public const string PackageId = "Oqtane.Framework"; public const string Version = "1.0.4"; public const string ReleaseVersions = "1.0.0,1.0.1,1.0.2,1.0.3,1.0.4"; @@ -41,10 +40,16 @@ namespace Oqtane.Shared public const string MasterTenant = "Master"; public const string DefaultSite = "Default Site"; - public const string AllUsersRole = "All Users"; - public const string HostRole = "Host Users"; - public const string AdminRole = "Administrators"; - public const string RegisteredRole = "Registered Users"; + const string RoleObsoleteMessage = "Use the corresponding memeber from Oqtane.Shared.RoleNames"; + + [Obsolete(RoleObsoleteMessage)] + public const string AllUsersRole = RoleNames.Everyone; + [Obsolete(RoleObsoleteMessage)] + public const string HostRole = RoleNames.Host; + [Obsolete(RoleObsoleteMessage)] + public const string AdminRole = RoleNames.Admin; + [Obsolete(RoleObsoleteMessage)] + public const string RegisteredRole = RoleNames.Registered; public const string ImageFiles = "jpg,jpeg,jpe,gif,bmp,png,svg,ico"; public const string UploadableFiles = "jpg,jpeg,jpe,gif,bmp,png,svg,ico,mov,wmv,avi,mp4,mp3,doc,docx,xls,xlsx,ppt,pptx,pdf,txt,zip,nupkg,csv"; diff --git a/Oqtane.Shared/Shared/RoleNames.cs b/Oqtane.Shared/Shared/RoleNames.cs new file mode 100644 index 00000000..f526d613 --- /dev/null +++ b/Oqtane.Shared/Shared/RoleNames.cs @@ -0,0 +1,8 @@ +namespace Oqtane.Shared { + public class RoleNames { + public const string Everyone = "All Users"; + public const string Host = "Host Users"; + public const string Admin = "Administrators"; + public const string Registered = "Registered Users"; + } +}