Description of problem:
if any Command calls:
getReturnValue().setSucceeded(true);
and after that follows some logic, then if this logic fails, outcome of this command will be presented as successful. Compensation/rollbacks etc. are called, but data sent to user are wrong.
example:
if commands ends with:
if (1 == 1) throw new RuntimeException();
getReturnValue().setSucceeded(true);
rest will return:
<fault>
<reason>Operation Failed</reason>
<detail>[Internal Engine Error]</detail>
</fault>
but if command would be like this:
getReturnValue().setSucceeded(true);
if (1 == 1) throw new RuntimeException();
result will be:
<action>
<status>
<state>complete</state>
</status>
</action>
How reproducible:
100%
Actual results:
see above.
Expected results:
command failed in both scenarios and returned data should reflect that.
I think that it should be the responsibility of the command to call setSucceeded(true) only once nothing is left to do... in my opinion, commands that do further logic after that assumes that this logic is insignificant for the successful completion of the command.
Yair/Ravi - thoughts?
I think, on contrary, that if somewhere there need not to be some convention(setSucceed must be last call of command) or faith in assumptions("insignificant" code will not fail), there should not be such things.
Exception can occur unexpectedly, and so does db rollback. Flagging each command from which exception 'leaks' as unsuccessful will spell more robust architecture — user should not get info "OK" if it's not OK, and we should not rely on each command to behave as expected to provide user right response. It's command 'invoker' responsibility.
Description of problem: if any Command calls: getReturnValue().setSucceeded(true); and after that follows some logic, then if this logic fails, outcome of this command will be presented as successful. Compensation/rollbacks etc. are called, but data sent to user are wrong. example: if commands ends with: if (1 == 1) throw new RuntimeException(); getReturnValue().setSucceeded(true); rest will return: <fault> <reason>Operation Failed</reason> <detail>[Internal Engine Error]</detail> </fault> but if command would be like this: getReturnValue().setSucceeded(true); if (1 == 1) throw new RuntimeException(); result will be: <action> <status> <state>complete</state> </status> </action> How reproducible: 100% Actual results: see above. Expected results: command failed in both scenarios and returned data should reflect that.