search

camunda-community-hub / camunda-platform-7-camel

Community Extension to add Apache Camel support for Camunda Platform 7
Apache License 2.0
82 stars 57 forks source link

An optional type should be available with the CamundaBpmCorrelationKey #23

Closed ArashAmiri closed 7 years ago

ArashAmiri commented 8 years ago

When sending the CamundaBpmCorrelationKey, it is automatically extracted as a String from the properties. This should be the default value. We added a new constant "CamundaBpmCorrelationKeyType" where e.g. the value "java.lang.Long" can be passed. This saves us from parsing the String over and over in out code, since we know it is a Long.

It was a minor fix in the org.camunda.bpm.camel.component.producer.MessageProducer class. Tests were added and a new constant in org.camunda.bpm.camel.component.CamundaBpmConstants (CAMUNDA_BPM_CORRELATION_KEY_TYPE)

meyerdan commented 8 years ago

Hi ArashAmiri,

thank you for providing this pull request. The maintainer of this community extension @berndruecker is currently on holiday and will look into it when he is back.

Best Regards, Daniel