package org.opendaylight.controller.config.yangjmxgenerator.attribute;
import com.google.common.base.Preconditions;
+import java.util.Arrays;
+import java.util.List;
+import java.util.Optional;
+import javax.management.openmbean.ArrayType;
+import javax.management.openmbean.CompositeType;
+import javax.management.openmbean.OpenDataException;
+import javax.management.openmbean.OpenType;
+import javax.management.openmbean.SimpleType;
import org.opendaylight.controller.config.api.IdentityAttributeRef;
import org.opendaylight.controller.config.yangjmxgenerator.TypeProviderWrapper;
-import org.opendaylight.yangtools.sal.binding.model.api.Type;
+import org.opendaylight.mdsal.binding.model.api.Type;
import org.opendaylight.yangtools.yang.model.api.LeafListSchemaNode;
import org.opendaylight.yangtools.yang.model.api.LeafSchemaNode;
import org.opendaylight.yangtools.yang.model.api.TypeDefinition;
+import org.opendaylight.yangtools.yang.model.api.type.EnumTypeDefinition;
import org.opendaylight.yangtools.yang.model.api.type.IdentityrefTypeDefinition;
import org.opendaylight.yangtools.yang.model.api.type.UnionTypeDefinition;
-
-import javax.management.openmbean.ArrayType;
-import javax.management.openmbean.CompositeType;
-import javax.management.openmbean.OpenDataException;
-import javax.management.openmbean.OpenType;
-import javax.management.openmbean.SimpleType;
-import java.util.Arrays;
-import java.util.List;
+import org.opendaylight.yangtools.yang.model.util.type.CompatUtils;
public class JavaAttribute extends AbstractAttribute implements TypedAttribute {
private final TypeProviderWrapper typeProviderWrapper;
private final TypeDefinition<?> typeDefinition;
- public JavaAttribute(LeafSchemaNode leaf,
- TypeProviderWrapper typeProviderWrapper) {
+ public JavaAttribute(final LeafSchemaNode leaf,
+ final TypeProviderWrapper typeProviderWrapper) {
super(leaf);
this.type = typeProviderWrapper.getType(leaf);
- this.typeDefinition = leaf.getType();
+ this.typeDefinition = CompatUtils.compatLeafType(leaf);
this.typeProviderWrapper = typeProviderWrapper;
- this.nullableDefault = leaf.getDefault();
- this.nullableDefaultWrappedForCode = leaf.getDefault() == null ? null : typeProviderWrapper.getDefault(leaf);
- this.nullableDescription = leaf.getDescription();
+
+ final Optional<? extends Object> typeDefault = leaf.getType().getDefaultValue();
+ if (typeDefault.isPresent()) {
+ nullableDefault = (String) typeDefault.get();
+ nullableDefaultWrappedForCode = typeProviderWrapper.getDefault(leaf);
+ } else {
+ nullableDefault = null;
+ nullableDefaultWrappedForCode = null;
+ }
+
+ this.nullableDescription = leaf.getDescription().orElse(null);
}
- public JavaAttribute(LeafListSchemaNode leaf,
- TypeProviderWrapper typeProviderWrapper) {
+ public JavaAttribute(final LeafListSchemaNode leaf,
+ final TypeProviderWrapper typeProviderWrapper) {
super(leaf);
this.type = typeProviderWrapper.getType(leaf);
this.typeDefinition = leaf.getType();
this.typeProviderWrapper = typeProviderWrapper;
- this.nullableDefault = nullableDefaultWrappedForCode = null;
- this.nullableDescription = leaf.getDescription();
+ this.nullableDefault = this.nullableDefaultWrappedForCode = null;
+ this.nullableDescription = leaf.getDescription().orElse(null);
}
public boolean isUnion() {
- TypeDefinition<?> base = getBaseType(typeProviderWrapper, typeDefinition);
+ final TypeDefinition<?> base = getBaseType(this.typeProviderWrapper, this.typeDefinition);
return base instanceof UnionTypeDefinition;
}
+ public boolean isEnum() {
+ final TypeDefinition<?> base = getBaseType(this.typeProviderWrapper, this.typeDefinition);
+ return base instanceof EnumTypeDefinition;
+ }
+
public TypeDefinition<?> getTypeDefinition() {
- return typeDefinition;
+ return this.typeDefinition;
}
/**
* Returns the most base type
*/
- private TypeDefinition<?> getBaseType(TypeProviderWrapper typeProviderWrapper, TypeDefinition<?> baseType) {
+ private TypeDefinition<?> getBaseType(final TypeProviderWrapper typeProviderWrapper, TypeDefinition<?> baseType) {
while(baseType.getBaseType()!=null) {
baseType = baseType.getBaseType();
}
}
public String getNullableDefaultWrappedForCode() {
- return nullableDefaultWrappedForCode;
+ return this.nullableDefaultWrappedForCode;
}
@Override
public Type getType() {
- return type;
+ return this.type;
}
@Override
public String getNullableDescription() {
- return nullableDescription;
+ return this.nullableDescription;
}
@Override
public String getNullableDefault() {
- return nullableDefault;
+ return this.nullableDefault;
}
@Override
- public boolean equals(Object o) {
- if (this == o)
+ public boolean equals(final Object o) {
+ if (this == o) {
return true;
- if (o == null || getClass() != o.getClass())
+ }
+ if (o == null || getClass() != o.getClass()) {
return false;
- if (!super.equals(o))
+ }
+ if (!super.equals(o)) {
return false;
+ }
- JavaAttribute that = (JavaAttribute) o;
+ final JavaAttribute that = (JavaAttribute) o;
- if (nullableDefault != null ? !nullableDefault
- .equals(that.nullableDefault) : that.nullableDefault != null)
+ if (this.nullableDefault != null ? !this.nullableDefault
+ .equals(that.nullableDefault) : that.nullableDefault != null) {
return false;
- if (nullableDescription != null ? !nullableDescription
+ }
+ if (this.nullableDescription != null ? !this.nullableDescription
.equals(that.nullableDescription)
- : that.nullableDescription != null)
+ : that.nullableDescription != null) {
return false;
- if (type != null ? !type.equals(that.type) : that.type != null)
+ }
+ if (this.type != null ? !this.type.equals(that.type) : that.type != null) {
return false;
+ }
return true;
}
@Override
public int hashCode() {
int result = super.hashCode();
- result = 31 * result + (type != null ? type.hashCode() : 0);
+ result = 31 * result + (this.type != null ? this.type.hashCode() : 0);
result = 31
* result
- + (nullableDescription != null ? nullableDescription.hashCode()
+ + (this.nullableDescription != null ? this.nullableDescription.hashCode()
: 0);
result = 31 * result
- + (nullableDefault != null ? nullableDefault.hashCode() : 0);
+ + (this.nullableDefault != null ? this.nullableDefault.hashCode() : 0);
return result;
}
@Override
public String toString() {
- return "JavaAttribute{" + getAttributeYangName() + "," + "type=" + type
+ return "JavaAttribute{" + getAttributeYangName() + "," + "type=" + this.type
+ '}';
}
@Override
public OpenType<?> getOpenType() {
- TypeDefinition<?> baseTypeDefinition = getBaseType(typeProviderWrapper, typeDefinition);
- Type baseType = typeProviderWrapper.getType(baseTypeDefinition, baseTypeDefinition);
+ final TypeDefinition<?> baseTypeDefinition = getBaseType(this.typeProviderWrapper, this.typeDefinition);
+ final Type baseType = this.typeProviderWrapper.getType(baseTypeDefinition, baseTypeDefinition);
if (isArray()) {
return getArrayType();
- } else if (isEnum(baseType)) {
- return getSimpleType(baseType);
+ } else if (isEnum()) {
+ return getEnumType(baseTypeDefinition);
} else if (isUnion()) {
return getCompositeTypeForUnion(baseTypeDefinition);
} else if (isDerivedType(baseType, getType())) {
return getSimpleType(getType());
}
+ private OpenType<?> getEnumType(final TypeDefinition<?> baseType) {
+ final String fullyQualifiedName = this.typeProviderWrapper.getType(this.node, getTypeDefinition()).getFullyQualifiedName();
+ final String[] items = {"instance"};
+ final String description = getNullableDescription() == null ? getAttributeYangName() : getNullableDescription();
+
+ try {
+ return new CompositeType(fullyQualifiedName, description, items, items, new OpenType[]{SimpleType.STRING});
+ } catch (final OpenDataException e) {
+ throw new RuntimeException("Unable to create enum type" + fullyQualifiedName + " as open type", e);
+ }
+ }
+
public boolean isIdentityRef() {
- return typeDefinition instanceof IdentityrefTypeDefinition;
+ return this.typeDefinition instanceof IdentityrefTypeDefinition;
}
- private OpenType<?> getCompositeTypeForUnion(TypeDefinition<?> baseTypeDefinition) {
+ private OpenType<?> getCompositeTypeForUnion(final TypeDefinition<?> baseTypeDefinition) {
Preconditions.checkArgument(baseTypeDefinition instanceof UnionTypeDefinition,
"Expected %s instance but was %s", UnionTypeDefinition.class, baseTypeDefinition);
- List<TypeDefinition<?>> types = ((UnionTypeDefinition) baseTypeDefinition).getTypes();
+ final List<TypeDefinition<?>> types = ((UnionTypeDefinition) baseTypeDefinition).getTypes();
- String[] itemNames = new String[types.size()+1];
- OpenType<?>[] itemTypes = new OpenType[itemNames.length];
+ final String[] itemNames = new String[types.size()+1];
+ final OpenType<?>[] itemTypes = new OpenType[itemNames.length];
addArtificialPropertyToUnionCompositeType(baseTypeDefinition, itemNames, itemTypes);
- String description = getNullableDescription() == null ? getAttributeYangName() : getNullableDescription();
+ final String description = getNullableDescription() == null ? getAttributeYangName() : getNullableDescription();
int i = 1;
- for (TypeDefinition<?> innerTypeDefinition : types) {
+ for (final TypeDefinition<?> innerTypeDefinition : types) {
- Type innerType = typeProviderWrapper.getType(innerTypeDefinition, innerTypeDefinition);
+ final Type innerType = this.typeProviderWrapper.getType(innerTypeDefinition, innerTypeDefinition);
- TypeDefinition<?> baseInnerTypeDefinition = getBaseType(typeProviderWrapper, innerTypeDefinition);
- Type innerTypeBaseType = typeProviderWrapper.getType(baseInnerTypeDefinition, baseInnerTypeDefinition);
+ final TypeDefinition<?> baseInnerTypeDefinition = getBaseType(this.typeProviderWrapper, innerTypeDefinition);
+ final Type innerTypeBaseType = this.typeProviderWrapper.getType(baseInnerTypeDefinition, baseInnerTypeDefinition);
OpenType<?> innerCompositeType;
innerCompositeType = SimpleTypeResolver.getSimpleType(innerType);
}
- itemNames[i] = typeProviderWrapper.getJMXParamForUnionInnerType(innerTypeDefinition);
+ itemNames[i] = this.typeProviderWrapper.getJMXParamForUnionInnerType(innerTypeDefinition);
itemTypes[i++] = innerCompositeType;
}
- String[] descriptions = Arrays.copyOf(itemNames, itemNames.length);
+ final String[] descriptions = itemNames.clone();
descriptions[0] = DESCRIPTION_OF_VALUE_ATTRIBUTE_FOR_UNION;
try {
return new CompositeType(getUpperCaseCammelCase(), description, itemNames, descriptions, itemTypes);
- } catch (OpenDataException e) {
+ } catch (final OpenDataException e) {
throw new RuntimeException("Unable to create " + CompositeType.class + " with inner elements "
+ Arrays.toString(itemTypes), e);
}
public static final Class<Character> TYPE_OF_ARTIFICIAL_UNION_PROPERTY = char.class;
- private void addArtificialPropertyToUnionCompositeType(TypeDefinition<?> baseTypeDefinition, String[] itemNames, OpenType<?>[] itemTypes) {
- String artificialPropertyName = typeProviderWrapper.getJMXParamForBaseType(baseTypeDefinition);
+ private void addArtificialPropertyToUnionCompositeType(final TypeDefinition<?> baseTypeDefinition, final String[] itemNames, final OpenType<?>[] itemTypes) {
+ final String artificialPropertyName = this.typeProviderWrapper.getJMXParamForBaseType(baseTypeDefinition);
itemNames[0] = artificialPropertyName;
- OpenType<?> artificialPropertyType = getArrayOpenTypeForSimpleType(TYPE_OF_ARTIFICIAL_UNION_PROPERTY.getName(),
+ final OpenType<?> artificialPropertyType = getArrayOpenTypeForSimpleType(TYPE_OF_ARTIFICIAL_UNION_PROPERTY.getName(),
SimpleTypeResolver.getSimpleType(TYPE_OF_ARTIFICIAL_UNION_PROPERTY.getName()));
itemTypes[0] = artificialPropertyType;
}
- private boolean isEnum(Type baseType) {
- return baseType.getFullyQualifiedName().equals(Enum.class.getName());
- }
-
- private OpenType<?> getSimpleType(Type type) {
- SimpleType<?> simpleType = SimpleTypeResolver.getSimpleType(type);
+ private OpenType<?> getSimpleType(final Type type) {
+ final SimpleType<?> simpleType = SimpleTypeResolver.getSimpleType(type);
return simpleType;
}
- private OpenType<?> getCompositeType(Type baseType, TypeDefinition<?> baseTypeDefinition) {
+ private OpenType<?> getCompositeType(final Type baseType, final TypeDefinition<?> baseTypeDefinition) {
- SimpleType<?> innerItemType = SimpleTypeResolver.getSimpleType(baseType);
- String innerItemName = typeProviderWrapper.getJMXParamForBaseType(baseTypeDefinition);
+ final SimpleType<?> innerItemType = SimpleTypeResolver.getSimpleType(baseType);
+ final String innerItemName = this.typeProviderWrapper.getJMXParamForBaseType(baseTypeDefinition);
- String[] itemNames = new String[]{innerItemName};
- String description = getNullableDescription() == null ? getAttributeYangName() : getNullableDescription();
+ final String[] itemNames = new String[]{innerItemName};
+ final String description = getNullableDescription() == null ? getAttributeYangName() : getNullableDescription();
- OpenType<?>[] itemTypes = new OpenType[]{innerItemType};
+ final OpenType<?>[] itemTypes = new OpenType[]{innerItemType};
try {
return new CompositeType(getUpperCaseCammelCase(), description, itemNames, itemNames, itemTypes);
- } catch (OpenDataException e) {
+ } catch (final OpenDataException e) {
throw new RuntimeException("Unable to create " + CompositeType.class + " with inner element of type "
+ itemTypes, e);
}
}
public OpenType<?> getCompositeTypeForIdentity() {
- String[] itemNames = new String[]{IdentityAttributeRef.QNAME_ATTR_NAME};
- String description = getNullableDescription() == null ? getAttributeYangName() : getNullableDescription();
- OpenType<?>[] itemTypes = new OpenType[]{SimpleType.STRING};
+ final String[] itemNames = new String[]{IdentityAttributeRef.QNAME_ATTR_NAME};
+ final String description = getNullableDescription() == null ? getAttributeYangName() : getNullableDescription();
+ final OpenType<?>[] itemTypes = new OpenType[]{SimpleType.STRING};
try {
return new CompositeType(getUpperCaseCammelCase(), description, itemNames, itemNames, itemTypes);
- } catch (OpenDataException e) {
+ } catch (final OpenDataException e) {
throw new RuntimeException("Unable to create " + CompositeType.class + " with inner element of type "
+ itemTypes, e);
}
}
private OpenType<?> getArrayType() {
- String innerTypeFullyQName = getInnerType(getType());
- SimpleType<?> innerSimpleType = SimpleTypeResolver.getSimpleType(innerTypeFullyQName);
+ final String innerTypeFullyQName = getInnerType(getType());
+ final SimpleType<?> innerSimpleType = SimpleTypeResolver.getSimpleType(innerTypeFullyQName);
return getArrayOpenTypeForSimpleType(innerTypeFullyQName, innerSimpleType);
}
- private OpenType<?> getArrayOpenTypeForSimpleType(String innerTypeFullyQName, SimpleType<?> innerSimpleType) {
+ private OpenType<?> getArrayOpenTypeForSimpleType(final String innerTypeFullyQName, final SimpleType<?> innerSimpleType) {
try {
- ArrayType<Object> arrayType = isPrimitive(innerTypeFullyQName) ? new ArrayType<>(innerSimpleType, true)
+ final ArrayType<Object> arrayType = isPrimitive(innerTypeFullyQName) ? new ArrayType<>(innerSimpleType, true)
: new ArrayType<>(1, innerSimpleType);
return arrayType;
- } catch (OpenDataException e) {
+ } catch (final OpenDataException e) {
throw new RuntimeException("Unable to create " + ArrayType.class + " with inner element of type "
+ innerSimpleType, e);
}
}
// TODO verify
- private boolean isPrimitive(String innerTypeFullyQName) {
- if (innerTypeFullyQName.contains("."))
+ private boolean isPrimitive(final String innerTypeFullyQName) {
+ if (innerTypeFullyQName.contains(".")) {
return false;
+ }
return true;
}
private boolean isArray() {
- return type.getName().endsWith("[]");
+ return this.type.getName().endsWith("[]");
}
- private boolean isDerivedType(Type baseType, Type currentType) {
+ private boolean isDerivedType(final Type baseType, final Type currentType) {
return baseType.equals(currentType) == false;
}
- private static String getInnerType(Type type) {
- String fullyQualifiedName = type.getFullyQualifiedName();
+ private static String getInnerType(final Type type) {
+ final String fullyQualifiedName = type.getFullyQualifiedName();
return fullyQualifiedName.substring(0, fullyQualifiedName.length() - 2);
}
Code Review / controller.git / blobdiff