diff --git a/Oqtane.Client/Modules/Admin/Languages/Index.razor b/Oqtane.Client/Modules/Admin/Languages/Index.razor index b9b3d3f7..72892e51 100644 --- a/Oqtane.Client/Modules/Admin/Languages/Index.razor +++ b/Oqtane.Client/Modules/Admin/Languages/Index.razor @@ -99,56 +99,64 @@ else } @code { - private List _languages; - private List _packages; - private Package _package; + private List _languages; + private List _packages; + private Package _package; - public override SecurityAccessLevel SecurityAccessLevel => SecurityAccessLevel.Admin; + public override SecurityAccessLevel SecurityAccessLevel => SecurityAccessLevel.Admin; - protected override async Task OnParametersSetAsync() - { - await GetLanguages(); + protected override async Task OnParametersSetAsync() + { + await GetLanguages(); + + if (UserSecurity.IsAuthorized(PageState.User, RoleNames.Host)) + { + _packages = await PackageService.GetPackagesAsync("translation"); + } + } - if (UserSecurity.IsAuthorized(PageState.User, RoleNames.Host)) - { - _packages = await PackageService.GetPackagesAsync("translation"); - } - } - private async Task GetLanguages() - { - _languages = await LanguageService.GetLanguagesAsync(PageState.Site.SiteId, Constants.ClientId); - } + { + _languages = await LanguageService.GetLanguagesAsync(PageState.Site.SiteId, Constants.ClientId); + } - private async Task DeleteLanguage(Language language) - { - try - { - await LanguageService.DeleteLanguageAsync(language.LanguageId); - await logger.LogInformation("Language Deleted {Language}", language); - await GetLanguages(); - StateHasChanged(); - } - catch (Exception ex) - { - await logger.LogError(ex, "Error Deleting Language {Language} {Error}", language, ex.Message); + private async Task DeleteLanguage(Language language) + { + try + { + await LanguageService.DeleteLanguageAsync(language.LanguageId); + await logger.LogInformation("Language Deleted {Language}", language); + await GetLanguages(); + StateHasChanged(); + } + catch (Exception ex) + { + await logger.LogError(ex, "Error Deleting Language {Language} {Error}", language, ex.Message); - AddModuleMessage(Localizer["Error.Language.Delete"], MessageType.Error); - } - } + AddModuleMessage(Localizer["Error.Language.Delete"], MessageType.Error); + } + } - private Package TranslationAvailable(string code, string version) - { - return _packages?.FirstOrDefault(item => item.PackageId == (Constants.PackageId + "." + code)); - } + private Package TranslationAvailable(string code, string version) + { + return _packages?.FirstOrDefault(item => item.PackageId == (Constants.PackageId + "." + code)); + } - private async Task GetPackage(string code, string version) - { - try - { - _package = await PackageService.GetPackageAsync(Constants.PackageId + "." + code, version); - StateHasChanged(); - } + private async Task GetPackage(string code, string version) + { + try + { + _package = await PackageService.GetPackageAsync(Constants.PackageId + "." + code, version); + if (_package != null) + { + StateHasChanged(); + } + else + { + await logger.LogError("Error Getting Package {PackageId} {Version}", Constants.PackageId + "." + code, Constants.Version); + AddModuleMessage(Localizer["Error.Translation.Download"], MessageType.Error); + } + } catch (Exception ex) { await logger.LogError(ex, "Error Getting Package {PackageId} {Version}", Constants.PackageId + "." + code, Constants.Version); diff --git a/Oqtane.Client/Modules/Admin/ModuleDefinitions/Add.razor b/Oqtane.Client/Modules/Admin/ModuleDefinitions/Add.razor index 548a45f5..700978fc 100644 --- a/Oqtane.Client/Modules/Admin/ModuleDefinitions/Add.razor +++ b/Oqtane.Client/Modules/Admin/ModuleDefinitions/Add.razor @@ -217,8 +217,13 @@ _packagelicense = package.License.Replace("\n", "
"); } _packageversion = package.Version; + StateHasChanged(); + } + else + { + await logger.LogError("Error Getting Package {PackageId} {Version}", packageid, version); + AddModuleMessage(Localizer["Error.Module.Download"], MessageType.Error); } - StateHasChanged(); } catch (Exception ex) { diff --git a/Oqtane.Client/Modules/Admin/ModuleDefinitions/Edit.razor b/Oqtane.Client/Modules/Admin/ModuleDefinitions/Edit.razor index d030299c..6a2c6e5a 100644 --- a/Oqtane.Client/Modules/Admin/ModuleDefinitions/Edit.razor +++ b/Oqtane.Client/Modules/Admin/ModuleDefinitions/Edit.razor @@ -319,55 +319,63 @@ AddModuleMessage(Localizer["Message.DuplicateName"], MessageType.Warning); } } - catch (Exception ex) - { - await logger.LogError(ex, "Error Saving ModuleDefinition {ModuleDefinitionId} {Error}", _moduleDefinitionId, ex.Message); - AddModuleMessage(Localizer["Error.Module.Save"], MessageType.Error); - } - } - else - { - AddModuleMessage(SharedLocalizer["Message.InfoRequired"], MessageType.Warning); - } - } + catch (Exception ex) + { + await logger.LogError(ex, "Error Saving ModuleDefinition {ModuleDefinitionId} {Error}", _moduleDefinitionId, ex.Message); + AddModuleMessage(Localizer["Error.Module.Save"], MessageType.Error); + } + } + else + { + AddModuleMessage(SharedLocalizer["Message.InfoRequired"], MessageType.Warning); + } + } - private void HideModal() - { - _package = null; - StateHasChanged(); - } + private void HideModal() + { + _package = null; + StateHasChanged(); + } - private string TranslationAvailable(string packagename, string version) - { - if (_packages != null) - { - var package = _packages.Where(item => item.PackageId == packagename).FirstOrDefault(); - if (package != null) - { - if (string.IsNullOrEmpty(version)) - { - return "install"; - } - else - { - if (Version.Parse(package.Version).CompareTo(Version.Parse(version)) > 0) - { - return "upgrade"; - } - } - } - } - return ""; - } + private string TranslationAvailable(string packagename, string version) + { + if (_packages != null) + { + var package = _packages.Where(item => item.PackageId == packagename).FirstOrDefault(); + if (package != null) + { + if (string.IsNullOrEmpty(version)) + { + return "install"; + } + else + { + if (Version.Parse(package.Version).CompareTo(Version.Parse(version)) > 0) + { + return "upgrade"; + } + } + } + } + return ""; + } - private async Task GetPackage(string packagename) - { - var version = _packages.Where(item => item.PackageId == packagename).FirstOrDefault().Version; - try - { - _package = await PackageService.GetPackageAsync(packagename, version); - StateHasChanged(); - } + private async Task GetPackage(string packagename) + { + var version = _packages.Where(item => item.PackageId == packagename).FirstOrDefault().Version; + try + { + _package = await PackageService.GetPackageAsync(packagename, version); + if (_package != null) + { + StateHasChanged(); + } + else + { + await logger.LogError("Error Getting Package {PackageId} {Version}", packagename, version); + AddModuleMessage(Localizer["Error.Translation.Download"], MessageType.Error); + } + } catch (Exception ex) { await logger.LogError(ex, "Error Getting Package {PackageId} {Version}", packagename, version); diff --git a/Oqtane.Client/Modules/Admin/Themes/Add.razor b/Oqtane.Client/Modules/Admin/Themes/Add.razor index 76c95d83..7c4bd170 100644 --- a/Oqtane.Client/Modules/Admin/Themes/Add.razor +++ b/Oqtane.Client/Modules/Admin/Themes/Add.razor @@ -217,8 +217,13 @@ } _packageid = package.PackageId; _version = package.Version; + StateHasChanged(); + } + else + { + await logger.LogError("Error Getting Package {PackageId} {Version}", packageid, version); + AddModuleMessage(Localizer["Error.Theme.Download"], MessageType.Error); } - StateHasChanged(); } catch (Exception ex) {