It would be good if FormService.SubmitFormData and SubmitTaskData can accept null as a parameter to the Map<String, String>. Reason: If I have a cdi scoped bean filled up from a jsf form. I just need a trigger to put the waiting task to move forward. under these circumstances one has to really create an Instance of Map and send it empty thereby consuming unnecessary memory. But if the Submit methods of FormService can check for the existence of null it can just ignore and trigger the task to move forward.
I may be wrong but I have to do this time and again.
But it does not harm checking for an existence of a null value , since many a time we may not be interested in sending those string data types in the Map, when we already have facilities from activiti like spring beans and cdi beans.