holunda-io / camunda-bpm-taskpool

Library for pooling user tasks and process related business objects.
https://www.holunda.io/camunda-bpm-taskpool/
Apache License 2.0
66 stars 26 forks source link

StringIndexOutOfBoundsException with empty sort string #883

Closed MichaelVonB closed 10 months ago

MichaelVonB commented 10 months ago

Steps to reproduce

Expected behaviour

Polyflow should be robust enough to handle an empty string as sort

Actual behaviour

Throws StringIndexOutOfBoundsException

(In case of exceptions provide full stack trace)

java.lang.StringIndexOutOfBoundsException: begin 0, end 1, length 0
    at java.base/java.lang.String.checkBoundsBeginEnd(String.java:4604) ~[na:na]
    at java.base/java.lang.String.substring(String.java:2707) ~[na:na]
    at io.holunda.polyflow.view.query.PageableSortableQuery$DefaultImpls.sanitizeSort(PageableSortableQuery.kt:33) ~[polyflow-view-api-4.0.0-20231010.140436-1.jar:4.0.0-SNAPSHOT]
    at io.holunda.polyflow.view.query.task.TasksWithDataEntriesForUserQuery.sanitizeSort(TasksWithDataEntriesForUserQuery.kt:17) ~[polyflow-view-api-4.0.0-20231010.140436-1.jar:4.0.0-SNAPSHOT]
    at io.holunda.polyflow.view.query.PageableSortableQuery$DefaultImpls.sanitizeSort(PageableSortableQuery.kt:21) ~[polyflow-view-api-4.0.0-20231010.140436-1.jar:4.0.0-SNAPSHOT]
    at io.holunda.polyflow.view.query.task.TasksWithDataEntriesForUserQuery.sanitizeSort(TasksWithDataEntriesForUserQuery.kt:17) ~[polyflow-view-api-4.0.0-20231010.140436-1.jar:4.0.0-SNAPSHOT