ext/soap: Convert bailouts to normal exception throwing in SoapClient::__constructor()

Use ValueErrors as the conditions checked are programming errors

Author: Girgias

ext/soap/soap.c

@@ -1935,8 +1935,6 @@ PHP_METHOD(SoapClient, __construct)
 		RETURN_THROWS();
 	}
 
-	SOAP_CLIENT_BEGIN_CODE();
-
 	cache_wsdl = SOAP_GLOBAL(cache_enabled) ? SOAP_GLOBAL(cache_mode) : 0;
 
 	if (options != NULL) {
@@ -1949,7 +1947,8 @@ PHP_METHOD(SoapClient, __construct)
 			    Z_TYPE_P(tmp) == IS_STRING) {
 				ZVAL_STR_COPY(Z_CLIENT_URI_P(this_ptr), Z_STR_P(tmp));
 			} else {
-				php_error_docref(NULL, E_ERROR, "'uri' option is required in nonWSDL mode");
+				zend_argument_value_error(2, "must provide \"uri\" option as it is required in nonWSDL mode");
+				RETURN_THROWS();
 			}
 
 			if ((tmp = zend_hash_str_find(ht, "style", sizeof("style")-1)) != NULL &&
@@ -1977,7 +1976,8 @@ PHP_METHOD(SoapClient, __construct)
 		    Z_TYPE_P(tmp) == IS_STRING) {
 			ZVAL_STR_COPY(Z_CLIENT_LOCATION_P(this_ptr), Z_STR_P(tmp));
 		} else if (!wsdl) {
-			php_error_docref(NULL, E_ERROR, "'location' option is required in nonWSDL mode");
+			zend_argument_value_error(2, "must provide \"location\" option as it is required in nonWSDL mode");
+			RETURN_THROWS();
 		}
 
 		if ((tmp = zend_hash_str_find(ht, "soap_version", sizeof("soap_version")-1)) != NULL) {
@@ -2056,7 +2056,8 @@ PHP_METHOD(SoapClient, __construct)
 
 			encoding = xmlFindCharEncodingHandler(Z_STRVAL_P(tmp));
 			if (encoding == NULL) {
-				php_error_docref(NULL, E_ERROR, "Invalid 'encoding' option - '%s'", Z_STRVAL_P(tmp));
+				zend_argument_value_error(2, "provided \"encoding\" option \"%s\" is invalid", Z_STRVAL_P(tmp));
+				RETURN_THROWS();
 			} else {
 				xmlCharEncCloseFunc(encoding);
 				ZVAL_STR_COPY(Z_CLIENT_ENCODING_P(this_ptr), Z_STR_P(tmp));
@@ -2112,10 +2113,15 @@ PHP_METHOD(SoapClient, __construct)
 				"The \"ssl_method\" option is deprecated. "
 				"Use \"ssl\" stream context options instead");
 		}
-	} else if (!wsdl) {
-		php_error_docref(NULL, E_ERROR, "'location' and 'uri' options are required in nonWSDL mode");
 	}
 
+	if (options == NULL && wsdl == NULL) {
+		zend_argument_value_error(2, "must provide \"uri\" and \"location\" options as they are required in nonWSDL mode");
+		RETURN_THROWS();
+	}
+
+	SOAP_CLIENT_BEGIN_CODE();
+
 	ZVAL_LONG(Z_CLIENT_SOAP_VERSION_P(this_ptr), soap_version);
 
 	if (wsdl) {

ext/soap/tests/SoapClient/invalid-encoding-option.phpt

@@ -25,5 +25,5 @@ try {
 
 ?>
 --EXPECT--
-SoapFault: SoapClient::__construct(): Invalid 'encoding' option - 'non-sense'
-SoapFault: SoapClient::__construct(): Invalid 'encoding' option - 'non-sense'
+ValueError: SoapClient::__construct(): Argument #2 ($options) provided "encoding" option "non-sense" is invalid
+ValueError: SoapClient::__construct(): Argument #2 ($options) provided "encoding" option "non-sense" is invalid

ext/soap/tests/SoapClient/missing-options-non-wsdl-mode.phpt

@@ -48,11 +48,11 @@ try {
 ?>
 --EXPECT--
 $options not provided
-SoapFault: SoapClient::__construct(): 'location' and 'uri' options are required in nonWSDL mode
-SoapFault: SoapClient::__construct(): 'location' and 'uri' options are required in nonWSDL mode
+ValueError: SoapClient::__construct(): Argument #2 ($options) must provide "uri" and "location" options as they are required in nonWSDL mode
+ValueError: SoapClient::__construct(): Argument #2 ($options) must provide "uri" and "location" options as they are required in nonWSDL mode
 Empty $options array
-SoapFault: SoapClient::__construct(): 'uri' option is required in nonWSDL mode
-SoapFault: SoapClient::__construct(): 'uri' option is required in nonWSDL mode
+ValueError: SoapClient::__construct(): Argument #2 ($options) must provide "uri" option as it is required in nonWSDL mode
+ValueError: SoapClient::__construct(): Argument #2 ($options) must provide "uri" option as it is required in nonWSDL mode
 $options array only sets "uri" option
-SoapFault: SoapClient::__construct(): 'location' option is required in nonWSDL mode
-SoapFault: SoapClient::__construct(): 'location' option is required in nonWSDL mode
+ValueError: SoapClient::__construct(): Argument #2 ($options) must provide "location" option as it is required in nonWSDL mode
+ValueError: SoapClient::__construct(): Argument #2 ($options) must provide "location" option as it is required in nonWSDL mode