flowable / flowable-engine

A compact and highly efficient workflow and Business Process Management (BPM) platform for developers, system admins and business users.
https://www.flowable.org
Apache License 2.0
8.01k stars 2.63k forks source link

flowable-ui default tenantId '' in oracle #2904

Closed ferry135 closed 3 years ago

ferry135 commented 3 years ago

Describe the bug I find a problem when use flowable-ui moder http://localhost:8080/flowable-ui/modeler with oracle database. Can't not query process model of my created. In oracle empty string is equals null,the select sql can't use select from ACT_DE_MODEL WHERE model_type = '0' and tenant_id = '' order by last_updated desc. 4-21 14:31:22.474 DEBUG 24348 --- [nio-8080-exec-2] o.f.u.c.tenant.DefaultTenantProvider : Using user tenantId: '' 2021-04-21 14:31:22.474 DEBUG 24348 --- [nio-8080-exec-2] org.mybatis.spring.SqlSessionUtils : Creating a new SqlSession 2021-04-21 14:31:22.474 DEBUG 24348 --- [nio-8080-exec-2] org.mybatis.spring.SqlSessionUtils : Registering transaction synchronization for SqlSession [org.apache.ibatis.session.defaults.DefaultSqlSession@526c025] 2021-04-21 14:31:22.475 DEBUG 24348 --- [nio-8080-exec-2] o.m.s.t.SpringManagedTransaction : JDBC Connection [HikariProxyConnection@1961793543 wrapping oracle.jdbc.driver.T4CConnection@60ddb2ce] will be managed by Spring 2021-04-21 14:31:22.475 DEBUG 24348 --- [nio-8080-exec-2] o.f.u.m.d.Model.selectModelByParameters : ==> Preparing: select from ACT_DE_MODEL WHERE model_type = ? and tenant_id = ? order by last_updated desc 2021-04-21 14:31:22.475 DEBUG 24348 --- [nio-8080-exec-2] o.f.u.m.d.Model.selectModelByParameters : ==> Parameters: 0(Integer), (String) 2021-04-21 14:31:22.478 DEBUG 24348 --- [nio-8080-exec-2] o.f.u.m.d.Model.selectModelByParameters : <== Total: 0

Expected behavior For the default tenantId is '' select * from ACT_DE_MODEL WHERE model_type = '0' and tenant_id is null order by last_updated desc.

Code http://localhost:8080/flowable-ui/modeler-app/rest/models?filter=processes&modelType=0&sort=modifiedDesc

Additional context flowable-ui vesion is 6.6.0.

stop-bullshit commented 3 years ago

duplicate #2674

PascalSchumacher commented 3 years ago

I guess this can be closed as https://github.com/flowable/flowable-engine/issues/2674 was fixed.

filiphr commented 3 years ago

Thanks @PascalSchumacher