Kelvin,
My feeling is that runtime code should never throw a
NullPointerException. In this particular case where it is very easy
for users to input an invalid string value because of typos, case
sensitivity, or just not remembering the correct property name, it is
pretty ugly. If there is no spec behavior defined for what happens in
the case of an invalid set, then the SDO runtime should probably use
its former behavior of throwing an IllegalArgumentException.
Brent
On 11/30/06, Kelvin Goodson (JIRA) <tuscany-dev@ws.apache.org> wrote:
[ http://issues.apache.org/jira/browse/TUSCANY-951?page=all ]
Kelvin Goodson resolved TUSCANY-951.
------------------------------------
Fix Version/s: Java-M2
Java-Mx
Resolution: Duplicate
This is an instance of the issue in TUSCANY-578. In this case its the variant
of the issue where the spec isn't specific about the type of exception should
be thrown, and the changes made under TUSCANY-885 have altered the behaviour
of the tuscany implementation without violating the spec. I'm going to
transfer this into the TUSCANY-885 bucket defect for consideration, but my
feeling is that this is not a bug.
> NullPointerException when setting an invalid property
> -----------------------------------------------------
>
> Key: TUSCANY-951
> URL: http://issues.apache.org/jira/browse/TUSCANY-951
> Project: Tuscany
> Issue Type: Bug
> Components: Java SDO Implementation
> Reporter: Brent Daniel
> Fix For: Java-M2, Java-Mx
>
>
> Calling set() with an invalid property name results in a
NullPointerException. For example:
> dataObject.setInt("invalidProperty", 5);
> results in:
> java.lang.NullPointerException
> at
org.apache.tuscany.sdo.util.DataObjectUtil.getSetValue(DataObjectUtil.java:1440)
> at
org.apache.tuscany.sdo.util.DataObjectUtil.setInt(DataObjectUtil.java:537)
> at
org.apache.tuscany.sdo.impl.DataObjectImpl.setInt(DataObjectImpl.java:526)
>
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
