Merge "Bug 735 - Part 1 - add ietf-restconf.yang back into yang-tools."

Bug:700 Mapped description from the yang file to the generated java file

Change-Id: I732d7d4b2c708295ec1358994b4078d35de1769b
Signed-off-by: Debolina Bandyopadhyay <dbandyop@cisco.com>

Bug 735 - Part 1 - add ietf-restconf.yang back into yang-tools.

Added a bundle into yang-tools to ship the ietf-restconf.yang yang file, required by MD-SAL.

NOTE: We are adding the first draft (00) of the ietf-restconf.yang so we
dont have to upgrade all of the other core yang files. It appears that
ietf-yang-types would need to be upgraded as well if we went with the
latest ietf releases. That should happen, but as part of an intention
upgrade to all ietf core.

Note: Gerrit DRAFT https://git.opendaylight.org/gerrit/#/c/6185 provideds
the fixes to the controller project to resolve the issue with the restconf
/modules endpoint, but we can not push that one forward until the yangtools
(this CS) is pushed to master because of dependencies.

Signed-off-by: Devin Avery <devin.avery@brocade.com>

Bug 613: Minimize use of xtend in binding-generator-impl.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Merge "Bug 447 - Yang documentation generator improvements"

Merge "Fix for Bug 586."

Bug 447 - Yang documentation generator improvements

-added summaries at the top of page + linked localy
-added nested child nodes of grouping and augmentation nodes
-added local links between type definitions and their users
-minor fixes

Change-Id: I573f4ee257e66c140e52327534e840801f7b3c5f
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Fix for Bug 586.

Fixed generation of list key properties.
Updated test.

Change-Id: Icfe7e1fa6a5bfb3c8c2b4bedf808e5fbae797359
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fix for Bug 527.

Fixed codec loading for choice case nodes.
Fixed bug in parser when handling uses/augment target path.
Helper methods from BindingGeneratorImpl moved to SchemaContextUtil class.
Added test.

Change-Id: Id1b835150cdad2c278bd9c125d5369822b75639a
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Bug 525 : Document the InstanceIdentifier API

Change-Id: If42682fe4dfd715d95f5f081eae8027836f316e0
Signed-off-by: Moiz Raja <moraja@cisco.com>

Another set of warning fixes

Change-Id: Ic2bd07eb6490174e7f6a8808652dabeec122866c
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Fixed possible NPE in CodecMapping."

Fixed possible NPE in CodecMapping.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Merge "Fix for Bug 511 (model + integration-test)"

Merge "Fix for Bug 511 (yang)"

Merge "Fix for Bug 511 (code-generator)"

Fix for Bug 511 (model + integration-test)

Removed \r characters from yang-ext.yang and from integration-test project.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fix for Bug 511 (yang)

Removed \r characters from all projects under yang directory.

Change-Id: I46a47d523449bc159ea508143d9a06726f8259ff
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fix for Bug 511 (code-generator)

Removed \r characters from all projects under code-generator directory.

Change-Id: Ic6af8cc1b686c62caeaab9e5f2595a597750cafe
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Add ability to get yang sources from SchemaContext.

SchemaContext should provide yang source code of parsed module files.
To identify modules, move YANGModuleIdentifier from binding-generator-spi
to yang-model-api: ModuleIdentifier. This interface can represent
both modules and submodules in SchemaContext. Add methods to SchemaContext
to iterate ove ModuleIdentifiers and get their source.
In order to maintain backwards compatibility no new parsing method was created,
so submodules cannot be supported now in new methods.
Adding this functionality removes need for most methods defined in
org.opendaylight.yangtools.yang.model.parser.api.YangModelParser .
It is advised that after this change parser api would be refactored to
have a method taking collection of input streams as input
and returning SchemaContext.
Other small changes:
Externalize SimpleDateFormat wrapped in ThreadLocal
Add toString to AbstractObjectRegistration
Add guava dependency to yang-model-api for Optional goodness
Add ModuleIdentifierImpl convenience class allowing users to seach source by
 module name.

Change-Id: I8a46fa699bbd829e5d4a488ebc99012a66fc196d
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Merge "Fix for Bug 497."

Merge "Better logging for codegen's generateModuleInfos"

Merge "Fix for Bug 496."

Merge "Migrate RuntimeGeneratedMappingServiceImpl to use SchemaContextListener"

