topic Modeler - Email Task deployment fails in Alfresco Archive

Modeler - Email Task deployment fails

skay — Wed, 06 Mar 2013 08:07:41 GMT

Hi,When adding an Email task with the Kis Modeler, the Process deployement fails.You may reproduce this bug with latest version 5.12 just adding an email task to the demo process :"Create of BPMN XML failedOne of the following is mandatory on a field declaration: one of attributes stringValue|expres

Re: Modeler - Email Task deployment fails

jbarrez — Mon, 11 Mar 2013 10:15:01 GMT

Created http://jira.codehaus.org/browse/ACT-1604

I don't believe all parameters are mandatory. Did you try finding the minimal set that works?

Re: Modeler - Email Task deployment fails

skay — Mon, 25 Mar 2013 11:02:06 GMT

:cry: Actually all parameters are mandatory.
I tryed again adding each parameter following this order
1) TO alone
2) TO + FROM alone
3) TO + FROM + SUBJECT
3) TO + FROM + SUBJECT + TEXT
4) TO + FROM + SUBJECT + TEXT + CC
5) TO + FROM + SUBJECT + TEXT + CC + BCC + CHARSET

All these combination are KO.

And then, if I add the latest parameter HTML / whitch is redundant with TEXT
6) TO + FROM + SUBJECT + TEXT + CC + BCC + CHARSET + HTML

Then this is OK !

If I go just 1 step back, e.g. remove value of CHARSET, I got same exception at deployment, pointing the line of the declaration of the mail task :

org.activiti.engine.ActivitiException: One of the following is mandatory on a fi
eld declaration: one of attributes stringValue|expression or one of child elemen
ts string|expression | Demo model.bpmn20.xml | line 33 | column 85

        at org.activiti.engine.impl.util.xml.Parse.throwActivitiExceptionForErro
rs(Parse.java:186)
        at org.activiti.engine.impl.bpmn.parser.BpmnParse.execute(BpmnParse.java
:242)
        at org.activiti.engine.impl.bpmn.deployer.BpmnDeployer.deploy(BpmnDeploy
er.java:92)

THis is the resulting XML fragment / after removal of the charset parameter :

<serviceTask id="sid-6FB5241A-FE3E-4148-8DDA-D692C2002910" activiti:type="mail">
      <extensionElements>
        <activiti:field name="to" stringValue="jbarrez@activiti.org"></activiti:field>
        <activiti:field name="from" stringValue="jbarrez@activiti.org"></activiti:field>
        <activiti:field name="subject" stringValue="aaa"></activiti:field>
        <activiti:field name="cc" stringValue="jbarrez@activiti.org"></activiti:field>
        <activiti:field name="bcc" stringValue="jbarrez@activiti.org"></activiti:field>
        <activiti:field name="text" stringValue="aaa"></activiti:field>
        <activiti:field name="html" stringValue="aaa"></activiti:field>
        <activiti:field name="charset"></activiti:field>
      </extensionElements>
    </serviceTask>

So clearly ALL parameters seem to be compulsory here.

I didn't investigate further. Is it the way of how the mail task is designed :

public class MailActivityBehavior extends AbstractBpmnActivityBehavior {

protected Expression to;
protected Expression from;
protected Expression cc;
protected Expression bcc;
protected Expression subject;
protected Expression text;
protected Expression html;
protected Expression charset;

public void execute(ActivityExecution execution) {
    String toStr = getStringFromField(to, execution);
    String fromStr = getStringFromField(from, execution);
    String ccStr = getStringFromField(cc, execution);
    String bccStr = getStringFromField(bcc, execution);
    String subjectStr = getStringFromField(subject, execution);
    String textStr = getStringFromField(text, execution);
    String htmlStr = getStringFromField(html, execution);
    String charSetStr = getStringFromField(charset, execution);

OR is it the
activiti-bpmn-extensions-5.12.xsd
definitions that could be updated in order to resolve this issue ?

BR

Re: Modeler - Email Task deployment fails

jbarrez — Mon, 25 Mar 2013 13:01:01 GMT

Thanks for researching that. I'll look at it shortly. Do keep an eye on the Jira issue if you want to be informed of fixes.

Re: Modeler - Email Task deployment fails

skay — Mon, 25 Mar 2013 14:18:10 GMT

I began again from scratch / demo process.

I add an EMail task and just fill the
TO
parameter

Then get back to Activiti explorer workspace and export the bpmn.xml file pproduced through KIS Modeler.

This is the genetated XML fragment :

<serviceTask id="sid-EA7D5296-0C91-46A6-AB05-CE4106BCE406" activiti:type="mail">
      <extensionElements>
        <activiti:field name="to" stringValue="jbarrez@activiti.org"></activiti:field>
        <activiti:field name="from"></activiti:field>
        <activiti:field name="subject"></activiti:field>
        <activiti:field name="cc"></activiti:field>
        <activiti:field name="bcc"></activiti:field>
        <activiti:field name="text"></activiti:field>
        <activiti:field name="html"></activiti:field>
        <activiti:field name="charset"></activiti:field>
      </extensionElements>
    </serviceTask>

I'm not shure about if this is a point / still investigating.

Re: Modeler - Email Task deployment fails

skay — Mon, 25 Mar 2013 14:30:09 GMT

Despite of the XSD definition :

<element name="field">
    <annotation>
      <documentation>
        Extension Element for Service Tasks to inject values into the fields of
        delegate classes.
      </documentation>
    </annotation>
    <complexType>
      <choice minOccurs="0" maxOccurs="1">
       <element name="string" type="string" />
       <element name="expression" type="tns:tExpression" />
      </choice>
      <attribute name="name" type="string" use="required"/>
      <attribute name="stringValue" type="string" use="optional" />
      <attribute name="expression" type="tns:tExpression" use="optional" />
    </complexType>
</element>

The error is raised from BpmnParse :

protected FieldDeclaration parseFieldDeclaration(Element serviceTaskElement, Element fieldDeclarationElement) {
    String fieldName = fieldDeclarationElement.attribute("name");

    FieldDeclaration fieldDeclaration = parseStringFieldDeclaration(fieldDeclarationElement, serviceTaskElement, fieldName);
    if (fieldDeclaration == null) {
      fieldDeclaration = parseExpressionFieldDeclaration(fieldDeclarationElement, serviceTaskElement, fieldName);
    }

    if (fieldDeclaration == null) {
      addError("One of the following is mandatory on a field declaration: one of attributes stringValue|expression "
              + "or one of child elements string|expression", serviceTaskElement);
    }
    return fieldDeclaration;
}

My understanding is that we should allow a null fieldDeclaration…
..And then let the XSD drive the rules of the bpmn20.xml :?:

BR

Re: Modeler - Email Task deployment fails

jbarrez — Tue, 26 Mar 2013 10:03:37 GMT

Thanks for gathering that information. It is indeed a bug. We'll fix it asap.

Re: Modeler - Email Task deployment fails

trademak — Tue, 26 Mar 2013 13:51:04 GMT

Hi,

The export of all the empty mail task fields in the Modeler is now fixed.
It should only export the ones that are not empty.
Thanks for reporting the issue.

Best regards,