Merge pull request #4500 from csears123/patch-4

PHP 8.1 deprecation error, add return type for CMssql-classes

Author: marcovtwout

CHANGELOG

@@ -5,6 +5,7 @@ Version 1.1.28 under development
 --------------------------------
 
 - Bug #4491: Fixed limit and Offset not working correctly with MSSQL version 11 (2012) and newer (shnoulle, wtommyw)
+- Bug #4500: PHP 8.1 compatibility: Fix deprecation warnings in CMysql classes (csears123)
 
 Version 1.1.27 November 21, 2022
 --------------------------------

framework/db/schema/mssql/CMssqlPdoAdapter.php

@@ -23,6 +23,7 @@ class CMssqlPdoAdapter extends PDO
 	 * @param string|null sequence name. Defaults to null
 	 * @return integer last inserted id
 	 */
+	#[ReturnTypeWillChange]
 	public function lastInsertId ($sequence=NULL)
 	{
 		return $this->query('SELECT CAST(COALESCE(SCOPE_IDENTITY(), @@IDENTITY) AS bigint)')->fetchColumn();
@@ -36,6 +37,7 @@ public function lastInsertId ($sequence=NULL)
 	 *
 	 * @return boolean
 	 */
+	#[ReturnTypeWillChange]
 	public function beginTransaction ()
 	{
 		$this->exec('BEGIN TRANSACTION');
@@ -50,6 +52,7 @@ public function beginTransaction ()
 	 *
 	 * @return boolean
 	 */
+	#[ReturnTypeWillChange]
 	public function commit ()
 	{
 		$this->exec('COMMIT TRANSACTION');
@@ -64,6 +67,7 @@ public function commit ()
 	 *
 	 * @return boolean
 	 */
+	#[ReturnTypeWillChange]
 	public function rollBack ()
 	{
 		$this->exec('ROLLBACK TRANSACTION');

framework/db/schema/mssql/CMssqlSqlsrvPdoAdapter.php

@@ -30,6 +30,7 @@ class CMssqlSqlsrvPdoAdapter extends PDO
 	 * @param string|null $sequence the sequence/table name. Defaults to null.
 	 * @return integer last inserted ID value.
 	 */
+	#[ReturnTypeWillChange]
 	public function lastInsertId($sequence=null)
 	{
 		$parts = explode('.', phpversion('pdo_sqlsrv'));