flowable / flowable-engine

A compact and highly efficient workflow and Business Process Management (BPM) platform for developers, system admins and business users.
https://www.flowable.org
Apache License 2.0
7.77k stars 2.59k forks source link

flowable-modeler: Provide more informative error message when deployment fails? #720

Open PascalSchumacher opened 6 years ago

PascalSchumacher commented 6 years ago

When deploying apps from the modeler some error messages could be even more helpful.

For example when the deployment fails the modeler displays Invalid Deploy result code: HTTP 500

The server logfile includes a detailed error message with the reson why the deployment failed:

05:12:40,957 [http-nio-8080-exec-60] ERROR org.flowable.engine.impl.bpmn.deployer.ParsedDeploymentBuilder  - Could not parse resource Test.bpmn
org.flowable.bpmn.exceptions.XMLException: cvc-datatype-valid.1.2.1: 'Test 2' is not a valid value for 'NCName'.

It could be helpful to propagate these errors to the modeler, what do you think?

pcpiela commented 6 years ago

I am new to Flowable, and just ran into this during my first deploy. I certainly agree with propagating the errors back to the modeler, and ideally the modeler itself could validate this kind of data.