includes/admin/upgrades/upgrade-functions.php ➔ give_v20_logs_upgrades()

Complexity

Conditions	15
Paths	400

Size

Total Lines	94
Code Lines	60

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
function give_v20_logs_upgrades() {
	if ( ! current_user_can( 'manage_give_settings' ) ) {
		wp_die( esc_html__( 'You do not have permission to do Give upgrades.', 'give' ), esc_html__( 'Error', 'give' ), array(
			'response' => 403,
		) );
	}

	ignore_user_abort( true );

	if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
		@set_time_limit( 0 );
	}

	global $wpdb;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

	$step = isset( $_GET['step'] ) ? absint( $_GET['step'] ) : 1;

	// form query
	$forms = new WP_Query( array(
			'paged'          => $step,
			'status'         => 'any',
			'order'          => 'DESC',
			'post_type'      => 'give_log',
			'posts_per_page' => 20,
		)
	);

	if ( $forms->have_posts() ) {
		while ( $forms->have_posts() ) {
			$forms->the_post();
			global $post;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

			$term_name = get_the_terms( $post->ID, 'give_log_type' );
			$term_name = ! is_wp_error( $term_name ) && 1 === count( $term_name) ? $term_name[0]->slug : '';

			$log_data = array(
				'ID'       => $post->ID,
				'log_title'    => $post->post_title,
				'log_content'  => $post->post_content,
				'log_parent'   => $post->post_parent,
				'log_type'     => $term_name,
				'log_date'     => $post->post_date,
				'log_date_gmt' => $post->post_date_gmt,
			);
			$log_meta = array();

			if( $old_log_meta = get_post_meta( $post->ID ) ) {
				foreach ( $old_log_meta as $meta_key => $meta_value ) {
					switch ( $meta_key ) {
						case '_give_log_payment_id':
							$log_data['log_parent'] = current( $meta_value );
							$log_meta['_give_log_form_id'] = $post->post_parent;
							break;

						default:
							$log_meta[$meta_key] = current(  $meta_value );
					}
				}
			}

			if( 'api_request' === $term_name ){
				$log_meta['api_query'] = $post->post_excerpt;
			}

			$wpdb->insert( "{$wpdb->prefix}give_logs", $log_data );

			if( ! empty( $log_meta ) ) {
				foreach ( $log_meta as $meta_key => $meta_value ){
					Give()->logs->logmeta_db->update_meta( $post->ID, $meta_key, $meta_value );
				}
			}

		}// End while().

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

		wp_reset_postdata();

		// Forms found so upgrade them
		$step ++;
		$redirect = add_query_arg( array(
			'page'         => 'give-upgrades',
			'give-upgrade' => 'give_v20_logs_upgrades',
			'step'         => $step,
		), admin_url( 'index.php' ) );
		wp_redirect( $redirect );
		exit();

The function give_v20_logs_upgrades() contains an exit expression.

	} else {
		// No more forms found, finish up.
		update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ) );
		delete_option( 'give_doing_upgrade' );
		give_set_upgrade_complete( 'v20_upgrades_form_metadata' );

		wp_redirect( admin_url() );
		exit;

The function give_v20_logs_upgrades() contains an exit expression.

	}
}