Kirki_GoogleFonts_Field_Processor

Complexity

Total Complexity

21

Size/Duplication

Total Lines	93
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes	2
Bugs	1	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
C	generate_google_fonts()	0	75	21

<?php

class Kirki_GoogleFonts_Field_Processor extends Kirki_Fonts {

Since you have declared the constructor as private, maybe you should also declare the class as final.

	/**
	 * The one true instance of this object
	 */
	private static $instance = null;

The property $instance is not used and could be removed.

Consider using a different property name as you override a private property of the parent class.

	/**
	 * iterate all our fields and find the ones that need are typography-related.
	 * Typography fields will be processed
	 * and call the Kirki_GoogleFonts_Manager class to add their fonts.
	 * Other typography fields will add their values globally
	 * to the Kirki_GoogleFonts_Manager class.
	 *
	 * @return void
	 */
	public static function generate_google_fonts( $field ) {
		/**
		 * Process typography fields
		 */
		if ( isset( $field['type'] ) && 'typography' == $field['type'] ) {
			// Get the value
			$value = Kirki_Values::get_sanitized_field_value( $field );
			// If we don't have a font-family then we can skip this.
			if ( ! isset( $value['font-family'] ) ) {
				return;
			}
			// Set a default value for font-weight
			if ( ! isset( $value['font-weight'] ) ) {
				$value['font-weight'] = 400;
			}
			if ( isset( $value['subset'] ) ) {
				// Add the subset directly to the array of subsets
				// in the Kirki_GoogleFonts_Manager object.
				if ( ! is_array( $value['subset'] ) ) {
					Kirki_GoogleFonts_Manager::$subsets[] = $value['subset'];
				} else {
					foreach ( $value['subset'] as $subset ) {
						Kirki_GoogleFonts_Manager::$subsets[] = $subset;
					}
				}
			}
			// Get the font-style
			if ( ! isset( $value['font-style'] ) ) {
				$value['font-style'] = 'regular';
			}
			// Add the requested google-font
			Kirki_GoogleFonts_Manager::add_font( $value['font-family'], $value['font-weight'], $value['font-style'] );
			// Add font-weight 700 so that bold works for all fonts
			Kirki_GoogleFonts_Manager::add_font( $value['font-family'], 700, $value['font-style'] );
			// Any additional font-weights we may need
			foreach ( Kirki_GoogleFonts_Manager::$font_weights as $font_weight ) {
				Kirki_GoogleFonts_Manager::add_font( $value['font-family'], 700, $value['font-style'] );
			}
		}

		/**
		 * Process non-typography fields
		 */
		else {
			if ( isset( $field['output'] ) && is_array( $field['output'] ) ) {
				foreach ( $field['output'] as $output ) {
					// If we don't have a typography-related output argument we can skip this.
					if ( ! isset( $output['property'] ) ||  ! in_array( $output['property'], array( 'font-family', 'font-weight', 'font-subset', 'subset', 'font-style' ) ) ) {

Expected 1 space before "!"; 2 found

						continue;
					}
					// Get the value
					$value = Kirki_Values::get_sanitized_field_value( $field );

					// font-family
					if ( 'font-family' == $output['property'] ) {
						Kirki_GoogleFonts_Manager::add_font( $value );
					}
					// font-weight
					elseif ( 'font-weight' == $output['property'] ) {
						Kirki_GoogleFonts_Manager::$font_weights[] = $value;
					}
					// subsets
					elseif ( 'font-subset' == $output['property'] || 'subset' == $output['property'] ) {
						if ( ! is_array( $value ) ) {
							Kirki_GoogleFonts_Manager::$subsets[] = $value;
						} else {
							foreach ( $value as $subset ) {
								Kirki_GoogleFonts_Manager::$subsets[] = $subset;
							}
						}
					}
				}
			}
		}
	}

}