Drop usages of E_RECOVERABLE_ERROR

There were only 2 remaining usages.

Author: Girgias

Zend/zend_operators.c

@@ -2540,7 +2540,7 @@ ZEND_API bool ZEND_FASTCALL zend_object_is_true(zval *op) /* {{{ */
 	if (zobj->handlers->cast_object(zobj, &tmp, _IS_BOOL) == SUCCESS) {
 		return Z_TYPE(tmp) == IS_TRUE;
 	}
-	zend_error(E_RECOVERABLE_ERROR, "Object of class %s could not be converted to bool", ZSTR_VAL(zobj->ce->name));
+	zend_type_error("Object of class %s could not be converted to bool", ZSTR_VAL(zobj->ce->name));
 	return false;
 }
 /* }}} */

ext/session/session.c

@@ -551,8 +551,8 @@ static PHP_INI_MH(OnUpdateSaveHandler) /* {{{ */
 	}
 
 	/* "user" save handler should not be set by user */
-	if (!PS(set_handler) &&  tmp == ps_user_ptr) {
-		php_error_docref(NULL, E_RECOVERABLE_ERROR, "Cannot set 'user' save handler by ini_set() or session_module_name()");
+	if (!PS(set_handler) && tmp == ps_user_ptr) {
+		php_error_docref(NULL, E_ERROR, "Cannot set 'user' save handler by ini_set()");
 		return FAILURE;
 	}
 
@@ -1896,6 +1896,10 @@ PHP_FUNCTION(session_module_name)
 	}
 
 	if (name) {
+		if (zend_string_equals_literal_ci(name, "user")) {
+			zend_argument_value_error(1, "cannot be \"user\"");
+			RETURN_THROWS();
+		}
 		if (!_php_find_ps_module(ZSTR_VAL(name))) {
 			php_error_docref(NULL, E_WARNING, "Cannot find named PHP session module (%s)", ZSTR_VAL(name));
 

ext/session/tests/bug60860.phpt

@@ -15,5 +15,5 @@ echo "ok\n";
 
 ?>
 --EXPECT--
-Recoverable fatal error: PHP Startup: Cannot set 'user' save handler by ini_set() or session_module_name() in Unknown on line 0
+Fatal error: PHP Startup: Cannot set 'user' save handler by ini_set() in Unknown on line 0
 ok

ext/session/tests/bug73100.phpt

@@ -14,13 +14,17 @@ var_dump(session_start());
 session_module_name("user");
 var_dump(session_destroy());
 
-session_module_name("user");
+try {
+    session_module_name("user");
+} catch (\ValueError $e) {
+    echo $e->getMessage() . \PHP_EOL;
+}
 ?>
 ===DONE===
 --EXPECTF--
 bool(true)
 
 Warning: session_module_name(): Cannot change save handler module when session is active in %s on line 4
 bool(true)
-
-Recoverable fatal error: session_module_name(): Cannot set 'user' save handler by ini_set() or session_module_name() in %s on line 7
+session_module_name(): Argument #1 ($module) cannot be "user"
+===DONE===

ext/session/tests/error_when_setting_save_handler_ini_setting_in_runtime.phpt

@@ -0,0 +1,12 @@
+--TEST--
+Error when setting session.save_handler to user via ini_set
+--SKIPIF--
+<?php
+include('skipif.inc');
+?>
+--FILE--
+<?php
+ini_set('session.save_handler', 'user');
+?>
+--EXPECTF--
+Fatal error: ini_set(): Cannot set 'user' save handler by ini_set() in %s on line %d

ext/session/tests/session_set_save_handler_class_014.phpt

@@ -21,5 +21,5 @@ session_set_save_handler($handler);
 session_start();
 ?>
 --EXPECT--
-Recoverable fatal error: PHP Startup: Cannot set 'user' save handler by ini_set() or session_module_name() in Unknown on line 0
+Fatal error: PHP Startup: Cannot set 'user' save handler by ini_set() in Unknown on line 0
 *** Testing session_set_save_handler() : calling default handler when save_handler=user ***