diff --git a/Oqtane.Client/Modules/Admin/Files/Add.razor b/Oqtane.Client/Modules/Admin/Files/Add.razor index 09bd63b1..b5abe13c 100644 --- a/Oqtane.Client/Modules/Admin/Files/Add.razor +++ b/Oqtane.Client/Modules/Admin/Files/Add.razor @@ -81,6 +81,7 @@ { validated = true; var interop = new Interop(JSRuntime); + if (await interop.FormValid(form)) { if (_url == string.Empty || _folderId == -1) diff --git a/Oqtane.Client/Modules/Admin/Site/Index.razor b/Oqtane.Client/Modules/Admin/Site/Index.razor index 5da7e315..db2cf232 100644 --- a/Oqtane.Client/Modules/Admin/Site/Index.razor +++ b/Oqtane.Client/Modules/Admin/Site/Index.razor @@ -646,8 +646,8 @@ settings = SettingService.SetSetting(settings, "NotificationRetention", _retention.ToString(), true); //File Extensions - settings = SettingService.SetSetting(settings, "ImageFiles", _ImageFiles, true); - settings = SettingService.SetSetting(settings, "UploadableFiles", _UploadableFiles, true); + settings = SettingService.SetSetting(settings, "ImageFiles", _ImageFiles, false); + settings = SettingService.SetSetting(settings, "UploadableFiles", _UploadableFiles, false); await SettingService.UpdateSiteSettingsAsync(settings, site.SiteId); diff --git a/Oqtane.Client/Modules/Controls/FileManager.razor b/Oqtane.Client/Modules/Controls/FileManager.razor index 2b44c06a..d779531e 100644 --- a/Oqtane.Client/Modules/Controls/FileManager.razor +++ b/Oqtane.Client/Modules/Controls/FileManager.razor @@ -344,7 +344,7 @@ _message = string.Empty; var interop = new Interop(JSRuntime); var uploads = await interop.GetFiles(_fileinputid); - + if (uploads.Length > 0) { string restricted = ""; diff --git a/Oqtane.Server/Controllers/FileController.cs b/Oqtane.Server/Controllers/FileController.cs index 96173a0b..65cd3fec 100644 --- a/Oqtane.Server/Controllers/FileController.cs +++ b/Oqtane.Server/Controllers/FileController.cs @@ -288,8 +288,7 @@ namespace Oqtane.Controllers folder = _folders.GetFolder(FolderId); } - var _ImageFiles = _settingRepository.GetSetting(EntityNames.Site, _alias.SiteId, "ImageFiles").SettingValue; - var _UploadableFiles = _ImageFiles + "," + _settingRepository.GetSetting(EntityNames.Site, _alias.SiteId, "UploadableFiles").SettingValue; + var _UploadableFiles = (_settingRepository.GetSetting(EntityNames.Site, _alias.SiteId, "UploadableFiles")?.SettingValue ?? Constants.UploadableFiles) ?? Constants.UploadableFiles; if (folder != null && folder.SiteId == _alias.SiteId && _userPermissions.IsAuthorized(User, PermissionNames.Edit, folder.PermissionList)) { @@ -367,13 +366,8 @@ namespace Oqtane.Controllers } // Get the UploadableFiles extensions - string imagefilesSetting = _settingRepository.GetSetting(EntityNames.Site, _alias.SiteId, "ImageFiles")?.SettingValue; - string imagefiles = imagefilesSetting ?? Constants.ImageFiles; - string uploadfilesSetting = _settingRepository.GetSetting(EntityNames.Site, _alias.SiteId, "UploadableFiles")?.SettingValue; - string uploadfiles = uploadfilesSetting ?? Constants.UploadableFiles; - - string _UploadableFiles = $"{imagefiles},{uploadfiles}"; + string _UploadableFiles = uploadfilesSetting ?? Constants.UploadableFiles; // ensure filename is valid string token = ".part_"; @@ -617,8 +611,8 @@ namespace Oqtane.Controllers public IActionResult GetImage(int id, int width, int height, string mode, string position, string background, string rotate, string recreate) { var file = _files.GetFile(id); - - var _ImageFiles = _settingRepository.GetSetting(EntityNames.Site, _alias.SiteId, "ImageFiles").SettingValue; + + var _ImageFiles = (_settingRepository.GetSetting(EntityNames.Site, _alias.SiteId, "ImageFiles")?.SettingValue ?? Constants.ImageFiles) ?? Constants.ImageFiles; if (file != null && file.Folder.SiteId == _alias.SiteId && _userPermissions.IsAuthorized(User, PermissionNames.View, file.Folder.PermissionList)) { if (_ImageFiles.Split(',').Contains(file.Extension.ToLower())) @@ -785,7 +779,7 @@ namespace Oqtane.Controllers private Models.File CreateFile(string filename, int folderid, string filepath) { var file = _files.GetFile(folderid, filename); - var _ImageFiles = _settingRepository.GetSetting(EntityNames.Site, _alias.SiteId, "ImageFiles").SettingValue; + var _ImageFiles = (_settingRepository.GetSetting(EntityNames.Site, _alias.SiteId, "ImageFiles")?.SettingValue ?? Constants.ImageFiles) ?? Constants.ImageFiles; int size = 0; var folder = _folders.GetFolder(folderid, false);