Fix GH-12727: NumberFormatter constructor throws an exception on

invalid locale.

Also re-establishing exception throwing on IntlDateFormatter constructor
overwritten by accident most likely so postponing it for next major
release.

Author: devnexen

ext/intl/dateformat/dateformat_create.cpp

@@ -113,7 +113,8 @@ static zend_result datefmt_ctor(INTERNAL_FUNCTION_PARAMETERS, zend_error_handlin
 	locale = Locale::createFromName(locale_str);
 	/* get*Name accessors being set does not preclude being bogus */
 	if (locale.isBogus() || strlen(locale.getISO3Language()) == 0) {
-		goto error;
+		intl_error_set(NULL, U_ILLEGAL_ARGUMENT_ERROR, "datefmt_create: invalid locale", 0);
+		return FAILURE;
 	}
 
 	/* process calendar */

ext/intl/formatter/formatter_main.c

@@ -17,6 +17,7 @@
 #endif
 
 #include <unicode/ustring.h>
+#include <unicode/uloc.h>
 
 #include "php_intl.h"
 #include "formatter_class.h"
@@ -63,6 +64,11 @@ static int numfmt_ctor(INTERNAL_FUNCTION_PARAMETERS, zend_error_handling *error_
 		locale = intl_locale_get_default();
 	}
 
+	if (strlen(uloc_getISO3Language(locale)) == 0) {
+		zend_argument_value_error(1, "is invalid");
+		return FAILURE;
+	}
+    
 	/* Create an ICU number formatter. */
 	FORMATTER_OBJECT(nfo) = unum_open(style, spattern, spattern_len, locale, NULL, &INTL_DATA_ERROR_CODE(nfo));
 

ext/intl/tests/gh12282.phpt

@@ -5,18 +5,25 @@ intl
 --FILE--
 <?php
 
-var_dump(new IntlDateFormatter(
-	'xx',
-	IntlDateFormatter::FULL,
-	IntlDateFormatter::FULL,
-	null,
-	null,
-	'w'
-));
-Locale::setDefault('xx');
-var_dump(new IntlDateFormatter(Locale::getDefault()));
---EXPECT--
-object(IntlDateFormatter)#1 (0) {
+try {
+    new IntlDateFormatter(
+	    'xx',
+	    IntlDateFormatter::FULL,
+	    IntlDateFormatter::FULL,
+	    null,
+	    null,
+	    'w'
+    );
+} catch (\IntlException $e) {
+    echo $e->getMessage() . PHP_EOL;
 }
-object(IntlDateFormatter)#1 (0) {
+
+Locale::setDefault('xx');
+try {
+    new IntlDateFormatter(Locale::getDefault());
+} catch (\IntlException $e) {
+    echo $e->getMessage();
 }
+--EXPECT--
+datefmt_create: invalid locale: U_ILLEGAL_ARGUMENT_ERROR
+datefmt_create: invalid locale: U_ILLEGAL_ARGUMENT_ERROR

ext/intl/tests/gh12727.phpt

@@ -0,0 +1,15 @@
+--TEST--
+numfmt creation failures
+--EXTENSIONS--
+intl
+--FILE--
+<?php
+
+try {
+    new NumberFormatter('xx', NumberFormatter::DECIMAL);
+} catch (ValueError $e) {
+    echo $e->getMessage();
+}
+?>
+--EXPECT--
+NumberFormatter::__construct(): Argument #1 ($locale) is invalid