handle PHP8.4 deprecation warnings

corsacca · corsacca · commit 4427691d8e90 · 2025-06-30T15:00:52.000+01:00
diff --git a/dt-mapping/class-migration-engine.php b/dt-mapping/class-migration-engine.php
@@ -144,7 +144,7 @@ public static function get_current_db_migration(){
 
 class DT_Mapping_Module_Migration_Lock_Exception extends Exception
 {
-    public function __construct( $message = null, $code = 0, Exception $previous = null ) {
+    public function __construct( $message = null, $code = 0, ?Exception $previous = null ) {
         /*
          * Instead of throwing a simple exception that the migration lock is
          * held, it would be good for the user to if there any previous errors,
diff --git a/includes/class-migration-engine.php b/includes/class-migration-engine.php
@@ -140,7 +140,7 @@ protected static function sanity_check_expected_tables( array $expected_tables )
 
 class DT_Webform_Migration_Lock_Exception extends Exception
 {
-    public function __construct( $message = null, $code = 0, Exception $previous = null ) {
+    public function __construct( $message = null, $code = 0, ?Exception $previous = null ) {
         /*
          * Instead of throwing a simple exception that the migration lock is
          * held, it would be good for the user to if there any previous errors,

Original file line number	Diff line number	Diff line change
`@@ -144,7 +144,7 @@ public static function get_current_db_migration(){`
`144`	`144`
`145`	`145`	`class DT_Mapping_Module_Migration_Lock_Exception extends Exception`
`146`	`146`	`{`
`147`		`- public function __construct( $message = null, $code = 0, Exception $previous = null ) {`
	`147`	`+ public function __construct( $message = null, $code = 0, ?Exception $previous = null ) {`
`148`	`148`	`/*`
`149`	`149`	`* Instead of throwing a simple exception that the migration lock is`
`150`	`150`	`* held, it would be good for the user to if there any previous errors,`
Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,7 @@ protected static function sanity_check_expected_tables( array $expected_tables )`
`140`	`140`
`141`	`141`	`class DT_Webform_Migration_Lock_Exception extends Exception`
`142`	`142`	`{`
`143`		`- public function __construct( $message = null, $code = 0, Exception $previous = null ) {`
	`143`	`+ public function __construct( $message = null, $code = 0, ?Exception $previous = null ) {`
`144`	`144`	`/*`
`145`	`145`	`* Instead of throwing a simple exception that the migration lock is`
`146`	`146`	`* held, it would be good for the user to if there any previous errors,`