Bug 149251 Inconsistent Handling of a Non-inheritable ROM Property
For a ROM property defined to be non-inheritable, an empty string property value should be consistently handled as an empty string, not as null. For example, an oda.dataSource extension defines a data source property with canInherit=false. When a oda-data-source handle get saved in a report design, it correctly includes the tag for such property, even when an empty string value is specified. <oda-data-source extensionID="my.oda.plugin" name="my Data Source" id="24"> ... <property name="ApplicationServer"></property> </oda-data-source> However, later when using the DE API to read its property value, a null is returned. Then when the report design is subsequently saved, the property tag is lost. Expects that an empty string value of a non-inhertiable property should be preserved.
model/org.eclipse.birt.report.model/src/org/eclipse/birt/report/model/api/metadata/PropertyValueException.java
public PropertyValueException( DesignElement obj, String propName,
			Object value, String errCode )
	{
		super( obj, errCode );
		this.propertyName = propName;
		this.invalidValue = value;
		PropertyDefn propDefn = element.getPropertyDefn( propertyName );
		assert propDefn != null;
		this.propertyTypeName = propDefn.getType( ).getName( );
	}
	/**
	 * Constructs an exception given the definition of the property, an invalid
	 * value and its error code. Using this constructor when the definition of
	 * the property is available
	 * 
	 * @param obj
	 *            design element on which the property was being set
	 * @param propDefn
	 *            definition of the property.
	 * @param value
	 *            invalid value of the property.
	 * @param errCode
	 *            error code.
	 */
	public PropertyValueException( DesignElement obj, IPropertyDefn propDefn,
			Object value, String errCode )
	{
		super( obj, errCode );
		assert propDefn != null;
		this.invalidValue = value;