Jetpack_Import_Stats

Complexity

Total Complexity

15

Size/Duplication

Total Lines	90
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	8	2
B	get_calling_class()	0	42	10
A	log_import_progress()	0	22	3

<?php

class Jetpack_Import_Stats {
	static $known_importers = array(
		'Blogger_Importer' => 'blogger',
		'LJ_API_Import' => 'livejournal',
		'MT_Import' => 'mt',
		'RSS_Import' => 'rss',
		'WP_Import' => 'wordpress',
	);

	static $action_event_name_map = array(
		'import_start' => 'jetpack_import_start',
		'import_done'  => 'jetpack_import_done',
		'import_end'   => 'jetpack_import_done',
	);

	public static function init() {
		// Only handle import actions for sites that have agreed to TOS 
		if ( Jetpack::jetpack_tos_agreed() ) {
			add_action( 'import_start', array( 'Jetpack_Import_Stats', 'log_import_progress' ) );
			add_action( 'import_done',  array( 'Jetpack_Import_Stats', 'log_import_progress' ) );
			add_action( 'import_end',   array( 'Jetpack_Import_Stats', 'log_import_progress' ) );
		}
	}

	private static function get_calling_class() {
		// If WP_Importer doesn't exist, neither will any importer that extends it
		if ( ! class_exists( 'WP_Importer' ) ){
			return 'unknown';
		}

		$action = current_filter();
		$backtrace = wp_debug_backtrace_summary( null, 0, false );

		$do_action_pos = -1;
		for ( $i = 0; $i < count( $backtrace ); $i++ ) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

			// Find the location in the stack of the calling action
			if ( preg_match( "/^do_action\\(\'([^\']+)/", $backtrace[ $i ], $matches ) ) {
				if ( $matches[1] === $action ) {
					$do_action_pos = $i;
					break;
				}
			}
		}

		// if the action wasn't called, the calling class is unknown
		if ( -1 === $do_action_pos ) {
			return 'unknown';
		}

		// continue iterating the stack looking for a caller that extends WP_Import
		for ( $i = $do_action_pos + 1; $i < count( $backtrace ); $i++ ) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

			// grab only class_name from the trace
			list( $class_name ) = explode( '->', $backtrace[ $i ] );

			// check if the class extends WP_Importer
			if ( class_exists( $class_name ) ) {
				$parents = class_parents( $class_name );
				if ( $parents && in_array( 'WP_Importer', $parents ) ) {

The expression $parents of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

					return $class_name;
				}
			}
		}

		// If we've exhausted the stack without a match, the calling class is unknown
		return 'unknown';
	}

	public static function log_import_progress( $importer ) {
		// prefer self-reported importer-names
		if ( ! $importer ) {
			// fall back to inferring by calling class name
			$importer = self::get_calling_class();
		}
		
		// Give known importers a "friendly" name
		if ( isset( self::$known_importers[ $importer ] ) ) {
			$importer = self::$known_importers[ $importer ];
		}
		$action = current_filter();
		// map action to event name
		$event_name = self::$action_event_name_map[ $action ];
		
		$current_user = wp_get_current_user();

		// Record event to Tracks
		jetpack_tracks_record_event( $current_user, $event_name, array(
			'importer' => $importer,
		) );
	}
}

add_action( 'init', array( 'Jetpack_Import_Stats', 'init' ) );