[ 
https://issues.apache.org/jira/browse/ISIS-256?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Minto van der Sluis updated ISIS-256:
-------------------------------------

    Comment: was deleted

(was: Debugging revealed the PojoAdapter.checkLock() was called with 
otherVersion = null. This resulted in checkLoch() throwing a 
ConcurrencyException.

Adding a check on the value of otherVersion solved my problem.

Actually instead of the changed if statement I preferred the following:


        if (otherVersion != null && otherVersion.different(version)) {
        ....

This however resulted in failing unit test. In my opinion a sign that the unit 
tests are coupled too tightly to the implementation of the method. 
)
    
> NullPointerReference exceptions when attempting to persist an object
> --------------------------------------------------------------------
>
>                 Key: ISIS-256
>                 URL: https://issues.apache.org/jira/browse/ISIS-256
>             Project: Isis
>          Issue Type: Bug
>          Components: Viewer: RestfulObjects
>    Affects Versions: 0.2.0-incubating, 0.3.1-incubating
>            Reporter: Barry Woods
>            Assignee: Dan Haywood
>              Labels: patch
>             Fix For: 0.3.1-incubating
>
>         Attachments: DomainResourceHelper.java
>
>   Original Estimate: 2h
>  Remaining Estimate: 2h
>
> NullPointerReference exceptions when attempting to persist an object. This 
> occurs under two scenarios:
> 1. Omitting the json for a field annotated as @Optional in the pojo
> 2. Attempting to persist an object that has getter only fields

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to