Fix for Bug 497.

Added generation of toString in YangModuleInfoTemplate.

Change-Id: I4a4781618dc58baa4764be378e92c26c7d15f5a2
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Better logging for codegen's generateModuleInfos

Change-Id: I14798f5f7caf9324344351141751be0001ef3a83
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Merge "Added initial draft of Normalized Yang Data Tree model."

Migrate RuntimeGeneratedMappingServiceImpl to use SchemaContextListener

Moves the implemented interface higher up the chain, allowing its users
to start relying on SchemaContextListener -- fixing deprecations
warnings in the process.

Drive-by:
- organized imports to remove unused import
- remove unused field

Change-Id: Ic77e3775317daedd94f6cff6ea6eabdb0ea3dde0
Signed-off-by: Robert Varga <rovarga@cisco.com>

Fix for Bug 496.

If leafref type contains invalid path, no warning was reported and no methods were generated from this leaf/leaf-list.
Added exception throwing if leafref target is not found.
Updated tests.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Introduce ClassLoadingStrategy,ModuleInfoRegistry interface, copy SchemaStoreProvider from md-sal.

To decouple implementation from interface introduce ClassLoadingStrategy and ModuleInfoRegistry interfaces.This allows intercepting
read/write operations.
Copy SchemaContextProvider from org.opendaylight.controller.sal.dom.broker.impl .

Change-Id: Ifebc0f0adbb265f4a4e492c0a9b54f07cc17fe01
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Fix for Bug 489.

Current code generator version was unable to generate sources for augmentation defined under uses statement
if augment points to nested node (not top level node added by uses).

Failing example:

container c {
    uses route-subobjects {
        augment "links/link" {
            leaf id {
                type string;
            }
        }
    }
}

AugmentationSchemaBuilderImpl: fixed bug in parsing augment target path for augmentations defined under uses.
BindingGeneratorImpl: fixed resolving of augment package name.
BindingGeneratorUtil: added new method to generate package name.

Added tests.

Change-Id: Ie48b2fffc0e6449bf5c13b5a6b636cbd837ffb54
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Merge "Fix for Bug 295."

Fix for Bug 295.

Changes in GeneratorJavaFile:
 - produced warning when more than one file with same name is going to be generated
 - constructor parameter type changed from Set to Collection
 - removed unused method generateStringToJavaFile

Fixed implementation of isAcceptable(Type type) in InterfaceGenerator.
Added test.

Change-Id: If2cc5ccf27b15f3749628a6c7059de6703dda213
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Merge "Fixed for Bug 468."

Fixed for Bug 468.

Fixed prefix resolution of augment target path parts on parser level.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fixed missing predicate in package name regex. Improved error reporting.

Change-Id: If76fe681e37a5421621754d088de99b14eeae05d
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Bug 466:Fixed namespace and revision resolution for empty augmentations.

Change-Id: I85af4d15d7f4a6b604461084e9ff4f73b103cca1
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Fixed possible NPE, improved logging.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fixed NPE caused by tryToLoadAugmentations in LazyGeneratedCodecRegistry.

Change-Id: I3142a629b9bd366a960c79447134d87f36f47e25
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Fixed generation of toString method in builder template.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fix for Bug-427.

Added test for mapping from dom (with nested augmenation nodes) to data object.
Added javadocs.

Change-Id: I0d0c08092c6485afb820d9bfbb352cf934f282f3
Signed-off-by: Martin Vitez <mvitez@cisco.com>
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Added initial draft of Normalized Yang Data Tree model.

Change-Id: Ic50335f8a7bfe0fa4d7c239c997f6ca090b9b506
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

BUG-432: Start fixing the Registration contract

Registration should be a generic construct, not tied to a particular
object. Unfortunately that requires API-level breakage which needs to be
orchestrated with the users.

As a first step introduce ObjectRegistration concept which is equivalent
with the current Registration concept and gently nudge the users towards
it.

Change-Id: I05723cffcc41bfa2355b5e75d3e675c4809a772a
Signed-off-by: Robert Varga <rovarga@cisco.com>

Fix various warnings

Change-Id: I68b53f84356885213577a0d793e69f38d5971a6a
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Add injectable LoadingStrategy to RuntimeGeneratedMappingServiceImpl."

