From ed7c89409f02665e8ee27dfb77a24898d0808c46 Mon Sep 17 00:00:00 2001 From: Harold Putman Date: Tue, 15 Sep 2020 09:42:36 -0400 Subject: [PATCH] Use orElseGet for basename rendition `getRenditionForBaseName()` was being called whether or not getRenditionForName found the rendition. Using orElseGet will only call it if the rendition was not found. --- .../cacheablerenditions/servlets/CacheableRenditionServlet.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/com/avionos/aem/assets/cacheablerenditions/servlets/CacheableRenditionServlet.java b/src/main/java/com/avionos/aem/assets/cacheablerenditions/servlets/CacheableRenditionServlet.java index eaba35b..f3296b6 100644 --- a/src/main/java/com/avionos/aem/assets/cacheablerenditions/servlets/CacheableRenditionServlet.java +++ b/src/main/java/com/avionos/aem/assets/cacheablerenditions/servlets/CacheableRenditionServlet.java @@ -76,7 +76,7 @@ protected void doGet(final SlingHttpServletRequest request, final SlingHttpServl } private Rendition getRendition(final Asset asset, final String renditionName) { - return getRenditionForName(asset, renditionName).orElse(getRenditionForBaseName(asset, renditionName)); + return getRenditionForName(asset, renditionName).orElseGet(() -> getRenditionForBaseName(asset, renditionName)); } private Rendition getRenditionForBaseName(final Asset asset, final String renditionName) {