includeCaseVariables is ignored in CMMN rest-api GET /cmmn-runtime/case-instances

pstapleton · September 14, 2018, 12:12pm

Hi,

I have been using the CMMN rest-api GET /cmmn-runtime/case-instances to retreive case instances. I also wanted to retreive the case variables in the same request.

GET /cmmn-runtime/case-instances?includeCaseVariables=true

Unfortunately this request does not return the case variables in the response. I looked in the source code and saw that this boolean is ignored (with commented out code)

github.com

flowable/flowable-engine/blob/e84b0fb7cce3248ebaaef3a514ffe1c67aac914f/modules/flowable-cmmn-rest/src/main/java/org/flowable/cmmn/rest/service/api/runtime/caze/BaseCaseInstanceResource.java#L84


    query.caseInstanceBusinessKey(queryRequest.getCaseBusinessKey());
}
if (queryRequest.getInvolvedUser() != null) {
    //query.involvedUser(queryRequest.getInvolvedUser());
}
if (queryRequest.getCaseInstanceParentId() != null) {
    query.caseInstanceParentId(queryRequest.getCaseInstanceParentId());
}
if (queryRequest.getIncludeCaseVariables() != null) {
    if (queryRequest.getIncludeCaseVariables()) {
        //query.includeCaseVariables();
    }
}
if (queryRequest.getVariables() != null) {
    addVariables(query, queryRequest.getVariables());
}


if (queryRequest.getTenantId() != null) {
    query.caseInstanceTenantId(queryRequest.getTenantId());
}

Is there some reason that this code is commented out? There appears to be support for this in the CaseInstanceQueryImpl class. I could try to create a PR uncommenting out the code, but just wondering if there is some other problem with it that I am not seeing?

Regards,
Paul

wirob · October 1, 2018, 6:34am

I’m wondering about this as well, anyone that has any answer to this?

tijs · October 1, 2018, 2:37pm

Thanks for pointing at this. It’s fixed now on master.

Best regards,

Tijs

Topic		Replies	Views
includecaseVariables in docs should be includeCaseVariables	2	282	September 25, 2020
businessStatus filter not available in Query case instances	4	38	April 15, 2025
CMMN query for multiple business keys or names Flowable Engine	8	443	April 29, 2025
Typo in REST link?	1	297	October 28, 2020
Send case instance id in Http Task Request Body	2	578	August 18, 2022

includeCaseVariables is ignored in CMMN rest-api GET /cmmn-runtime/case-instances

Related topics