Add injectable LoadingStrategy to RuntimeGeneratedMappingServiceImpl.

This allows config-managed to inject ModuleInfoBackedContext as LoadingStrategy in order to resolve Identity classes.

Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Fixed getClassName method.

If string contained point character, it wasn't escaped and cause compile errors.

Change-Id: I3d2179133a160adb8e1c79649e880aee124e40f3
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fixed incorrect path construction in JUnit tests.

Change-Id: I0a9d73876ea0dd6263d414e91070d405fef9069b
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fix of $YangModuleInfoImpl for ext. yang models

Fixed bug in writeJavaSource method in CodeGeneratorImpl. If package does not exist the dir structure is created.
Without this fix yang models containing only extension definitions could not be generated.

Signed-off-by: Lukas Sedlak <lsedlak@cisco.com>

Merge "Fixed DataObjectReadingUtil to resolve multiple items from lists if InstanceIdentifier is wildcarded."

Fixed DataObjectReadingUtil to resolve multiple items from lists if InstanceIdentifier is wildcarded.

Change-Id: Iac3d30c548435d1f3a8575ec6f11ac09cd72bb3d
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Fixed bug in code generator when handling case defined in grouping as augment target."

Revert "Added ietf-restconf yang module"

This reverts commit 9003fb310abfa2773e570424dd4b38953beb0673.

Change-Id: Ifd9255b63b76fa1e697361a6b16826921999695a
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Temporarily reverted "Update ietf-inet-types and ietf-yang-types with RFC6991""

Temporarily reverted "Update ietf-inet-types and ietf-yang-types with RFC6991"

Upstream dependencies did not honor warnings which required them to specify model revision.

Change-Id: I7ac5d5820bf47fb2e429eb8b8c6ef060b6eab652
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Added ietf-restconf yang module"

Merge "Update ietf-inet-types and ietf-yang-types with RFC6991"

Update ietf-inet-types and ietf-yang-types with RFC6991

Change-Id: Ie6f6891835f761c440650a2d0d7990db20f9d52f
Signed-off-by: Robert Varga <rovarga@cisco.com>

Added ietf-restconf yang module

Change-Id: Ifd8012f2ba2b436b68d590caaed512900733fe17
Signed-off-by: Martin Sunal <msunal@cisco.com>

Fixed concurency bug when multiple threads tried to compute package name.

Change-Id: Ifed593dc386124a596cc85e6aea512360a9d942b
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Added utility functions which allows for resolving InstanceIdentifier against concrete DataObject

Change-Id: I12c90a8f75782b9be524d33763ec012e8b0a31f3
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Fixed incorrect path construction in JUnit tests.

Change-Id: I8d10bd6aa6cc84e3435a438f7d4b2aa94a463259
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Fixed bug in code generator when handling case defined in grouping as augment target.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Merge "Added Export-Packages for propper bundle startup in osgi container"

Bug fixes for codecs.

Added proper serialization of Empty Type Definitions into TransformerGenerator.xtend.
Added overload method "toDocument" which creates XML Document without schema definition for particular Data DOM tree.
This fix is primary needed for md-sal NetconfMapping. If this fix is NOT applied the toRpcMessage in NetconfMapping.xtend will NOT work properly.
Added javadocs for both toDocument methods.

Signed-off-by: Lukas Sedlak <lsedlak@cisco.com>
Change-Id: Ia1af32d3a9f11d81da8abf4274b84ca3b5d610d5

Added Export-Packages for propper bundle startup in osgi container

Change-Id: I5f03ed56b1bd6cff36597bdf667b6042e1e9c046
Signed-off-by: Martin Bobak <mbobak@cisco.com>

Added SchemaContextHolder interface to RuntimeGeneratedMappingServiceImpl to provide its schemaContext

Change-Id: Iceb5af637ba36ec23c8874a830b7d804ea7f345d
Signed-off-by: Martin Bobak <mbobak@cisco.com>
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Fixed performance bug when Augmentation codecs we're not cached properly for fast access.

Change-Id: I33e41bc778c4b11abded46650a2e4b0fcabb6519
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Added test for OSGi metadata.

Change-Id: I225ffa4b5b85fda03e31b238f51151fd89bb6b17
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Added support for discovering YangModuleInfo via ServiceLoader.

  - Added ModuleInfoBackedContext which provides
     -  GeneratedClassLoadingStrategy which supports proactive search
     -  Creation of SchemaContext based on registered YangModuleInfo

