Fix to allow for module.run to fail state execution

There was code to check retcode-- but it was done improperly so it would never cause a failure. Also, it seems reasonable that if the return is a boolean we should use that as the result