chemistry-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Steven Li (JIRA)" <>
Subject [jira] [Created] (CMIS-958) InMemory failing CmisJS tests, cause found, patch suggested
Date Thu, 19 Nov 2015 03:28:11 GMT
Steven Li created CMIS-958:

             Summary: InMemory failing CmisJS tests, cause found, patch suggested
                 Key: CMIS-958
             Project: Chemistry
          Issue Type: Bug
          Components: opencmis-server
    Affects Versions: OpenCMIS 0.14.0
         Environment: CentOS 7.1
            Reporter: Steven Li

The InMemory server is failing the test suites from the CmisJS project (,
I traced it down to a somewhat straightforward bug in (in package org.apache.chemistry.opencmis.server.impl.browser),
around line 494, see below:

String changeToken = (changeTokens != null && changeTokens.size() > i ? changeTokens.get(i)
: null);
                if (changeToken.length() == 0) {
                    changeToken = null;

The first statement would sometimes yields a "null" value, which is then used immediately
always as valid object, resulting in NullPointerException. A reasonable patch would be to
change the 2nd line to:

        if ( (changeToken!=null) && (changeToken.length()==0) ) {

Wondering if this makes sense to you all...

This message was sent by Atlassian JIRA

View raw message