Refactor if else statement to switch for readability

Author: MaxwellGarceau

mailchimp.php

@@ -1003,54 +1003,70 @@ function mailchimp_sf_merge_submit( $mv ) {
 
 		$opt_val = isset( $_POST[ $opt ] ) ? map_deep( stripslashes_deep( $_POST[ $opt ] ), 'sanitize_text_field' ) : '';
 
-		/**
-		 * US Phone validation
-		 *
-		 * - Merge field is phone
-		 * - Merge field is "included" in the Mailchimp admin options
-		 * - Phone format is set in Mailchimp account
-		 * - Phone format is US in Mailchimp account
-		 */
-		if ( 'phone' === $mv_var['type'] && 'on' === get_option( $opt ) && isset( $mv_var['options']['phone_format'] ) && 'US' === $mv_var['options']['phone_format'] ) {
-			$opt_val = mailchimp_sf_merge_validate_phone( $opt_val, $mv_var );
-			if ( is_wp_error( $opt_val ) ) {
-				return $opt_val;
-			}
-		}
+		switch ( $mv_var['type'] ) {
+			/**
+			 * US Phone validation
+			 *
+			 * - Merge field is phone
+			 * - Merge field is "included" in the Mailchimp admin options
+			 * - Phone format is set in Mailchimp account
+			 * - Phone format is US in Mailchimp account
+			 */
+			case 'phone':
+				if (
+					'on' === get_option( $opt )
+					&& isset( $mv_var['options']['phone_format'] )
+					&& 'US' === $mv_var['options']['phone_format']
+				) {
+					$opt_val = mailchimp_sf_merge_validate_phone( $opt_val, $mv_var );
+					if ( is_wp_error( $opt_val ) ) {
+						return $opt_val;
+					}
+				}
+				break;
 
-		/**
-		 * Address validation
-		 *
-		 * - Merge field is address
-		 * - Merge field is "included" in the Mailchimp admin options
-		 * - Merge field is an array (address contains multiple <input> elements)
-		 */
-		elseif ( 'address' === $mv_var['type'] && 'on' === get_option( $opt ) && is_array( $opt_val ) ) { // Handle address logic
-			$validate = mailchimp_sf_merge_validate_address( $opt_val, $mv_var );
-			if ( is_wp_error( $validate ) ) {
-				return $validate;
-			}
+			/**
+			 * Address validation
+			 *
+			 * - Merge field is address
+			 * - Merge field is "included" in the Mailchimp admin options
+			 * - Merge field is an array (address contains multiple <input> elements)
+			 */
+			case 'address':
+				if ( 'on' === get_option( $opt ) && is_array( $opt_val ) ) {
+					$validate = mailchimp_sf_merge_validate_address( $opt_val, $mv_var );
+					if ( is_wp_error( $validate ) ) {
+						return $validate;
+					}
 
-			if ( $validate ) {
-				$merge->$tag = $validate;
-			}
-			continue;
-		}
+					if ( $validate ) {
+						$merge->$tag = $validate;
+					}
+				}
+				break;
 
-		/**
-		 * Not sure what this is for
-		 */
-		elseif ( is_array( $opt_val ) ) {
-			$keys = array_keys( $opt_val );
-			$val  = new stdClass();
-			foreach ( $keys as $key ) {
-				$val->$key = $opt_val[ $key ];
-			}
-			$opt_val = $val;
+			/**
+			 * Handle generic array values
+			 *
+			 * Not sure what this does or is for
+			 *
+			 * - Merge field is an array, not specifically phone or address
+			 */
+			default:
+				if ( is_array( $opt_val ) ) {
+					$keys = array_keys( $opt_val );
+					$val  = new stdClass();
+					foreach ( $keys as $key ) {
+						$val->$key = $opt_val[ $key ];
+					}
+					$opt_val = $val;
+				}
+				break;
 		}
 
 		/**
 		 * Required fields
+		 *
 		 * If the field is required and empty, return an error
 		 */
 		if ( 'Y' === $mv_var['required'] && trim( $opt_val ) === '' ) {