[ 
https://issues.apache.org/jira/browse/JCR-2068?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12697505#action_12697505
 ] 

Sergey Sachkov  commented on JCR-2068:
--------------------------------------

I think it would be good decision to throw it in setProperty() method, as 
sometimes it takes a lot of time to find out why nullpointer is thrown 
especialy in huge J2EE aplications.
Thanks

> null value could be set into propertyDefinition
> -----------------------------------------------
>
>                 Key: JCR-2068
>                 URL: https://issues.apache.org/jira/browse/JCR-2068
>             Project: Jackrabbit Content Repository
>          Issue Type: Improvement
>          Components: jackrabbit-core
>    Affects Versions: 1.5.3
>         Environment: Windows XP, Tomcat 6.0, JRE6
>            Reporter: Sergey Sachkov 
>         Attachments: JackRabbitTest.java, test.xml
>
>
> It is possible to set null value into propertyDefinition and NullPointer is 
> thrown inside JackRabbit in EffectiveNodeType class, 
> checkSetPropertyValueConstraintsmethod  line 615. So it would be good to 
> check those values for null, as null pointer inside framwork is not a good 
> practice

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to