nav[aria-label="Primary Navigation"] { padding: 0; & ul { list-style: none; width: 100%; display: flex; flex-direction: row; justify-content: start; align-items: start; gap: 30px; padding: 0; & li { margin: 0; } & ul li { list-style: none; } } }

As we transition to the new Hyland Community, some post links may be temporarily disrupted as we finalize redirects. We apologize for the inconvenience and are working to resolve this as quickly as possible. Details.

History query returns incorrect result set

frauke

Hello everybody,

the query

List<HistoricProcessInstance> his =processEngine.getHistoryService()
                    .createHistoricProcessInstanceQuery()
                    .processInstanceId(((ExecutionEntity) execution).getProcessInstanceId())
                    .list();
‍‍‍‍‍‍

returns a list of all history entries, instead of a single line as expected.
And the single result query throws an exception

HistoricProcessInstance his =processEngine.getHistoryService()
                    .createHistoricProcessInstanceQuery()
                    .processInstanceId(((ExecutionEntity) execution).getProcessInstanceId())
                    .singleResult();
‍‍‍‍‍‍

When I inspect the database table, there is only one record with the expected process instance id.

Frauke

Find more posts tagged with

Comments

tombaeyens

I created http://jira.codehaus.org/browse/ACT-248 for this

frederikherema1

Frauke,

I double-checked the HistoricProcessInctance tests an queries, couldn't find any issues there.

Which version of activiti are you running?
Can you provide a bit more context about where you are using the code pasted above?

regards

frauke

Hello Frederik,

I am using Activiti 5.0 beta2 and the place were I call the history query is inside an instance of BpmnActivityBehavior.

Thank you for checking.
Best regards Frauke

frederikherema1

Frauke,

Can you add your BpmnActivityBehaviour to a simple process and a testcase that shows the behaviour that you were talking about? I can't seem to reproduce it.

You can just attach it to the issue http://jira.codehaus.org/browse/ACT-248

frauke

Ok, I attached the test files to the jira issue.

frederikherema1

This test runs green in eclipse on 5.0.rc1-SNAPSHOT on trunk.

However, you shouldn't do engine API-calls from within a BpmnJavaDelegation (or Activitybehaviour/EventListener). This could mess up transactions/command contexts, and is probably why you get unexpected results.

frauke

Ok, I have the following problem:
the process modifies some object and I would like to store some process information with the object when saving it in the database.
So to say, which process modified it at which time, etc. So I thought reading the process history entry before storing the object would do.

Are there any other possibilitites to solve such a problem with Activiti?

jbarrez

Could you clarify that a bit with a code snippet?
What exactly do you want to store in the object ?

frauke

Ok, I think about something like this:
But I got the point about messing up the transactional context that frederik mentioned in his post.

<br>public class CreateObjectTask extends BpmnActivityBehavior implements ActivityBehavior {<br>&nbsp;&nbsp;&nbsp; @Override<br>&nbsp;&nbsp;&nbsp; public void execute(final ActivityExecution execution) throws Exception {<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; MyObject o = new MyObject();<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; String pid = ((ExecutionEntity) execution).getProcessInstanceId();<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; HistoricProcessInstance his = processEngine.getHistoryService()<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; .createHistoricProcessInstanceQuery()<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; .processInstanceId(pid)<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; .singleResult();<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if (his != null) {<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; o.setProcessInstance(his.getProcessInstanceId());<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; o.setProcessHistoryId(his.getId());<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; o.setProcessStartedOn(his.getStartTime());<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; }<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; o.save();<br>&nbsp;&nbsp;&nbsp; }<br>}<br>