Use Optional.isEmpty() 63/95263/3
authorJaroslav Tóth <jtoth@frinx.io>
Mon, 22 Feb 2021 08:04:13 +0000 (09:04 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Tue, 11 May 2021 11:36:46 +0000 (13:36 +0200)
Java 11 introduced Optional.isEmpty(), which allows us to convert
!Optional.isPresent() callsites. Perform this conversion across the
entire repostitory.

Change-Id: I515a550bb6aa56b6d89cea020f5399ca27884833
Signed-off-by: Jaroslav Tóth <jtoth@frinx.io>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
34 files changed:
netconf/mdsal-netconf-connector/src/main/java/org/opendaylight/netconf/mdsal/connector/TransactionProvider.java
netconf/mdsal-netconf-connector/src/main/java/org/opendaylight/netconf/mdsal/connector/ops/AbstractConfigOperation.java
netconf/mdsal-netconf-connector/src/main/java/org/opendaylight/netconf/mdsal/connector/ops/AbstractEdit.java
netconf/mdsal-netconf-connector/src/main/java/org/opendaylight/netconf/mdsal/connector/ops/CopyConfig.java
netconf/mdsal-netconf-connector/src/main/java/org/opendaylight/netconf/mdsal/connector/ops/RuntimeRpc.java
netconf/mdsal-netconf-connector/src/main/java/org/opendaylight/netconf/mdsal/connector/ops/get/FilterContentValidator.java
netconf/mdsal-netconf-connector/src/main/java/org/opendaylight/netconf/mdsal/connector/ops/get/Get.java
netconf/mdsal-netconf-connector/src/main/java/org/opendaylight/netconf/mdsal/connector/ops/get/GetConfig.java
netconf/mdsal-netconf-notification/src/main/java/org/opendaylight/netconf/mdsal/notification/impl/ops/CreateSubscription.java
netconf/netconf-api/src/main/java/org/opendaylight/netconf/api/messages/NetconfHelloMessage.java
netconf/netconf-api/src/main/java/org/opendaylight/netconf/api/xml/XmlElement.java
netconf/netconf-api/src/main/java/org/opendaylight/netconf/api/xml/XmlUtil.java
netconf/netconf-topology-singleton/src/main/java/org/opendaylight/netconf/topology/singleton/impl/actors/NetconfDataTreeServiceActor.java
netconf/netconf-topology-singleton/src/main/java/org/opendaylight/netconf/topology/singleton/impl/actors/ReadAdapter.java
netconf/netconf-topology-singleton/src/test/java/org/opendaylight/netconf/topology/singleton/impl/MountPointEndToEndTest.java
netconf/netconf-util/src/main/java/org/opendaylight/netconf/util/messages/NetconfMessageUtil.java
netconf/netconf-util/src/main/java/org/opendaylight/netconf/util/messages/SubtreeFilter.java
netconf/sal-netconf-connector/src/main/java/org/opendaylight/netconf/sal/connect/netconf/DeviceMountPointContext.java
netconf/sal-netconf-connector/src/main/java/org/opendaylight/netconf/sal/connect/netconf/LibraryModulesSchemas.java
netconf/sal-netconf-connector/src/main/java/org/opendaylight/netconf/sal/connect/netconf/NetconfDevice.java
netconf/sal-netconf-connector/src/main/java/org/opendaylight/netconf/sal/connect/netconf/NetconfStateSchemas.java
restconf/restconf-nb-bierman02/src/main/java/org/opendaylight/netconf/sal/rest/impl/JsonNormalizedNodeBodyReader.java
restconf/restconf-nb-bierman02/src/main/java/org/opendaylight/netconf/sal/rest/impl/JsonToPatchBodyReader.java
restconf/restconf-nb-bierman02/src/main/java/org/opendaylight/netconf/sal/rest/impl/XmlNormalizedNodeBodyReader.java
restconf/restconf-nb-bierman02/src/main/java/org/opendaylight/netconf/sal/rest/impl/XmlToPatchBodyReader.java
restconf/restconf-nb-bierman02/src/main/java/org/opendaylight/netconf/sal/restconf/impl/BrokerFacade.java
restconf/restconf-nb-bierman02/src/main/java/org/opendaylight/netconf/sal/restconf/impl/ControllerContext.java
restconf/restconf-nb-bierman02/src/main/java/org/opendaylight/netconf/sal/restconf/impl/RestconfImpl.java
restconf/restconf-nb-bierman02/src/main/java/org/opendaylight/netconf/sal/streams/listeners/ListenerAdapter.java
restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/rests/services/impl/CreateStreamUtil.java
restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/rests/services/impl/RestconfDataStreamServiceImpl.java
restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/rests/utils/RestconfInvokeOperationsUtil.java
restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/utils/parser/ParserIdentifier.java
restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/utils/parser/YangInstanceIdentifierDeserializer.java

index 33b8fce4813c5a999b955baf9b06f423a05761cc..e41f019c9fdb37ceed08785023127ab4edb1d880 100644 (file)
@@ -74,7 +74,7 @@ public class TransactionProvider implements AutoCloseable {
                 ErrorType.PROTOCOL, ErrorTag.OPERATION_NOT_SUPPORTED, ErrorSeverity.ERROR);
         }
 
