vmiller/Digital-Dabei-Hamburg-Job-Manager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(07-01): use loose comparison for form ID check in registration handler

Browse Source 
- Changed !== to != to allow type coercion
- Fixes registration not working due to string vs int comparison
- Added type debugging to help diagnose similar issues
This commit is contained in:
Viktor Miller
2026-01-14 21:55:15 +09:00
parent 7286133bf4
commit d6304d934a
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
includes/class-formidable.php
View File

@@ -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;
}