diff --git a/Oqtane.Client/Modules/Admin/Languages/Index.razor b/Oqtane.Client/Modules/Admin/Languages/Index.razor index 947cddc7..e98db7d9 100644 --- a/Oqtane.Client/Modules/Admin/Languages/Index.razor +++ b/Oqtane.Client/Modules/Admin/Languages/Index.razor @@ -84,7 +84,7 @@ else var package = _packages.Where(item => item.PackageId == (Constants.PackageId + ".Client." + code)).FirstOrDefault(); if (package != null) { - upgradeavailable = (Version.Parse(package.Version).CompareTo(Version.Parse(Constants.Version)) > 0); + upgradeavailable = (Version.Parse(package.Version).CompareTo(Version.Parse(Constants.Version)) == 0); } } diff --git a/Oqtane.Client/Modules/Admin/ModuleDefinitions/Index.razor b/Oqtane.Client/Modules/Admin/ModuleDefinitions/Index.razor index 305b8542..049582c6 100644 --- a/Oqtane.Client/Modules/Admin/ModuleDefinitions/Index.razor +++ b/Oqtane.Client/Modules/Admin/ModuleDefinitions/Index.razor @@ -50,9 +50,12 @@ else @((MarkupString)PurchaseLink(context.PackageName)) - @if (UpgradeAvailable(context.PackageName, context.Version, out var upgradeToVersion)) + @{ + var version = UpgradeAvailable(context.PackageName, context.Version); + } + @if (version != context.Version) { - + } @@ -60,64 +63,60 @@ else } @code { - private List _moduleDefinitions; - private List _packages; + private List _moduleDefinitions; + private List _packages; - public override SecurityAccessLevel SecurityAccessLevel => SecurityAccessLevel.Host; + public override SecurityAccessLevel SecurityAccessLevel => SecurityAccessLevel.Host; - protected override async Task OnParametersSetAsync() - { - try - { - _moduleDefinitions = await ModuleDefinitionService.GetModuleDefinitionsAsync(PageState.Site.SiteId); - _packages = await PackageService.GetPackagesAsync("module"); + protected override async Task OnParametersSetAsync() + { + try + { + _moduleDefinitions = await ModuleDefinitionService.GetModuleDefinitionsAsync(PageState.Site.SiteId); + _packages = await PackageService.GetPackagesAsync("module"); + } + catch (Exception ex) + { + if (_moduleDefinitions == null) + { + await logger.LogError(ex, "Error Loading Modules {Error}", ex.Message); + AddModuleMessage(Localizer["Error.Module.Load"], MessageType.Error); + } + } + } + + private string PurchaseLink(string packagename) + { + string link = ""; + if (!string.IsNullOrEmpty(packagename) && _packages != null) + { + var package = _packages.Where(item => item.PackageId == packagename).FirstOrDefault(); + if (package != null) + { + if (package.ExpiryDate != null && package.ExpiryDate.Value.Date != DateTime.MaxValue.Date) + { + link += "" + package.ExpiryDate.Value.Date.ToString("MMM dd, yyyy") + ""; + if (!string.IsNullOrEmpty(package.PaymentUrl)) + { + link += "  " + SharedLocalizer["Extend"] + ""; + } + } + } + } + return link; + } + + private string UpgradeAvailable(string packagename, string version) + { + if (!string.IsNullOrEmpty(packagename) && _packages != null) + { + var package = _packages.Where(item => item.PackageId == packagename).FirstOrDefault(); + if (package != null && Version.Parse(package.Version).CompareTo(Version.Parse(version)) > 0) + { + return package.Version; + } } - catch (Exception ex) - { - if (_moduleDefinitions == null) - { - await logger.LogError(ex, "Error Loading Modules {Error}", ex.Message); - AddModuleMessage(Localizer["Error.Module.Load"], MessageType.Error); - } - } - } - - private string PurchaseLink(string packagename) - { - string link = ""; - if (!string.IsNullOrEmpty(packagename) && _packages != null) - { - var package = _packages.Where(item => item.PackageId == packagename).FirstOrDefault(); - if (package != null) - { - if (package.ExpiryDate != null && package.ExpiryDate.Value.Date != DateTime.MaxValue.Date) - { - link += "" + package.ExpiryDate.Value.Date.ToString("MMM dd, yyyy") + ""; - if (!string.IsNullOrEmpty(package.PaymentUrl)) - { - link += "  " + SharedLocalizer["Extend"] + ""; - } - } - } - } - return link; - } - - private bool UpgradeAvailable(string packagename, string version, out string upgradeToVersion) - { - var upgradeavailable = false; - upgradeToVersion = ""; - if (!string.IsNullOrEmpty(packagename) && _packages != null) - { - var package = _packages.Where(item => item.PackageId == packagename).FirstOrDefault(); - if (package != null) - { - upgradeToVersion = package.Version; - upgradeavailable = (Version.Parse(upgradeToVersion).CompareTo(Version.Parse(version)) > 0); - } - - } - return upgradeavailable; + return version; } private async Task DownloadModule(string packagename, string version) diff --git a/Oqtane.Client/Modules/Admin/Themes/Index.razor b/Oqtane.Client/Modules/Admin/Themes/Index.razor index 5c089dfa..90213b9e 100644 --- a/Oqtane.Client/Modules/Admin/Themes/Index.razor +++ b/Oqtane.Client/Modules/Admin/Themes/Index.razor @@ -40,9 +40,12 @@ else @((MarkupString)PurchaseLink(context.PackageName)) - @if (UpgradeAvailable(context.PackageName, context.Version)) + @{ + var version = UpgradeAvailable(context.PackageName, context.Version); + } + @if (version != context.Version) { - + } @@ -94,18 +97,17 @@ else return link; } - private bool UpgradeAvailable(string packagename, string version) - { - var upgradeavailable = false; - if (!string.IsNullOrEmpty(packagename) && _packages != null) - { - var package = _packages.Where(item => item.PackageId == packagename).FirstOrDefault(); - if (package != null) - { - upgradeavailable = (Version.Parse(package.Version).CompareTo(Version.Parse(version)) > 0); - } + private string UpgradeAvailable(string packagename, string version) + { + if (!string.IsNullOrEmpty(packagename) && _packages != null) + { + var package = _packages.Where(item => item.PackageId == packagename).FirstOrDefault(); + if (package != null && Version.Parse(package.Version).CompareTo(Version.Parse(version)) > 0) + { + return package.Version; + } } - return upgradeavailable; + return version; } private async Task DownloadTheme(string packagename, string version)