[jboss-dev-forums] [Design of Management Features on JBoss] - Re: JBAS-6672, null values and defaults

[ips]

anonymous wrote : "ips" wrote:
  | 
  | 1) Will:
  | 
  | managedComponent.getProperties().remove("foo")
  | 
  | be semantically equivalent to:
  | 
  | managedComponent.getProperty("foo").setRemoved(true)
  | 
  | ? (I think it should be)
  | 
  | 
  | Yes it should. 

Then it could be argued that the setRemoved/isRemoved API on ManagedProperty is not truly needed. The only advantages I see of including the API are that it's a bit easier to document Javadoc-wise, and it's easier to implement the isMandatory check I described in my previous post.

anonymous wrote : The latest jboss-managed 2.1.0.CR6 simply maps MetaValues with null values onto the corresponding primitive uninitialized value when the MetaValue is unwrapped, so essentially there is no way a primitive will have a null value. I view this as akin to auto promotion of nulls so that one does not have to track whether a property is a primitive. 

I don't love this auto-promotion, because I think many users will still try to use simpleValue.setValue(null) to remove/unspecify that property. For non-primitive SimpleValues, this isn't a big deal, because 99% of the time it will have the same ultimate effect on the managed resource. But for primitive SimpleValues, it will end up setting the primitive to the Java init default, which will often not be the same as the underlying managed resource's default for the properties.


View the original post : http://www.jboss.org/index.html?module=bb&op=viewtopic&p=4223074#4223074

Reply to the post : http://www.jboss.org/index.html?module=bb&op=posting&mode=reply&p=4223074