Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

BPE Unable to Execute Listener #387

Closed
hhund opened this issue Oct 13, 2022 · 0 comments · Fixed by #388 or #391
Closed

BPE Unable to Execute Listener #387

hhund opened this issue Oct 13, 2022 · 0 comments · Fixed by #388 or #391
Assignees
@hhund
Labels
bug Something isn't working ready for release Issue is fixed and merged into develop, ready for next release
Milestone
v0.9.0

Comments

@hhund
Copy link
Member

hhund commented Oct 13, 2022

MultiVersionClassDelegateExecutionListener has a copy/past bug.

highmed-dsf/dsf-bpe/dsf-bpe-server/src/main/java/org/highmed/dsf/bpe/camunda/MultiVersionClassDelegateExecutionListener.java

Lines 54 to 66 in 259490a

protected ExecutionListener getExecutionListenerInstance(ProcessKeyAndVersion processKeyAndVersion)
{
Object delegateInstance = instantiateDelegate(processKeyAndVersion, className, fieldDeclarations);
if (delegateInstance instanceof TaskListener)
{
return (ExecutionListener) delegateInstance;
}
else
{
throw new ProcessEngineException(
delegateInstance.getClass().getName() + " doesn't implement " + ExecutionListener.class);
}

Line 58 shoud be:

 	if (delegateInstance instanceof ExecutionListener)
@hhund hhund added the bug Something isn't working label Oct 13, 2022
@hhund hhund added this to the v0.9.0 milestone Oct 13, 2022
@hhund hhund self-assigned this Oct 13, 2022
@hhund hhund mentioned this issue Oct 13, 2022
Merged
@hhund hhund linked a pull request Oct 13, 2022 that will close this issue
Issues/387 unable to execute listener #388
Merged
@hhund hhund changed the title Unable to Execute Listener BPE Unable to Execute Listener Oct 13, 2022
@hhund hhund added the ready for release Issue is fixed and merged into develop, ready for next release label Oct 16, 2022
wetret added a commit that referenced this issue Oct 17, 2022
@wetret
Merge branch 'issues/139_user-tasks' of github.com:highmed/highmed-ds…
0caa131 
…f into issues/139_user-tasks

* 'issues/139_user-tasks' of github.com:highmed/highmed-dsf:
  moved DSF EndListener back to "end" phase of EndEvents, but at index 0
  upgrading dependencies and managing out-dated dependency
  replaced "Task" string with constants from ResourceType
  adds option to enable pre/post activity debug logging
  Absolute, versionless Task URLs now in log messages instead of id
  added check implementation of the correct interface
  EndListener now added on "start" of EndEvents
  added special case for MimeBodyPart containing MimeMultipart as content
  fixes instanceof check for ExecutionListener delegates, closes #387
@hhund hhund mentioned this issue Oct 17, 2022
Merged
@hhund hhund closed this as completed in 2366e3c Oct 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@hhund hhund

Labels
bug Something isn't working ready for release Issue is fixed and merged into develop, ready for next release
Projects
None yet
Milestone
v0.9.0
Development

Successfully merging a pull request may close this issue.

Issues/387 unable to execute listener highmed/highmed-dsf
Release/0.9.0 highmed/highmed-dsf
1 participant
@hhund