Operating System: All
Platform: All

Hi,

Since the bug is, to check for null, particularly for the presence of mandate statement with true value, in generated java DTO, we can narrow the change to, leaf element only.

Note: However as per RFC definition "A mandatory node is one of-
o A leaf, choice, or anyxml node with a "mandatory" statement with
the value "true".
o A list or leaf-list node with a "min-elements" statement with a
value greater than zero.
o A container node without a "presence" statement, which has at
least one mandatory node as a child."
We can raise new bug for the other two.

So, for this bug the expected location of change, as per my understanding and analysis, should be in one of the constructors of class LeafSchemaNodeBuilder.java (yang-parser-impl module).

Please correct me if I am wrong on this.Thereby the change look something like following :

public LeafSchemaNodeBuilder(final String moduleName, final int line, final QName qname, final SchemaPath path,
final LeafSchemaNode base) {
...
if ((base instanceof LeafSchemaNode) && constraints.isMandatory())

else

...
}

Please give me a go ahead on above so that I can proceed with checkin of same.

Rashmi

Right, the check should be done for either leaves, as well well any structural container children which are between the current object and a mandatory leaf.

The change should not be made to LeafSchemaNodeBuilder (or any yangtools component), but rather the code generator templates.

Furthermore I do not think this is feasible with the current binding specification, because the generated objects are shared between groupings and their instantiations. Each instantiation can specialize the mandatory flag (as well as constraints). This means that the class itself does not know the details of what really is mandatory and what not. Consider the these two cases:

• refine which makes a leaf mandatory
  grouping foo {
  leaf foo { type string; }

  }

container bar {
uses foo {
refine foo

}
}

• refine which makes a leaf non-mandatory
  grouping foo {
  leaf foo { type string; mandatory true; }

  }

container bar {
uses foo {
refine foo

}
}

No assignee -> Confirmed. Also depends on binding spec 2, as not in comment.