-        if (!getCandidateTransaction().isPresent()) {
+        if (getCandidateTransaction().isEmpty()) {
             // Validating empty transaction, just return true
             LOG.debug("Validating empty candidate transaction for session {}", netconfSessionIdForReporting);
             return;
@@ -93,7 +93,7 @@ public class TransactionProvider implements AutoCloseable {
     }
 
     public synchronized boolean commitTransaction() throws DocumentedException {
-        if (!getCandidateTransaction().isPresent()) {
+        if (getCandidateTransaction().isEmpty()) {
             //making empty commit without prior opened transaction, just return true
             LOG.debug("Making commit without open candidate transaction for session {}", netconfSessionIdForReporting);
             return true;
@@ -119,7 +119,7 @@ public class TransactionProvider implements AutoCloseable {
     public synchronized void abortTransaction() {
         LOG.debug("Aborting current candidateTransaction");
         final Optional<DOMDataTreeReadWriteTransaction> otx = getCandidateTransaction();
-        if (!otx.isPresent()) {
+        if (otx.isEmpty()) {
             LOG.warn("discard-changes triggerd on an empty transaction for session: {}", netconfSessionIdForReporting);
             return;
         }
index 9e3989cc4996cafa8658605089c850fc53a16cdc..f6b827c25296bf565ef889bfed0739723cb46029 100644 (file)
@@ -58,7 +58,7 @@ abstract class AbstractConfigOperation extends AbstractSingletonNetconfOperation
         }
 
         final Optional<XmlElement> urlElement = parent.getOnlyChildElementOptionally(URL_KEY);
-        if (!urlElement.isPresent()) {
+        if (urlElement.isEmpty()) {
             throw new DocumentedException("Invalid RPC, neither <config> not <url> element is present",
                 DocumentedException.ErrorType.PROTOCOL,
                 DocumentedException.ErrorTag.MISSING_ELEMENT,
index 2576641b8386f889c89d14cf3338065e0c30caac..acf55aab5bcf1bea52adda1c480120b216932d8b 100644 (file)
@@ -87,7 +87,7 @@ abstract class AbstractEdit extends AbstractConfigOperation {
         final String elementName = element.getName();
         final Optional<DataSchemaNode> schemaNode = module.findDataChildByName(QName.create(module.getQNameModule(),
                     element.getName()));
-        if (!schemaNode.isPresent()) {
+        if (schemaNode.isEmpty()) {
             throw new DocumentedException(
                 "Unable to find node " + elementName + " with namespace: " + namespace + "in module: "
                     + module.toString(),
index 939fcb1d3cdbe417ea017bd98a1fb9e245c52317..341f84bb3a25e5a7c6160289e1aa88e88e36417d 100644 (file)
@@ -121,7 +121,7 @@ public final class CopyConfig extends AbstractEdit {
 
     private static XmlElement getSourceElement(final XmlElement parent) throws DocumentedException {
         final Optional<XmlElement> sourceElement = parent.getOnlyChildElementOptionally(SOURCE_KEY);
-        if (!sourceElement.isPresent()) {
+        if (sourceElement.isEmpty()) {
             throw new DocumentedException("<source> element is missing",
                 DocumentedException.ErrorType.PROTOCOL,
                 DocumentedException.ErrorTag.MISSING_ELEMENT,
index 443b8c64fd32ddf95a706c5e557d157baf6bcce3..614d4b66c4b90faa1acf3b4c74460c7d4fe0bc50 100644 (file)
@@ -82,7 +82,7 @@ public class RuntimeRpc extends AbstractSingletonNetconfOperation {
         final URI namespaceURI = createNsUri(namespace);
         final Optional<? extends Module> module = getModule(namespaceURI);
 
-        if (!module.isPresent()) {
+        if (module.isEmpty()) {
             LOG.debug("Cannot handle rpc: {}, {}", netconfOperationName, namespace);
             return HandlingPriority.CANNOT_HANDLE;
         }
@@ -135,7 +135,7 @@ public class RuntimeRpc extends AbstractSingletonNetconfOperation {
         final URI namespaceURI = createNsUri(netconfOperationNamespace);
         final Optional<? extends Module> moduleOptional = getModule(namespaceURI);
 
-        if (!moduleOptional.isPresent()) {
+        if (moduleOptional.isEmpty()) {
             throw new DocumentedException("Unable to find module in Schema Context with namespace and name : "
                         + namespaceURI + " " + netconfOperationName + schemaContext.getCurrentContext(),
                     ErrorType.APPLICATION, ErrorTag.BAD_ELEMENT, ErrorSeverity.ERROR);
@@ -144,7 +144,7 @@ public class RuntimeRpc extends AbstractSingletonNetconfOperation {
         final Optional<RpcDefinition> rpcDefinitionOptional = getRpcDefinitionFromModule(moduleOptional.get(),
                 namespaceURI, netconfOperationName);
 
-        if (!rpcDefinitionOptional.isPresent()) {
+        if (rpcDefinitionOptional.isEmpty()) {
             throw new DocumentedException(
                     "Unable to find RpcDefinition with namespace and name : "
                         + namespaceURI + " " + netconfOperationName,
index 0ec329012ebb72008a192edfe0ee8860ccee168a..bcffae92161379e6467fdfc01fa0ce9a4224400a 100644 (file)
@@ -198,7 +198,7 @@ public class FilterContentValidator {
         for (final QName qualifiedName : keyDefinition) {
             final Optional<XmlElement> childElements =
                     current.getOnlyChildElementOptionally(qualifiedName.getLocalName());
-            if (!childElements.isPresent()) {
+            if (childElements.isEmpty()) {
                 return Collections.emptyMap();
             }
             final Optional<String> keyValue = childElements.get().getOnlyTextContentOptionally();
index 6aaa8d80020d560b613e9950e2327c4e543f2bb7..4751c3323b04e630481ee6bb65585f486909e823 100644 (file)
@@ -45,7 +45,7 @@ public class Get extends AbstractGet {
             throws DocumentedException {
 
         final Optional<YangInstanceIdentifier> dataRootOptional = getDataRootFromFilter(operationElement);
-        if (!dataRootOptional.isPresent()) {
+        if (dataRootOptional.isEmpty()) {
             return document.createElement(XmlNetconfConstants.DATA_KEY);
         }
 
@@ -57,7 +57,7 @@ public class Get extends AbstractGet {
                     LogicalDatastoreType.OPERATIONAL, dataRoot).get();
             transactionProvider.abortRunningTransaction(rwTx);
 
-            if (!normalizedNodeOptional.isPresent()) {
+            if (normalizedNodeOptional.isEmpty()) {
                 return document.createElement(XmlNetconfConstants.DATA_KEY);
             }
 
index 26207a9f11a241964eba206d9e9c84b79d6cfd49..aadf1b9a884ee4c3f3a3eb203482125ba85dbbca 100644 (file)
@@ -54,7 +54,7 @@ public class GetConfig extends AbstractGet {
         }
 
         final Optional<YangInstanceIdentifier> dataRootOptional = getDataRootFromFilter(operationElement);
-        if (!dataRootOptional.isPresent()) {
+        if (dataRootOptional.isEmpty()) {
             return document.createElement(XmlNetconfConstants.DATA_KEY);
         }
 
@@ -71,7 +71,7 @@ public class GetConfig extends AbstractGet {
                 transactionProvider.abortRunningTransaction(rwTx);
             }
 
-            if (!normalizedNodeOptional.isPresent()) {
+            if (normalizedNodeOptional.isEmpty()) {
                 return document.createElement(XmlNetconfConstants.DATA_KEY);
             }
 
index 3f3d14ed96fabe49f555dbd3ee7be59fa640b810..685ba761881c2cf9521dd096a0d25e1b0918dac0 100644 (file)
@@ -69,12 +69,12 @@ public class CreateSubscription extends AbstractSingletonNetconfOperation
         // Replay not supported
         final Optional<XmlElement> startTime =
                 operationElement.getOnlyChildElementWithSameNamespaceOptionally("startTime");
-        checkArgument(!startTime.isPresent(), "StartTime element not yet supported");
+        checkArgument(startTime.isEmpty(), "StartTime element not yet supported");
 
         // Stop time not supported
         final Optional<XmlElement> stopTime =
                 operationElement.getOnlyChildElementWithSameNamespaceOptionally("stopTime");
-        checkArgument(!stopTime.isPresent(), "StopTime element not yet supported");
+        checkArgument(stopTime.isEmpty(), "StopTime element not yet supported");
 
         final StreamNameType streamNameType = parseStreamIfPresent(operationElement);
 
index d46336a96296cc255299edad1cd9361c7c694466..564847304bb761059127103a7ff9dba02537bb74 100644 (file)
@@ -97,7 +97,7 @@ public final class NetconfHelloMessage extends NetconfMessage {
 
         final Optional<String> optNamespace = element.getNamespaceOptionally();
         // accept even if hello has no namespace
-        return !optNamespace.isPresent()
+        return optNamespace.isEmpty()
                 || XmlNetconfConstants.URN_IETF_PARAMS_XML_NS_NETCONF_BASE_1_0.equals(optNamespace.get());
     }
 }
index 4a92fddb52b09bb00b890d1715bcf9659e14b7f3..a07a785f1dcd1c0753a8162a0a0cc18a1356658e 100644 (file)
@@ -392,7 +392,7 @@ public final class XmlElement {
 
     public String getNamespace() throws MissingNameSpaceException {
         Optional<String> namespaceURI = getNamespaceOptionally();
-        if (!namespaceURI.isPresent()) {
+        if (namespaceURI.isEmpty()) {
             throw new MissingNameSpaceException(String.format("No namespace defined for %s", this),
                     DocumentedException.ErrorType.APPLICATION,
                     DocumentedException.ErrorTag.OPERATION_FAILED,
index d3c40614280e9a28c099be6d8eb37d0d3715d707..84548313176f326ed6fb071c7347a86f1ec75508 100644 (file)
@@ -122,7 +122,7 @@ public final class XmlUtil {
 
     public static Element createElement(final Document document, final String qname,
             final Optional<String> namespaceURI) {
-        if (!namespaceURI.isPresent()) {
+        if (namespaceURI.isEmpty()) {
             return document.createElement(qname);
         }
 
index e6923c22b4c0481ee1684e843c4491781cf8271e..7f007d146fc279e433d5f7c53f1ea60f03625b89 100644 (file)
@@ -187,7 +187,7 @@ public final class NetconfDataTreeServiceActor extends UntypedAbstractActor {
         Futures.addCallback(feature, new FutureCallback<>() {
             @Override
             public void onSuccess(final Optional<NormalizedNode<?, ?>> result) {
-                if (!result.isPresent()) {
+                if (result.isEmpty()) {
                     sender.tell(new EmptyReadResponse(), self);
                     return;
                 }
index b34f846d9e5edac62798ba0789c491d7d39e47f9..2a56fe7e2aa53860f6a755e426dc93c371b3d91c 100644 (file)
@@ -51,7 +51,7 @@ class ReadAdapter {
         tx.read(store, path).addCallback(new FutureCallback<Optional<NormalizedNode<?, ?>>>() {
             @Override
             public void onSuccess(final Optional<NormalizedNode<?, ?>> result) {
-                if (!result.isPresent()) {
+                if (result.isEmpty()) {
                     sender.tell(new EmptyReadResponse(), self);
                     return;
                 }
index 59916be1a9c22d6ac1f9b06acd0fa38378d4ad6b..83bd1d80ea67e5cdabefa24357db22b82e9744f5 100644 (file)
@@ -684,7 +684,7 @@ public class MountPointEndToEndTest extends AbstractBaseSchemasTest {
 
     private void awaitMountPointNotPresent(final DOMMountPointService mountPointService) {
         await().atMost(5, TimeUnit.SECONDS).until(
-            () -> !mountPointService.getMountPoint(yangNodeInstanceId).isPresent());
+            () -> mountPointService.getMountPoint(yangNodeInstanceId).isEmpty());
     }
 
     private static DOMDataBroker getDOMDataBroker(final DOMMountPoint mountPoint) {
index a32cd277b6873ca69b354b461ff3de99dcf53e76..d61b3c6d274fb625ebdf924b336f37f64ae84c64 100644 (file)
@@ -78,7 +78,7 @@ public final class NetconfMessageUtil {
         // Extract child element <capabilities> from <hello> with or without(fallback) the same namespace
         Optional<XmlElement> capabilitiesElement = responseElement
                 .getOnlyChildElementWithSameNamespaceOptionally(XmlNetconfConstants.CAPABILITIES);
-        if (!capabilitiesElement.isPresent()) {
+        if (capabilitiesElement.isEmpty()) {
             capabilitiesElement = responseElement.getOnlyChildElementOptionally(XmlNetconfConstants.CAPABILITIES);
         }
 
index 3b4043aa20980fc9ba54a1f0560443dd6945daf0..49fbc559be7c49a7da429603699800ed3d36ce36 100644 (file)
@@ -43,7 +43,7 @@ public final class SubtreeFilter {
             Optional<XmlElement> maybeFilter = operationNameAndNamespace.getOperationElement()
                     .getOnlyChildElementOptionally(XmlNetconfConstants.FILTER,
                             XmlNetconfConstants.URN_IETF_PARAMS_XML_NS_NETCONF_BASE_1_0);
-            if (!maybeFilter.isPresent()) {
+            if (maybeFilter.isEmpty()) {
                 return rpcReply;
             }
             XmlElement filter = maybeFilter.get();
index 0783ad6b64992e947651a2c2d2fd4b04cb775691..bc8703d7d503b9285cf9d87fdd7c17250a55fe14 100644 (file)
@@ -67,7 +67,7 @@ final class DeviceMountPointContext extends AbstractEffectiveModelContextProvide
 
     static MountPointContext create(final MountPointContext emptyContext, final ContainerNode mountData) {
         final Optional<DataContainerChild<?, ?>> optMountPoint = mountData.getChild(MOUNT_POINT);
-        if (!optMountPoint.isPresent()) {
+        if (optMountPoint.isEmpty()) {
             LOG.debug("mount-point list not present in {}", mountData);
             return emptyContext;
         }
@@ -102,7 +102,7 @@ final class DeviceMountPointContext extends AbstractEffectiveModelContextProvide
             final ChoiceNode schemaRef = (ChoiceNode) child;
 
             final Optional<DataContainerChild<?, ?>> maybeShared = schemaRef.getChild(SHARED_SCHEMA);
-            if (!maybeShared.isPresent()) {
+            if (maybeShared.isEmpty()) {
                 LOG.debug("Ignoring non-shared mountpoint entry {}", entry);
                 continue;
             }
index a82b39b7ce20d9aa8e3b640af047857b4c331ac2..6ef857742968ea71292f57265c606fd94bb9b4f5 100644 (file)
@@ -264,7 +264,7 @@ public final class LibraryModulesSchemas implements NetconfDeviceSchemas {
             LOG.warn("Unable to download yang library from {}", connection.getURL(), e);
         }
 
-        if (!optionalModulesStateNode.isPresent()) {
+        if (optionalModulesStateNode.isEmpty()) {
             return new LibraryModulesSchemas(ImmutableMap.of());
         }
 
index f48d57812fc2cf40e47ba9d63aadcc8a8acb4a89..ecbee257ce6b93c7522af64239fda1a52409ef6a 100644 (file)
@@ -313,7 +313,7 @@ public class NetconfDevice
     private ListenableFuture<MountPointContext> createMountPointContext(final EffectiveModelContext schemaContext,
             final BaseSchema baseSchema, final NetconfDeviceCommunicator listener) {
         final MountPointContext emptyContext = new EmptyMountPointContext(schemaContext);
-        if (!schemaContext.findModule(SchemaMountConstants.RFC8528_MODULE).isPresent()) {
+        if (schemaContext.findModule(SchemaMountConstants.RFC8528_MODULE).isEmpty()) {
             return Futures.immediateFuture(emptyContext);
         }
 
index 92705dccedacb4377961da6b479bc7095dcfbe6d..496809cf68e3b6d22c782b122b2e063fe400a1ee 100644 (file)
@@ -145,7 +145,7 @@ public final class NetconfStateSchemas implements NetconfDeviceSchemas {
 
         final Optional<? extends NormalizedNode<?, ?>> optSchemasNode = findSchemasNode(schemasNodeResult.getResult(),
                 schemaContext);
-        if (!optSchemasNode.isPresent()) {
+        if (optSchemasNode.isEmpty()) {
             LOG.warn("{}: Unable to detect available schemas, get to {} was empty", id, STATE_SCHEMAS_IDENTIFIER);
             return EMPTY;
         }
@@ -174,9 +174,7 @@ public final class NetconfStateSchemas implements NetconfDeviceSchemas {
         for (final MapEntryNode schemaNode : ((MapNode) child.get()).getValue()) {
             final Optional<RemoteYangSchema> fromCompositeNode =
                     RemoteYangSchema.createFromNormalizedNode(id, schemaNode);
-            if (fromCompositeNode.isPresent()) {
-                availableYangSchemas.add(fromCompositeNode.get());
-            }
+            fromCompositeNode.ifPresent(availableYangSchemas::add);
         }
 
         return new NetconfStateSchemas(availableYangSchemas);
@@ -188,7 +186,7 @@ public final class NetconfStateSchemas implements NetconfDeviceSchemas {
             return Optional.empty();
         }
         final Optional<DataContainerChild<?, ?>> rpcResultOpt = ((ContainerNode)result).getChild(NETCONF_DATA_NODEID);
-        if (!rpcResultOpt.isPresent()) {
+        if (rpcResultOpt.isEmpty()) {
             return Optional.empty();
         }
 
@@ -206,7 +204,7 @@ public final class NetconfStateSchemas implements NetconfDeviceSchemas {
 
         final Optional<DataContainerChild<?, ?>> nStateNode = ((DataContainerNode<?>) dataNode).getChild(
             toId(NetconfState.QNAME));
-        if (!nStateNode.isPresent()) {
+        if (nStateNode.isEmpty()) {
             return Optional.empty();
         }
 
@@ -246,7 +244,7 @@ public final class NetconfStateSchemas implements NetconfDeviceSchemas {
 
             childNode = NetconfMessageTransformUtil.IETF_NETCONF_MONITORING_SCHEMA_NAMESPACE;
             final Optional<String> namespaceValue = getSingleChildNodeValue(schemaNode, childNode);
-            if (!namespaceValue.isPresent()) {
+            if (namespaceValue.isEmpty()) {
                 LOG.warn("{}: Ignoring schema due to missing namespace", id);
                 return Optional.empty();
             }
index 3b125cdf4018dbb594c29cd51e8cd3265778f8f6..5d3e1da407a6898e342cb9c905e40e88baed2be3 100644 (file)
@@ -104,7 +104,7 @@ public class JsonNormalizedNodeBodyReader
                                                   final InputStream entityStream, final boolean isPost)
             throws IOException {
         final Optional<InputStream> nonEmptyInputStreamOptional = RestUtil.isInputStreamEmpty(entityStream);
-        if (!nonEmptyInputStreamOptional.isPresent()) {
+        if (nonEmptyInputStreamOptional.isEmpty()) {
             return new NormalizedNodeContext(path, null);
         }
         final NormalizedNodeResult resultHolder = new NormalizedNodeResult();
index 0f308b03382b7eaf95c9dbd46a4afbcc05bce008..de744eeef818773169e8d07064b3ab8649978e00 100644 (file)
@@ -104,7 +104,7 @@ public class JsonToPatchBodyReader extends AbstractIdentifierAwareJaxRsProvider
     private PatchContext readFrom(final InstanceIdentifierContext<?> path, final InputStream entityStream)
             throws IOException {
         final Optional<InputStream> nonEmptyInputStreamOptional = RestUtil.isInputStreamEmpty(entityStream);
-        if (!nonEmptyInputStreamOptional.isPresent()) {
+        if (nonEmptyInputStreamOptional.isEmpty()) {
             return new PatchContext(path, null, null);
         }
 
index 215c1b4d7820c53816642fe7544acf4f2267fb8d..8246d3bc9f013a3a1c9c827ad987bb8a550f2681 100644 (file)
@@ -107,7 +107,7 @@ public class XmlNormalizedNodeBodyReader extends AbstractIdentifierAwareJaxRsPro
             XMLStreamException, ParserConfigurationException, URISyntaxException {
         final InstanceIdentifierContext<?> path = getInstanceIdentifierContext();
         final Optional<InputStream> nonEmptyInputStreamOptional = RestUtil.isInputStreamEmpty(entityStream);
-        if (!nonEmptyInputStreamOptional.isPresent()) {
+        if (nonEmptyInputStreamOptional.isEmpty()) {
             // represent empty nopayload input
             return new NormalizedNodeContext(path, null);
         }
index a1f3b4b1591636a89a8e3d606c6664a91f66bbc6..ef5be04b77866049f851e65befceff4af4c52071 100644 (file)
@@ -99,7 +99,7 @@ public class XmlToPatchBodyReader extends AbstractIdentifierAwareJaxRsProvider i
         try {
             final InstanceIdentifierContext<?> path = getInstanceIdentifierContext();
             final Optional<InputStream> nonEmptyInputStreamOptional = RestUtil.isInputStreamEmpty(entityStream);
-            if (!nonEmptyInputStreamOptional.isPresent()) {
+            if (nonEmptyInputStreamOptional.isEmpty()) {
                 // represent empty nopayload input
                 return new PatchContext(path, null, null);
             }
index 0eba3a783a802cfd10d0b08c20e341b46c585e3a..3a4c0ec1cf310555d8c64df1fad14f9574d2f46a 100644 (file)
@@ -548,8 +548,8 @@ public class BrokerFacade implements Closeable {
 
         try {
             final Optional<NormalizedNode<?, ?>> optional = transaction.read(datastore, path).get();
-            return !optional.isPresent() ? null : withDefa == null ? optional.get() :
-                prepareDataByParamWithDef(optional.get(), path, withDefa);
+            return optional.map(normalizedNode -> withDefa == null ? normalizedNode :
+                prepareDataByParamWithDef(normalizedNode, path, withDefa)).orElse(null);
         } catch (InterruptedException e) {
             LOG.warn("Error reading {} from datastore {}", path, datastore.name(), e);
             throw new RestconfDocumentedException("Error reading data.", e);
index 627a9d9db2d40e28a3d07cb54f4793c26ebb81e0..1ff954517348d7cbade7e3cde0dc150fbc8ec755 100644 (file)
@@ -583,7 +583,7 @@ public final class ControllerContext implements EffectiveModelContextListener, C
 
                 final YangInstanceIdentifier partialPath = this.dataNormalizer.toNormalized(builder.build());
                 final Optional<DOMMountPoint> mountOpt = this.mountService.getMountPoint(partialPath);
-                if (!mountOpt.isPresent()) {
+                if (mountOpt.isEmpty()) {
                     LOG.debug("Instance identifier to missing mount point: {}", partialPath);
                     throw new RestconfDocumentedException("Mount point does not exist.", ErrorType.PROTOCOL,
                             ErrorTag.DATA_MISSING);
index 24a14d7c1e97723dc928c5b01c499c9b575bfb7e..57b7d0d94ffc9e20c610f1bfb0847758e7d327ba 100644 (file)
@@ -404,7 +404,7 @@ public final class RestconfImpl implements RestconfService {
 
         if (mountPoint != null) {
             final Optional<DOMRpcService> mountRpcServices = mountPoint.getService(DOMRpcService.class);
-            if (!mountRpcServices.isPresent()) {
+            if (mountRpcServices.isEmpty()) {
                 LOG.debug("Error: Rpc service is missing.");
                 throw new RestconfDocumentedException("Rpc service is missing.");
             }
@@ -498,7 +498,7 @@ public final class RestconfImpl implements RestconfService {
         final ListenableFuture<? extends DOMRpcResult> response;
         if (mountPoint != null) {
             final Optional<DOMRpcService> mountRpcServices = mountPoint.getService(DOMRpcService.class);
-            if (!mountRpcServices.isPresent()) {
+            if (mountRpcServices.isEmpty()) {
                 throw new RestconfDocumentedException("Rpc service is missing.");
             }
             response = mountRpcServices.get().invokeRpc(rpc.getQName(), input);
@@ -1321,7 +1321,7 @@ public final class RestconfImpl implements RestconfService {
             final String paramName) {
         final Optional<DataContainerChild<? extends PathArgument, ?>> optAugNode = value.getChild(
             SAL_REMOTE_AUG_IDENTIFIER);
-        if (!optAugNode.isPresent()) {
+        if (optAugNode.isEmpty()) {
             return null;
         }
         final DataContainerChild<? extends PathArgument, ?> augNode = optAugNode.get();
@@ -1330,7 +1330,7 @@ public final class RestconfImpl implements RestconfService {
         }
         final Optional<DataContainerChild<? extends PathArgument, ?>> enumNode = ((AugmentationNode) augNode).getChild(
                 new NodeIdentifier(QName.create(SAL_REMOTE_AUGMENT, paramName)));
-        if (!enumNode.isPresent()) {
+        if (enumNode.isEmpty()) {
             return null;
         }
         final Object rawValue = enumNode.get().getValue();
index 8afc8dedae20ae02f6f27078b69ea9781c443967..b9e738927a8bc071c1cb8e6a52823f6be8a4d58d 100644 (file)
@@ -209,7 +209,7 @@ public class ListenerAdapter extends AbstractCommonSubscriber implements Cluster
                 break;
         }
 
-        if (!optionalNormalizedNode.isPresent()) {
+        if (optionalNormalizedNode.isEmpty()) {
             LOG.error("No node present in notification for {}", candidateNode);
             return;
         }
index d3dc392da0c27aad23dfaab7fb39f07200b441e6..590bb7bd72c1a97f708f6d55525fd8367d66764a 100644 (file)
@@ -184,7 +184,7 @@ final class CreateStreamUtil {
     private static <T> T parseEnum(final ContainerNode data, final Class<T> clazz, final String paramName) {
         final Optional<DataContainerChild<? extends PathArgument, ?>> optAugNode = data.getChild(
                 RestconfStreamsConstants.SAL_REMOTE_AUG_IDENTIFIER);
-        if (!optAugNode.isPresent()) {
+        if (optAugNode.isEmpty()) {
             return null;
         }
         final DataContainerChild<? extends PathArgument, ?> augNode = optAugNode.get();
@@ -193,7 +193,7 @@ final class CreateStreamUtil {
         }
         final Optional<DataContainerChild<? extends PathArgument, ?>> enumNode = ((AugmentationNode) augNode).getChild(
                 new NodeIdentifier(QName.create(RestconfStreamsConstants.SAL_REMOTE_AUGMENT, paramName)));
-        if (!enumNode.isPresent()) {
+        if (enumNode.isEmpty()) {
             return null;
         }
         final Object value = enumNode.get().getValue();
index 5408b7015b5d9715b09da1866dff8afa6977dafb..6b06b5278a0b6c6e0eed7259a5de8dfb0fcfef32 100644 (file)
@@ -48,7 +48,7 @@ public class RestconfDataStreamServiceImpl implements RestconfDataStreamService
         final String streamName = ListenersBroker.createStreamNameFromUri(identifier);
         final Optional<BaseListenerInterface> listener = listenersBroker.getListenerFor(streamName);
 
-        if (!listener.isPresent()) {
+        if (listener.isEmpty()) {
             LOG.debug("Listener for stream with name {} was not found.", streamName);
             throw new RestconfDocumentedException("Data missing", ErrorType.APPLICATION, ErrorTag.DATA_MISSING);
         }
index 8bef7a3f46c83fdd37fa70b266c7ef9d14024721..9eee1b462ee821bf104db9c94bd7ccaa82b75729 100644 (file)
@@ -136,7 +136,7 @@ public final class RestconfInvokeOperationsUtil {
     public static DOMActionResult invokeActionViaMountPoint(final DOMMountPoint mountPoint, final ContainerNode data,
             final Absolute schemaPath, final YangInstanceIdentifier yangIId) {
         final Optional<DOMActionService> mountPointService = mountPoint.getService(DOMActionService.class);
-        if (!mountPointService.isPresent()) {
+        if (mountPointService.isEmpty()) {
             throw new RestconfDocumentedException("DomAction service is missing.");
         }
         return prepareActionResult(mountPointService.get().invokeAction(schemaPath, prepareDataTreeId(yangIId), data));
index e6567c1eea11889c5ce0285b8c45ff9f8b60ef65..305b93ff7a95829daa5c8f34ea992eee13158842 100644 (file)
@@ -84,7 +84,7 @@ public final class ParserIdentifier {
         if (identifier == null || !identifier.contains(RestconfConstants.MOUNT)) {
             return createIIdContext(schemaContext, identifier, null);
         }
-        if (!mountPointService.isPresent()) {
+        if (mountPointService.isEmpty()) {
             throw new RestconfDocumentedException("Mount point service is not available");
         }
 
index 334a774dd554cc345e17285f34484de234c5661f..de31eb364ddf2936c2b0629854ad6ee56af57119 100644 (file)
@@ -139,7 +139,7 @@ public final class YangInstanceIdentifierDeserializer {
             // parse value
             final QName key = keys.next();
             Optional<DataSchemaNode> leafSchemaNode = listSchemaNode.findDataChildByName(key);
-            RestconfDocumentedException.throwIf(!leafSchemaNode.isPresent(), ErrorType.PROTOCOL, ErrorTag.BAD_ELEMENT,
+            RestconfDocumentedException.throwIf(leafSchemaNode.isEmpty(), ErrorType.PROTOCOL, ErrorTag.BAD_ELEMENT,
                     "Schema not found for %s", key);
 
             final String value = findAndParsePercentEncoded(nextIdentifierFromNextSequence(IDENTIFIER_PREDICATE));