refactoring: use map instead of peek in some cases

CDCgov · May 15, 2024 · 79e8371 · 79e8371
1 parent 2729fed
commit 79e8371
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/shared/src/main/java/gov/hhs/cdc/trustedintermediary/external/hapi/HapiHelper.java b/shared/src/main/java/gov/hhs/cdc/trustedintermediary/external/hapi/HapiHelper.java
@@ -93,8 +93,8 @@ public static MessageHeader findOrInitializeMessageHeader(Bundle bundle) {
     public static void updateOrganizationIdentifierValue(Bundle bundle, String newValue) {
         bundle.getEntry().stream()
                 .map(Bundle.BundleEntryComponent::getResource)
-                .filter(resource -> resource instanceof Patient)
-                .map(resource -> (Patient) resource)
+                .filter(Patient.class::isInstance)
+                .map(Patient.class::cast)
                 .flatMap(patient -> patient.getIdentifier().stream())
                 .map(
                         identifier ->
@@ -117,7 +117,7 @@ public static void updateOrganizationIdentifierValue(Bundle bundle, String newVa
                                                         LOGGER.logInfo(
                                                                 "Updating Organization identifier from: "
                                                                         + orgIdentifier.getValue()))
-                                        .peek(orgIdentifier -> orgIdentifier.setValue(newValue)))
+                                        .map(orgIdentifier -> orgIdentifier.setValue(newValue)))
                 .forEach(
                         orgIdentifier ->
                                 LOGGER.logInfo(
@@ -144,8 +144,8 @@ private static Optional<Organization> getOrganizationFromAssigner(
 
         return bundle.getEntry().stream()
                 .map(Bundle.BundleEntryComponent::getResource)
-                .filter(resource -> resource instanceof Organization)
-                .map(resource -> (Organization) resource)
+                .filter(Organization.class::isInstance)
+                .map(Organization.class::cast)
                 .peek(org -> LOGGER.logInfo("Checking organization with ID: " + org.getId()))
                 .filter(
                         org -> {