reproducer bz802721 #56

Closed
wants to merge 8 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

rsynek commented Mar 13, 2012

CommandFactory.newStartProcess does not pass parameters to process instance

Member

mrietveld commented Apr 9, 2012

Radovan,

Thanks for the pull request. If possible, could you make sure to do the following in the future?

  • Make a branch for your pull request (like BZ-807221).
  • Rebase, don't use merge commits. Using merge commits means more work when merging the pull request.

Thanks!

Member

mrietveld commented Apr 9, 2012

Hi Radovan,

When I run your test (ProcessExecutionTest), I'm getting the following error:

(null: 15, 49): cvc-datatype-valid.1.2.1: '' is not a valid value for 'QName'.
(null: 15, 49): cvc-attribute.3: The value '' of attribute 'structureRef' on element 'itemDefinition' is not valid with respect to its type, 'QName'.

This is because of the following line in sampleProcess.bpmn:

<itemDefinition id="_xItem" structureRef="" />

Could you correct this? Thanks!

Contributor

rsynek commented Apr 15, 2012

Hi Marco,
I have fixed the reproducer and moved it to separate branch as you requested:
droolsjbpm#84

Thank you for your comments!

@ghost ghost assigned mrietveld Oct 31, 2012

Member

mrietveld commented Dec 20, 2012

Closing because PR 84 replaces this one.

@mrietveld mrietveld closed this Dec 20, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment