From 256b7d784924a2755956a434b825de6c5b4e5032 Mon Sep 17 00:00:00 2001 From: Robert Varga Date: Mon, 4 Oct 2021 21:44:10 +0200 Subject: [PATCH] Update ParserIdentifier a bit Hide a testing-only method and mark a method for movement. JIRA: NETCONF-631 Change-Id: I73f8e32803bec49dce747503206cde78df75fbce Signed-off-by: Robert Varga --- .../restconf/nb/rfc8040/utils/parser/ParserIdentifier.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/utils/parser/ParserIdentifier.java b/restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/utils/parser/ParserIdentifier.java index 14cef762c5..5b550dd711 100644 --- a/restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/utils/parser/ParserIdentifier.java +++ b/restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/utils/parser/ParserIdentifier.java @@ -79,6 +79,10 @@ public final class ParserIdentifier { * - mount point service * @return {@link InstanceIdentifierContext} */ + // FIXME: NETCONF-631: this method should not be here, it should be a static factory in InstanceIdentifierContext: + // + // @NonNull InstanceIdentifierContext forUrl(identifier, schemaContexxt, mountPointService) + // public static InstanceIdentifierContext toInstanceIdentifier(final String identifier, final EffectiveModelContext schemaContext, final Optional mountPointService) { if (identifier == null || !identifier.contains(RestconfConstants.MOUNT)) { @@ -170,7 +174,8 @@ public final class ParserIdentifier { * path parameter * @return {@link QName} */ - public static Entry makeQNameFromIdentifier(final String identifier) { + @VisibleForTesting + static Entry makeQNameFromIdentifier(final String identifier) { // check if more than one slash is not used as path separator if (identifier.contains("//")) { LOG.debug("URI has bad format. It should be \'moduleName/yyyy-MM-dd\' {}", identifier); -- 2.36.6