FooGallery_Upgrade

Complexity

Total Complexity

9

Size/Duplication

Total Lines	171
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	upgrade_gallery()	0	8	2
B	perform_gallery_settings_upgrade()	0	148	6

<?php
/**
 * Class used to upgrade internal gallery settings when needed
 * Date: 19/07/2017
 */
if ( ! class_exists( 'FooGallery_Upgrade' ) ) {

	class FooGallery_Upgrade {

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

		function __construct() {

It is generally recommended to explicitly declare the visibility for methods.

			//intercept the gallery load and check if we need to upgrade a gallery
			add_action( 'foogallery_foogallery_instance_after_load', array( $this, 'upgrade_gallery' ), 10, 2 );
		}

		/**
		 * Checks if the gallery needs to be upgraded based on the settings version
		 *
		 * @param $foogallery FooGallery
		 * @param $post
		 */
		function upgrade_gallery( $foogallery, $post ) {

The parameter $post is not used and could be removed.

It is generally recommended to explicitly declare the visibility for methods.

			if ( FOOGALLERY_SETTINGS_VERSION !== $foogallery->settings_version ) {
				$this->perform_gallery_settings_upgrade( $foogallery );

				//update the settings version for the gallery
				//update_post_meta( $post->ID, FOOGALLERY_META_SETTINGS_VERSION, FOOGALLERY_SETTINGS_VERSION );

47% of this comment could be valid code. Did you maybe forget this after debugging?

			}
		}

		function perform_gallery_settings_upgrade( $foogallery ) {

It is generally recommended to explicitly declare the visibility for methods.

			$mappings = array(
				array(
					'id' => 'border-style',
					'value' => 'border-style-square-white',
					'new' => array(
						array ( 'id' => 'theme', 'value' => 'fg-light' ),
						array ( 'id' => 'border-size', 'value' => 'fg-border-thin' ),
						array ( 'id' => 'drop-shadow', 'value' => 'fg-shadow-small' )
					)
				),
				array(
					'id' => 'border-style',
					'value' => 'border-style-circle-white',
					'new' => array(
						array ( 'id' => 'theme', 'value' => 'fg-light' ),
						array ( 'id' => 'border-size', 'value' => 'fg-border-thin' ),
						array ( 'id' => 'drop-shadow', 'value' => 'fg-shadow-small' ),
						array ( 'id' => 'rounded-corners', 'value' => 'fg-round-full' )
					)
				),
				array(
					'id' => 'border-style',
					'value' => 'border-style-square-black',
					'new' => array(
						array ( 'id' => 'theme', 'value' => 'fg-dark' ),
						array ( 'id' => 'border-size', 'value' => 'fg-border-thin' ),
						array ( 'id' => 'drop-shadow', 'value' => 'fg-shadow-small' )
					)
				),
				array(
					'id' => 'border-style',
					'value' => 'border-style-circle-black',
					'new' => array(
						array ( 'id' => 'theme', 'value' => 'fg-dark' ),
						array ( 'id' => 'border-size', 'value' => 'fg-border-thin' ),
						array ( 'id' => 'drop-shadow', 'value' => 'fg-shadow-small' ),
						array ( 'id' => 'rounded-corners', 'value' => 'fg-round-full' )
					)
				),
				array(
					'id' => 'border-style',
					'value' => 'border-style-inset',
					'new' => array(
						array ( 'id' => 'theme', 'value' => 'fg-light' ),
						array ( 'id' => 'inset-shadow', 'value' => 'fg-shadow-inset-large' )
					)
				),
				array(
					'id' => 'border-style',
					'value' => 'border-style-rounded',
					'new' => array(
						array ( 'id' => 'theme', 'value' => 'fg-light' ),
						array ( 'id' => 'rounded-corners', 'value' => 'fg-round-small' )
					)
				),
				array(
					'id' => 'spacing',
					'value' => 'spacing-width-0',
					'new' => array(
						array ( 'id' => 'spacing', 'value' => 'fg-gutter-0' )
					)
				),
				array(
					'id' => 'spacing',
					'value' => 'spacing-width-5',
					'new' => array(
						array ( 'id' => 'spacing', 'value' => 'fg-gutter-5' )
					)
				),
				array(
					'id' => 'spacing',
					'value' => 'spacing-width-10',
					'new' => array(
						array ( 'id' => 'spacing', 'value' => 'fg-gutter-10' )
					)
				),
				array(
					'id' => 'spacing',
					'value' => 'spacing-width-15',
					'new' => array(
						array ( 'id' => 'spacing', 'value' => 'fg-gutter-15' )
					)
				),
				array(
					'id' => 'spacing',
					'value' => 'spacing-width-20',
					'new' => array(
						array ( 'id' => 'spacing', 'value' => 'fg-gutter-20' )
					)
				),
				array(
					'id' => 'spacing',
					'value' => 'spacing-width-25',
					'new' => array(
						array ( 'id' => 'spacing', 'value' => 'fg-gutter-25' )
					)
				),

				array(
					'id' => 'alignment',
					'value' => 'alignment-left',
					'new' => array(
						array ( 'id' => 'alignment', 'value' => 'fg-left' )
					)
				),
				array(
					'id' => 'alignment',
					'value' => 'alignment-center',
					'new' => array(
						array ( 'id' => 'alignment', 'value' => 'fg-center' )
					)
				),
				array(
					'id' => 'alignment',
					'value' => 'alignment-right',
					'new' => array(
						array ( 'id' => 'alignment', 'value' => 'fg-right' )
					)
				),
			);

			//upgrade all template settings
			foreach ( foogallery_gallery_templates() as $template ) {

				foreach ( $mappings as $mapping ) {

					$settings_key = "{$template['slug']}_{$mapping['id']}";

					//check if the settings exists
					if ( array_key_exists( $settings_key, $foogallery->settings ) ) {

						$old_settings_value = $foogallery->settings[$settings_key];

						if ( $mapping['value'] === $old_settings_value ) {
							//we have found a match!

							foreach ( $mapping['new'] as $setting_to_create ) {
								$new_setting_key = "{$template['slug']}_{$setting_to_create['id']}";
								$new_setting_value = $setting_to_create['value'];
								$foogallery->settings[$new_setting_key] = $new_setting_value;
							}
						}
					}
				}
			}
		}
	}
}