X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=opendaylight%2Fnetconf%2Fconfig-netconf-connector%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fnetconf%2Fconfignetconfconnector%2Fmapping%2Fconfig%2FModuleConfig.java;h=6a17e97350b3245735385fb9dd68b0497b8e8fb0;hb=803d525860fbb1974b65ba5605ba5a9dfe1928a4;hp=064cd9fe6f332cb01124c7243a3e25430b8be6a9;hpb=e7c8982a45c9ee5789d7ba25bb0324065efbe23e;p=controller.git diff --git a/opendaylight/netconf/config-netconf-connector/src/main/java/org/opendaylight/controller/netconf/confignetconfconnector/mapping/config/ModuleConfig.java b/opendaylight/netconf/config-netconf-connector/src/main/java/org/opendaylight/controller/netconf/confignetconfconnector/mapping/config/ModuleConfig.java index 064cd9fe6f..6a17e97350 100644 --- a/opendaylight/netconf/config-netconf-connector/src/main/java/org/opendaylight/controller/netconf/confignetconfconnector/mapping/config/ModuleConfig.java +++ b/opendaylight/netconf/config-netconf-connector/src/main/java/org/opendaylight/controller/netconf/confignetconfconnector/mapping/config/ModuleConfig.java @@ -8,9 +8,12 @@ package org.opendaylight.controller.netconf.confignetconfconnector.mapping.config; -import com.google.common.base.Preconditions; -import com.google.common.collect.Maps; +import com.google.common.base.Optional; +import com.google.common.collect.HashMultimap; +import com.google.common.collect.Multimap; import org.opendaylight.controller.config.api.jmx.ObjectNameUtil; +import org.opendaylight.controller.netconf.api.NetconfDocumentedException; +import org.opendaylight.controller.netconf.confignetconfconnector.operations.editconfig.EditConfig; import org.opendaylight.controller.netconf.confignetconfconnector.operations.editconfig.EditStrategyType; import org.opendaylight.controller.netconf.util.xml.XmlElement; import org.opendaylight.controller.netconf.util.xml.XmlNetconfConstants; @@ -21,15 +24,14 @@ import org.w3c.dom.Element; import javax.management.ObjectName; import java.util.Collection; -import java.util.HashMap; +import java.util.Date; import java.util.Map; public class ModuleConfig { private final String moduleName; private final InstanceConfig instanceConfig; - // TODO 2 services from same namespace ? - private final Map providedServices; + private final Multimap providedServices; public ModuleConfig(String moduleName, InstanceConfig mbeanMapping, Collection providedServices) { this.moduleName = moduleName; @@ -37,12 +39,11 @@ public class ModuleConfig { this.providedServices = mapServices(providedServices); } - private Map mapServices(Collection providedServices) { - HashMap mapped = Maps.newHashMap(); + private Multimap mapServices(Collection providedServices) { + Multimap mapped = HashMultimap.create(); for (QName providedService : providedServices) { String key = providedService.getNamespace().toString(); - Preconditions.checkState(mapped.containsKey(key) == false); mapped.put(key, providedService.getLocalName()); } @@ -53,26 +54,23 @@ public class ModuleConfig { return instanceConfig; } - public Map getProvidedServices() { + public Multimap getProvidedServices() { return providedServices; } - public Element toXml(ObjectName instanceON, Services depTracker, Document document, String namespace) { - Element root = document.createElement(XmlNetconfConstants.MODULE_KEY); - // Xml.addNamespaceAttr(document, root, namespace); + public Element toXml(ObjectName instanceON, ServiceRegistryWrapper depTracker, Document document, String namespace) { + Element root = XmlUtil.createElement(document, XmlNetconfConstants.MODULE_KEY, Optional.absent()); + + // type belongs to config.yang namespace, but needs to be prefix:moduleName + + Element typeElement = XmlUtil.createTextElementWithNamespacedContent(document, XmlNetconfConstants.TYPE_KEY, + XmlNetconfConstants.PREFIX, namespace, moduleName); - final String prefix = getPrefix(namespace); - Element typeElement = XmlUtil.createPrefixedTextElement(document, XmlNetconfConstants.TYPE_KEY, prefix, - moduleName); - XmlUtil.addPrefixedNamespaceAttr(typeElement, prefix, namespace); - // Xml.addNamespaceAttr(document, typeElement, - // XMLUtil.URN_OPENDAYLIGHT_PARAMS_XML_NS_YANG_CONTROLLER_CONFIG); root.appendChild(typeElement); + // name belongs to config.yang namespace + String instanceName = ObjectNameUtil.getInstanceName(instanceON); + Element nameElement = XmlUtil.createTextElement(document, XmlNetconfConstants.NAME_KEY, instanceName, Optional.absent()); - Element nameElement = XmlUtil.createTextElement(document, XmlNetconfConstants.NAME_KEY, - ObjectNameUtil.getInstanceName(instanceON)); - // Xml.addNamespaceAttr(document, nameElement, - // XMLUtil.URN_OPENDAYLIGHT_PARAMS_XML_NS_YANG_CONTROLLER_CONFIG); root.appendChild(nameElement); root = instanceConfig.toXml(instanceON, depTracker, namespace, document, root); @@ -80,18 +78,10 @@ public class ModuleConfig { return root; } - private String getPrefix(String namespace) { - // if(namespace.contains(":")==false) - return XmlNetconfConstants.PREFIX; - // return namespace.substring(namespace.lastIndexOf(':') + 1, - // namespace.length()); - - } - - public ModuleElementResolved fromXml(XmlElement moduleElement, Services depTracker, String instanceName, - String moduleNamespace, EditStrategyType defaultStrategy) { + public ModuleElementResolved fromXml(XmlElement moduleElement, ServiceRegistryWrapper depTracker, String instanceName, + String moduleNamespace, EditStrategyType defaultStrategy, Map> identityMap) throws NetconfDocumentedException { - InstanceConfigElementResolved ice = instanceConfig.fromXml(moduleElement, depTracker, moduleNamespace, defaultStrategy); + InstanceConfigElementResolved ice = instanceConfig.fromXml(moduleElement, depTracker, moduleNamespace, defaultStrategy, providedServices, identityMap); return new ModuleElementResolved(instanceName, ice); }