From f702f2670b23127f24ba8970f1794dd365f64cd6 Mon Sep 17 00:00:00 2001 From: gbrodman Date: Tue, 18 May 2021 12:20:04 -0400 Subject: [PATCH] Use a flatMap in StaticPremiumPricingEngine (#1166) * Use a flatMap in StaticPremiumPricingEngine --- .../model/pricing/StaticPremiumListPricingEngine.java | 9 ++++----- .../java/google/registry/model/registry/Registry.java | 1 - 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/core/src/main/java/google/registry/model/pricing/StaticPremiumListPricingEngine.java b/core/src/main/java/google/registry/model/pricing/StaticPremiumListPricingEngine.java index 620a4730e..0e626c01e 100644 --- a/core/src/main/java/google/registry/model/pricing/StaticPremiumListPricingEngine.java +++ b/core/src/main/java/google/registry/model/pricing/StaticPremiumListPricingEngine.java @@ -38,11 +38,10 @@ public final class StaticPremiumListPricingEngine implements PremiumPricingEngin String tld = getTldFromDomainName(fullyQualifiedDomainName); String label = InternetDomainName.from(fullyQualifiedDomainName).parts().get(0); Registry registry = Registry.get(checkNotNull(tld, "tld")); - Optional premiumPrice = Optional.empty(); - if (registry.getPremiumList().isPresent()) { - premiumPrice = - PremiumListDao.getPremiumPrice(registry.getPremiumList().get().getName(), label); - } + Optional premiumPrice = + registry + .getPremiumList() + .flatMap(listKey -> PremiumListDao.getPremiumPrice(listKey.getName(), label)); return DomainPrices.create( premiumPrice.isPresent(), premiumPrice.orElse(registry.getStandardCreateCost()), diff --git a/core/src/main/java/google/registry/model/registry/Registry.java b/core/src/main/java/google/registry/model/registry/Registry.java index 8de68fecd..42b886859 100644 --- a/core/src/main/java/google/registry/model/registry/Registry.java +++ b/core/src/main/java/google/registry/model/registry/Registry.java @@ -606,7 +606,6 @@ public class Registry extends ImmutableObject implements Buildable, DatastoreAnd return anchorTenantAddGracePeriodLength; } - @Nullable public Optional> getPremiumList() { return Optional.ofNullable(premiumList); }