Change function name to make it more descriptive

AbdiTolesa · AbdiTolesa · commit 953736209f48 · 2026-01-27T19:18:05.000+03:00
diff --git a/classes/helpers/FrmFieldsHelper.php b/classes/helpers/FrmFieldsHelper.php
@@ -2817,7 +2817,7 @@ private static function get_ai_generate_options_button_class() {
 	 *
 	 * @return bool
 	 */
-	public static function should_disable_option( $choice_key, $field ) {
+	public static function should_hide_field_choice( $choice_key, $field ) {
 		/**
 		 * @since x.x
 		 *
@@ -2856,7 +2856,7 @@ public static function should_skip_rendering_options_for_field( $field ) {
 	 *
 	 * @return bool
 	 */
-	public static function should_echo_disabled_attribute( $choice_key, $is_selected_choice, $field ) {
+	public static function should_disable_option( $choice_key, $is_selected_choice, $field ) {
 		/**
 		 * @since x.x
 		 *
diff --git a/classes/views/frm-fields/front-end/checkbox-field.php b/classes/views/frm-fields/front-end/checkbox-field.php
@@ -26,7 +26,7 @@
 			continue;
 		}
 
-		if ( FrmFieldsHelper::should_disable_option( $opt_key, $field ) ) {
+		if ( FrmFieldsHelper::should_hide_field_choice( $opt_key, $field ) ) {
 			continue;
 		}
 
@@ -57,7 +57,7 @@
 		$other_opt  = false;
 		$other_args = FrmFieldsHelper::prepare_other_input( compact( 'field', 'field_name', 'opt_key', 'field_val' ), $other_opt, $checked );
 
-		$should_echo_disabled_att = FrmFieldsHelper::should_echo_disabled_attribute( $opt_key, $checked, $field );
+		$should_echo_disabled_att = FrmFieldsHelper::should_disable_option( $opt_key, $checked, $field );
 		?>
 		<div class="<?php echo esc_attr( apply_filters( 'frm_checkbox_class', 'frm_checkbox', $field, $field_val ) ); ?>" id="<?php echo esc_attr( FrmFieldsHelper::get_checkbox_id( $field, $opt_key ) ); ?>"><?php
 
diff --git a/classes/views/frm-fields/front-end/dropdown-field.php b/classes/views/frm-fields/front-end/dropdown-field.php
@@ -50,7 +50,7 @@
 	}
 
 	foreach ( $field['options'] as $opt_key => $opt ) {
-		if ( FrmFieldsHelper::should_disable_option( $opt_key, $field ) ) {
+		if ( FrmFieldsHelper::should_hide_field_choice( $opt_key, $field ) ) {
 			continue;
 		}
 
@@ -80,7 +80,7 @@
 			$option_params['class'] = 'frm_other_trigger';
 		}
 
-		if ( FrmFieldsHelper::should_echo_disabled_attribute( $opt_key, $selected, $field ) ) {
+		if ( FrmFieldsHelper::should_disable_option( $opt_key, $selected, $field ) ) {
 			$option_params['disabled'] = 'disabled';
 		}
 
diff --git a/classes/views/frm-fields/front-end/radio-field.php b/classes/views/frm-fields/front-end/radio-field.php
@@ -25,7 +25,7 @@
 			continue;
 		}
 
-		if ( FrmFieldsHelper::should_disable_option( $opt_key, $field ) ) {
+		if ( FrmFieldsHelper::should_hide_field_choice( $opt_key, $field ) ) {
 			continue;
 		}
 
@@ -45,7 +45,7 @@
 		<div class="<?php echo esc_attr( apply_filters( 'frm_radio_class', 'frm_radio', $field, $field_val ) ); ?>" id="<?php echo esc_attr( FrmFieldsHelper::get_checkbox_id( $field, $opt_key, 'radio' ) ); ?>"><?php
 
 		$checked                  = FrmAppHelper::check_selected( $field['value'], $field_val ) ? 'checked="checked" ' : ' ';
-		$should_echo_disabled_att = FrmFieldsHelper::should_echo_disabled_attribute( $opt_key, trim( $checked ) !== '', $field );
+		$should_echo_disabled_att = FrmFieldsHelper::should_disable_option( $opt_key, trim( $checked ) !== '', $field );
 
 		if ( $include_label ) {
 			$label_attributes = array(

Original file line number	Diff line number	Diff line change
`@@ -2817,7 +2817,7 @@ private static function get_ai_generate_options_button_class() {`
`2817`	`2817`	`*`
`2818`	`2818`	`* @return bool`
`2819`	`2819`	`*/`
`2820`		`- public static function should_disable_option( $choice_key, $field ) {`
	`2820`	`+ public static function should_hide_field_choice( $choice_key, $field ) {`
`2821`	`2821`	`/**`
`2822`	`2822`	`* @since x.x`
`2823`	`2823`	`*`
`@@ -2856,7 +2856,7 @@ public static function should_skip_rendering_options_for_field( $field ) {`
`2856`	`2856`	`*`
`2857`	`2857`	`* @return bool`
`2858`	`2858`	`*/`
`2859`		`- public static function should_echo_disabled_attribute( $choice_key, $is_selected_choice, $field ) {`
	`2859`	`+ public static function should_disable_option( $choice_key, $is_selected_choice, $field ) {`
`2860`	`2860`	`/**`
`2861`	`2861`	`* @since x.x`
`2862`	`2862`	`*`
Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@`
`26`	`26`	`continue;`
`27`	`27`	`}`
`28`	`28`
`29`		`- if ( FrmFieldsHelper::should_disable_option( $opt_key, $field ) ) {`
	`29`	`+ if ( FrmFieldsHelper::should_hide_field_choice( $opt_key, $field ) ) {`
`30`	`30`	`continue;`
`31`	`31`	`}`
`32`	`32`
`@@ -57,7 +57,7 @@`
`57`	`57`	`$other_opt = false;`
`58`	`58`	`$other_args = FrmFieldsHelper::prepare_other_input( compact( 'field', 'field_name', 'opt_key', 'field_val' ), $other_opt, $checked );`
`59`	`59`
`60`		`- $should_echo_disabled_att = FrmFieldsHelper::should_echo_disabled_attribute( $opt_key, $checked, $field );`
	`60`	`+ $should_echo_disabled_att = FrmFieldsHelper::should_disable_option( $opt_key, $checked, $field );`
`61`	`61`	`?>`
`62`	`62`	`<div class="<?php echo esc_attr( apply_filters( 'frm_checkbox_class', 'frm_checkbox', $field, $field_val ) ); ?>" id="<?php echo esc_attr( FrmFieldsHelper::get_checkbox_id( $field, $opt_key ) ); ?>"><?php`
`63`	`63`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@`
`50`	`50`	`}`
`51`	`51`
`52`	`52`	`foreach ( $field['options'] as $opt_key => $opt ) {`
`53`		`- if ( FrmFieldsHelper::should_disable_option( $opt_key, $field ) ) {`
	`53`	`+ if ( FrmFieldsHelper::should_hide_field_choice( $opt_key, $field ) ) {`
`54`	`54`	`continue;`
`55`	`55`	`}`
`56`	`56`
`@@ -80,7 +80,7 @@`
`80`	`80`	`$option_params['class'] = 'frm_other_trigger';`
`81`	`81`	`}`
`82`	`82`
`83`		`- if ( FrmFieldsHelper::should_echo_disabled_attribute( $opt_key, $selected, $field ) ) {`
	`83`	`+ if ( FrmFieldsHelper::should_disable_option( $opt_key, $selected, $field ) ) {`
`84`	`84`	`$option_params['disabled'] = 'disabled';`
`85`	`85`	`}`
`86`	`86`
Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@`
`25`	`25`	`continue;`
`26`	`26`	`}`
`27`	`27`
`28`		`- if ( FrmFieldsHelper::should_disable_option( $opt_key, $field ) ) {`
	`28`	`+ if ( FrmFieldsHelper::should_hide_field_choice( $opt_key, $field ) ) {`
`29`	`29`	`continue;`
`30`	`30`	`}`
`31`	`31`
`@@ -45,7 +45,7 @@`
`45`	`45`	`<div class="<?php echo esc_attr( apply_filters( 'frm_radio_class', 'frm_radio', $field, $field_val ) ); ?>" id="<?php echo esc_attr( FrmFieldsHelper::get_checkbox_id( $field, $opt_key, 'radio' ) ); ?>"><?php`
`46`	`46`
`47`	`47`	`$checked = FrmAppHelper::check_selected( $field['value'], $field_val ) ? 'checked="checked" ' : ' ';`
`48`		`- $should_echo_disabled_att = FrmFieldsHelper::should_echo_disabled_attribute( $opt_key, trim( $checked ) !== '', $field );`
	`48`	`+ $should_echo_disabled_att = FrmFieldsHelper::should_disable_option( $opt_key, trim( $checked ) !== '', $field );`
`49`	`49`
`50`	`50`	`if ( $include_label ) {`
`51`	`51`	`$label_attributes = array(`