Change-Id: I890c275031151b874b42b4e46bee4edcfce595ec
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Fixed situation when xmlClass property was not serialized.

Change-Id: I8a226441bc1370bcf3e7e87b15caec1f17c06e5f
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Make sure class loading strategy is initialized

Fixes a failure to verify in the controller

Change-Id: I90a516c5a8d88e17b419db2bf02c02ee36eb6cd6
Signed-off-by: Robert Varga <rovarga@cisco.com>

Extended BindingReflections to support detecting notification callbacks.

Change-Id: I406d92678b6784242d469d364053a588b76d1135
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Added getServiceType and support for classloading strategies.

Change-Id: I8e43f66fd7d22f34aa0dbc4f89df07c3c0962cd2
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Renamed getLength() method to length() in classes generated from typedefs."

Merge "Make sure builder-generated object do not emit null values"

Renamed getLength() method to length() in classes generated from typedefs.

Updated tests.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Make sure builder-generated object do not emit null values

Change-Id: Ifff51ab174ca03df129943c67155082c363554b8
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Add default output base directory for generated sources as compile root folder for maven project in sal code generator."

Add default output base directory for generated sources as compile root folder for maven project
in sal code generator.

Fixes: maven compilation fails with default outputBaseDir option.
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
Change-Id: Ia6d84079dbb76eac5fcfbd29d93b451c0f7e14f9

Remove CRLFs

Change-Id: I3110b88c70a90797cf699709761c04ab05b7124f
Signed-off-by: Robert Varga <rovarga@cisco.com>

Fixed resolving of NotificationDefinition.

  - Deduplicated same code in TransformerGenerator

Change-Id: I1a422aa18d43b86a3132884e539cd65b90a64dfe
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Fixed bug in generation of return type for list key properties."

Merge "Fixed missing slash in generated YangModuleInfo implementations"

Fixed missing slash in generated YangModuleInfo implementations

Change-Id: I2d667fe8c7ddb9c18d521a314f9b00c133141b6a
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Do not return null when no RPC is registered"

Do not return null when no RPC is registered

Change-Id: I6f416440d6a497d17050a1971e9a1eb111483e7a
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "BUG-374: cleanup error messages present in controller startup"

BUG-374: cleanup error messages present in controller startup

This tones down the "BUG" word which may alarm a casual user. These
still indicate run-time problems which should not occur in a
deterministic run-time environment.

Change-Id: I80871ea1999d2a92d6b343d83ddfc784e7e29ecc
Signed-off-by: SahilShah <sahil4.s@tcs.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

[maven-release-plugin] prepare for next development iteration

[maven-release-plugin] prepare release yangtools-0.6.1

Revert "Fixed resolving of NotificationDefinition."

This reverts commit 753056cd7ff59b415e6f916030b5ed38f9585e49.

Change-Id: Ife22a16ef87cfb2828b4209266d3d753c7bfd9f7
Signed-off-by: Robert Varga <rovarga@cisco.com>

[maven-release-plugin] prepare for next development iteration

[maven-release-plugin] prepare release yangtools-0.6.0

Fixed bug in generation of return type for list key properties.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fixed resolving of NotificationDefinition.

Added dipsatch methods for handling of generating body for NotificationDefinition.

Signed-off-by: Lukas Sedlak <lsedlak@cisco.com>

Fix for bug #233
Have made changes in augment-monitoring.xml
Signed-off-by: SahilShah <sahil4.s@tcs.com>
Change-Id: Id1d031e4ce56c32a3b51d0eebc0ae6de9b56e6a2
Signed-off-by: SahilShah <sahil4.s@tcs.com>

Merge "Fix YangModuleInfoImpl generation on windows"

Fix YangModuleInfoImpl generation on windows

Turns out the generated module name contains backslashes on Windows.
Convert them to slashes before emitting the string.

Change-Id: I0e09bdcec769a82b3c72ccf6fcc4910c615f736f
Signed-off-by: Prasanna Huddar <prasanna.huddar@ericsson.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Yang module source should be always available

Change-Id: Ifed7b030beccae2371c398186550b874d84eff7e
Signed-off-by: Robert Varga <rovarga@cisco.com>