Skip to content

Issue/397 fix 2.0.0 rc2 bugs #398

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
hhund merged 13 commits into from
Nov 25, 2025
Merged

Issue/397 fix 2.0.0 rc2 bugs #398

hhund merged 13 commits into from
Nov 25, 2025

Conversation

@hhund
Copy link
Member

@hhund hhund commented Nov 20, 2025 •
edited
Loading

  • Copy/Paste error in BPE PropertiesConfig
  • FHIR server: Binary migration script not working if binary_data is null
  • FHIR server: Start page links do not include _sort parameter
  • BPE server: Existing process models don't have a valid Tenant-Id set, resulting in no DSF StartListener being attached to Start-Events and thus processes failing because the Start-Task can not be found in process variables.
  • FHIR server: DateTime search parameter cast to timestamp not timestamptz
  • dsf-bpe-process-api-v1 needs camunda not operaton dependency
  • Extension url constants missing in v2 ReadAccessHelper and v2 ProcessAuthorizationHelper

closes #397

@hhund hhund added this to the 2.0.0 milestone Nov 20, 2025
@hhund hhund requested a review from schwzr November 20, 2025 22:53
@hhund hhund self-assigned this Nov 20, 2025
hhund added 3 commits November 24, 2025 16:54
@hhund
refactored codebase for camunda dependency of dsf-bpe-process-api-v1
c67267c 
- New dsf-bpe-process-api-v1-base and dsf-bpe-process-api-v1-operaton
modules.
- Modules dsf-bpe-process-api-v1 and dsf-bpe-process-api-v1-operaton
have dependencies to dsf-bpe-process-api-v1-base.
- Code with dependency to Operaton or Camunda now lives in
dsf-bpe-process-api-v1 and dsf-bpe-process-api-v1-operaton respectively,
code without Operaton or Camunda dependency in
dsf-bpe-process-api-v1-base.
- The dsf-bpe-process-api-v1-base module is shaded into
dsf-bpe-process-api-v1 and dsf-bpe-process-api-v1-operaton.
@hhund
rewrite UserTask.getCamundaFormKey() -> UserTask.getOperatonFormKey()
111e372 
- Added rewrite of
org.camunda.bpm.model.bpmn.instance.UserTask.getCamundaFormKey() to
org.operaton.bpm.model.bpmn.instance.UserTask.getOperatonFormKey().
Method is used in DefaultUserTaskListener and thus may also be used
special code written by process plugin developers.
- Rewrite not needed for DefaultUserTaskListener as it lives in
dsf-bpe-process-api-v1-operaton module.
@hhund
added missing extension url constants and refactored code
6eb117f
github-advanced-security[bot]
github-advanced-security bot found potential problems Nov 24, 2025
View reviewed changes
hhund added 2 commits November 24, 2025 23:07
@hhund
dependency version upgrade
5e3a619 
Not upgrading to HAPI 8.6.0: NullPointerException while validating due
to bug in cache code.
@hhund
added additional resources to cleanup validation logging
c3956b0
@hhund hhund marked this pull request as ready for review November 24, 2025 22:55
@schwzr schwzr self-requested a review November 25, 2025 08:53
hhund added 3 commits November 25, 2025 12:07
@hhund
removed deprecated version numbers
488182c
@hhund
added docker-compose.override.yml for dev-setup fhir and bpe
e85f676
@hhund
added Practitioner base profile as target to Task.requester
b886931 
- Change does not make a difference in validation but improves DSF
implementation guide readability
- Added special case to BundleGenerator for ignoring hl7 base profiles
during cycle detection
@hhund hhund merged commit 8067e69 into develop_2 Nov 25, 2025
4 checks passed
@hhund hhund linked an issue Nov 25, 2025 that may be closed by this pull request
Fix 2.0.0-RC2 Bugs #397
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@schwzr schwzr schwzr approved these changes

Assignees

@hhund hhund

Labels

None yet

Projects

None yet

Milestone

2.0.0

Development

Successfully merging this pull request may close these issues.

Fix 2.0.0-RC2 Bugs

3 participants

@hhund @schwzr