Code Review / netconf.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | review | tree
raw | inline | side by side
Remove duplicate ietf-yang-library models
[netconf.git] / netconf / mdsal-netconf-yang-library / src / main / java / org / opendaylight / netconf / mdsal / yang / library / SchemaServiceToMdsalWriter.java
diff --git a/netconf/mdsal-netconf-yang-library/src/main/java/org/opendaylight/netconf/mdsal/yang/library/SchemaServiceToMdsalWriter.java b/netconf/mdsal-netconf-yang-library/src/main/java/org/opendaylight/netconf/mdsal/yang/library/SchemaServiceToMdsalWriter.java
index 918e71d13f9aae83c3bf3a72fad293e702c0e2dc..b3fc2abfb0f5cbf4e989296685e3482def04d5bf 100644 (file)
--- a/netconf/mdsal-netconf-yang-library/src/main/java/org/opendaylight/netconf/mdsal/yang/library/SchemaServiceToMdsalWriter.java
+++ b/netconf/mdsal-netconf-yang-library/src/main/java/org/opendaylight/netconf/mdsal/yang/library/SchemaServiceToMdsalWriter.java
@@ -8,28 +8,26 @@
 
 package org.opendaylight.netconf.mdsal.yang.library;
 
-import com.google.common.collect.Lists;
 import com.google.common.util.concurrent.FutureCallback;
 import com.google.common.util.concurrent.Futures;
 import com.google.common.util.concurrent.MoreExecutors;
 import java.util.List;
 import java.util.Set;
 import java.util.concurrent.atomic.AtomicInteger;
+import java.util.stream.Collectors;
 import javax.annotation.Nullable;
 import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.controller.md.sal.binding.api.WriteTransaction;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
 import org.opendaylight.controller.sal.core.api.model.SchemaService;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.Uri;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160409.ModulesState;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160409.ModulesStateBuilder;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160409.OptionalRevision;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160409.module.list.Module.ConformanceType;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160409.module.list.ModuleBuilder;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160409.module.list.module.Submodules;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160409.module.list.module.SubmodulesBuilder;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160409.module.list.module.submodules.Submodule;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160409.module.list.module.submodules.SubmoduleBuilder;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160621.ModulesState;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160621.ModulesStateBuilder;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160621.RevisionUtils;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160621.module.list.Module.ConformanceType;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160621.module.list.ModuleBuilder;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160621.module.list.module.Submodule;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160621.module.list.module.SubmoduleBuilder;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.types.rev130715.YangIdentifier;
 import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 import org.opendaylight.yangtools.yang.model.api.Module;
@@ -96,46 +94,29 @@ public class SchemaServiceToMdsalWriter implements SchemaContextListener, AutoCl
     }
 
     private ModulesState createModuleStateFromModules(final Set<Module> modules) {
-        final ModulesStateBuilder modulesStateBuilder = new ModulesStateBuilder();
-        final List<org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160409.module.list
-                .Module> moduleList =
-                Lists.newArrayList();
-
-        for (final Module module : modules) {
-            moduleList.add(createModuleEntryFromModule(module));
-        }
-
-        return modulesStateBuilder.setModule(moduleList).setModuleSetId(String.valueOf(moduleSetId.getAndIncrement()))
+        return new ModulesStateBuilder()
+                .setModule(modules.stream().map(this::createModuleEntryFromModule).collect(Collectors.toList()))
+                .setModuleSetId(String.valueOf(moduleSetId.getAndIncrement()))
                 .build();
     }
 
-    private org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160409.module.list.Module
+    private org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.library.rev160621.module.list.Module
         createModuleEntryFromModule(final Module module) {
-        final ModuleBuilder moduleBuilder = new ModuleBuilder();
-
-        // TODO Conformance type is always set to Implement value, but it should it really be like this?
-        // TODO Add also deviations and features lists to module entries
-        moduleBuilder.setName(new YangIdentifier(module.getName()));
-
-        module.getQNameModule().getRevision().ifPresent(rev -> moduleBuilder.setRevision(
-            new OptionalRevision(rev.toString())));
-
-        return moduleBuilder.setNamespace(new Uri(module.getNamespace().toString()))
+        return new ModuleBuilder()
+                .setName(new YangIdentifier(module.getName()))
+                .setRevision(RevisionUtils.fromYangCommon(module.getQNameModule().getRevision()))
+                .setNamespace(new Uri(module.getNamespace().toString()))
+                // FIXME: Conformance type is always set to Implement value, but it should it really be like this?
                 .setConformanceType(ConformanceType.Implement)
-                .setSubmodules(createSubmodulesForModule(module))
+                .setSubmodule(createSubmodulesForModule(module))
+                // FIXME: Add also deviations and features lists to module entries
                 .build();
     }
 
-    private static Submodules createSubmodulesForModule(final Module module) {
-        final List<Submodule> submodulesList = Lists.newArrayList();
-        for (final Module subModule : module.getSubmodules()) {
-            final SubmoduleBuilder subModuleEntryBuilder = new SubmoduleBuilder()
-                    .setName(new YangIdentifier(subModule.getName()));
-            subModule.getQNameModule().getRevision().ifPresent(
-                rev -> subModuleEntryBuilder.setRevision(new OptionalRevision(rev.toString())));
-            submodulesList.add(subModuleEntryBuilder.build());
-        }
-
-        return new SubmodulesBuilder().setSubmodule(submodulesList).build();
+    private static List<Submodule> createSubmodulesForModule(final Module module) {
+        return module.getSubmodules().stream().map(subModule -> new SubmoduleBuilder()
+            .setName(new YangIdentifier(subModule.getName()))
+            .setRevision(RevisionUtils.fromYangCommon(subModule.getQNameModule().getRevision()))
+            .build()).collect(Collectors.toList());
     }
 }
Netconf