version_update() does not actually return anything (ie. @return void). This was indeed changed in MantisBT master a80f4476 (1.3.0-beta.1) prior to which it would indeed return true (and only true) in case of success - an error would be triggered otherwise, so no return value in this case.

Using the return value in this case is incorrect, so mc_project_version_update() as well as any other function using version_update's return value should have been updated following the above-mentioned change.

As mantis-axis-soap-client is just generating from the official wsdl you need to change the wsdl first: 0025838

IMO, instead of changing the WSDL, we should make sure that the API mc_project_version_update() returns the expected boolean type.

Could you kindly confirm that the patch below fixes the problem

diff --git a/api/soap/mc_project_api.php b/api/soap/mc_project_api.php
index 17f87582c..73764fa92 100644
--- a/api/soap/mc_project_api.php
+++ b/api/soap/mc_project_api.php
@@ -553,7 +553,8 @@ function mc_project_version_update( $p_username, $p_password, $p_version_id, std
    $t_version_data->date_order = $t_date_order;
    $t_version_data->obsolete = $t_obsolete;

-   return version_update( $t_version_data );
+   version_update( $t_version_data );
+   return true;
 }

 /**

+1

(maybe not only version_update but some other "return"s needs to be re-added to be wsdl conform again
[ from https://github.com/mantisbt/mantisbt/commit/a80f447687722b52160d00c96c21a5be5f4329e1 ]
e.g. token_delete_by_owner , user_pref_update , version_remove )

maybe not only version_update but some other "return"s needs to be re-added to be wsdl conform again
e.g. token_delete_by_owner , user_pref_update , version_remove

OK for version_remove(), I'll include the fix. The return values of the other 2 functions you mention are not used in SOAP API, AFAICT.

If you notice any additional errors / discrepancies between code and WSDL, please open additional issues with relevant details.

PR https://github.com/mantisbt/mantisbt/pull/1517