From d6304d934a3d41faa002d1a39b53ee0e6b8580c5 Mon Sep 17 00:00:00 2001 From: Viktor Miller Date: Wed, 14 Jan 2026 21:55:15 +0900 Subject: [PATCH] fix(07-01): use loose comparison for form ID check in registration handler - Changed !== to != to allow type coercion - Fixes registration not working due to string vs int comparison - Added type debugging to help diagnose similar issues --- includes/class-formidable.php | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/includes/class-formidable.php b/includes/class-formidable.php index 0687b7d..2361fd1 100644 --- a/includes/class-formidable.php +++ b/includes/class-formidable.php @@ -268,11 +268,11 @@ class DDHH_JM_Formidable { * @param int $form_id Form ID. */ public static function handle_registration_submission( $entry_id, $form_id ) { - error_log('DDHH Registration: Hook fired for form ' . $form_id . ', entry ' . $entry_id); + error_log('DDHH Registration: Hook fired for form ' . $form_id . ' (' . gettype($form_id) . '), entry ' . $entry_id); - // Only process our registration form - if ( $form_id !== self::get_registration_form_id() ) { - error_log('DDHH Registration: Wrong form ID. Expected ' . self::get_registration_form_id()); + // Only process our registration form (use loose comparison for type coercion) + if ( $form_id != self::get_registration_form_id() ) { + error_log('DDHH Registration: Wrong form ID. Expected ' . self::get_registration_form_id() . ' (' . gettype(self::get_registration_form_id()) . ')'); return; }