WordPress_GitHub_Sync_Import - Code Metrics - Inspection of "comment formatting fix" - kennyfraser/wordpress-github-sync - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — development (882501)

by Kenny

created 2016-03-17 20:46 UTC

WordPress_GitHub_Sync_Import B

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	325
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes	17
Bugs	3	Features	3

Metric	Value
wmc	46
c	17
b	3
f	3
lcom	1
cbo	8
dl	0
loc	325
rs	8.3999

11 Methods

Rating	Name	Size	Complexity
A	__construct()	3	1
C	payload()	36	7
A	master()	3	1
D	commit()	93	16
A	get_blob_processor()	12	4
A	is_excluded_path()	11	2
A	is_excluded_mime_type()	7	1
A	is_excluded_file_extension()	8	1
A	importable_blob()	13	4
A	blob_changed()	13	2
C	blob_to_post()	37	7

How to fix Complexity

<?php
/**
 * GitHub Import Manager
 *
 * @package WordPress_GitHub_Sync
 */

/**
 * Class WordPress_GitHub_Sync_Import
 */
class WordPress_GitHub_Sync_Import {

	/**
	 * Application container.
	 *
	 * @var WordPress_GitHub_Sync
	 */
	protected $app;

	/**
	 * Initializes a new import manager.
	 *
	 * @param WordPress_GitHub_Sync $app Application container.
	 */
	public function __construct( WordPress_GitHub_Sync $app ) {
		$this->app = $app;
	}

	/**
	 * Imports a payload.
	 *
	 * @param WordPress_GitHub_Sync_Payload $payload GitHub payload object.
	 *
	 * @return string|WP_Error
	 */
	public function payload( WordPress_GitHub_Sync_Payload $payload ) {
		/**
		 * Whether there's an error during import.
		 *
		 * @var false|WP_Error $error
		 */
		$error = false;

		$result = $this->commit( $this->app->api()->fetch()->commit( $payload->get_commit_id() ) );


		if ( is_wp_error( $result ) ) {
			$error = $result;
		}

		$removed = array();
		foreach ( $payload->get_commits() as $commit ) {
			$removed = array_merge( $removed, $commit->removed );
		}
		foreach ( array_unique( $removed ) as $path ) {
			$result = $this->app->database()->delete_post_by_path( $path );

			if ( is_wp_error( $result ) ) {
				if ( $error ) {
					$error->add( $result->get_error_code(), $result->get_error_message() );
				} else {
					$error = $result;
				}
			}
		}

		if ( $error ) {
			return $error;
		}

		return __( 'Payload processed', 'wordpress-github-sync' );
	}

	/**
	 * Imports the latest commit on the master branch.
	 *
	 * @return string|WP_Error
	 */
	public function master() {
		return $this->commit( $this->app->api()->fetch()->master() );

	}

	/**
	 * Imports a provided commit into the database.
	 *
	 * @param WordPress_GitHub_Sync_Commit|WP_Error $commit Commit to import.
	 *
	 * @return string|WP_Error
	 */
	protected function commit( $commit ) {
		if ( is_wp_error( $commit ) ) {
			return $commit;

		}

		if ( $commit->already_synced() ) {
			return new WP_Error( 'commit_synced', __( 'Already synced this commit.', 'wordpress-github-sync' ) );
		}

		$posts = array();
		$new   = array();

		$import_results = array();

		foreach ( $commit->tree()->blobs() as $blob ) {

			$import_results[] = $import_result = (object) array();
			$import_result->path = $blob->path();

			// is this blob importable?
			if ( !$this->importable_blob($blob) ) {

				$import_result->importable = false;
				continue;
			}

			// has the blob changed since the last import?
			if( !$this->blob_changed($blob) ){

				$import_result->changed = false;
				continue;
			}

			// fetch the function that will be used to import the blob
			// based on blob properties
			if( $processor = $this->get_blob_processor($blob) ){

				if( is_callable($processor) ) {


					// call the processor to import the blob
					$result = call_user_func_array($processor,[$blob,$this]);


					// if the result is a Post object ( WordPress_GitHub_Sync_Post )
					if( $result && is_a($result,'WordPress_GitHub_Sync_Post')){


						// add the post to the array of posts to be
						// imported in to the database
						$posts[] = $post = $result;

						if ( $post->is_new() ) {
							// keep track of new posts
							$new[] = $post;
						}

						$import_result->new = $post->is_new();

					}else{

						$import_result->result = $result?$result:false;
					}


				}else{

					$import_result->processor = "not callable";
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
				}


			}else{

				$import_result->processor = "no processor";
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
			}


		}

		// TODO: Log/output results
		// print_r($import_results);

		if( count($posts) ){


			$result = $this->app->database()->save_posts( $posts, $commit->author_email() );

			if ( is_wp_error( $result ) ) {
				return $result;
			}

			if ( $new ) {


				do_action('wpghs_post_new_post_import', $new);


				$result = $this->app->export()->new_posts( $new );

				if ( is_wp_error( $result ) ) {
					return $result;
class Author {
    private $name;

    public function __construct($name) {
        $this->name = $name;
    }

    public function getName() {
        return $this->name;
    }
}

abstract class Post {
    public function getAuthor() {
        return 'Johannes';
    }
}

class BlogPost extends Post {
    public function getAuthor() {
        return new Author('Johannes');
    }
}

class ForumPost extends Post { /* ... */ }

function my_function(Post $post) {
    echo strtoupper($post->getAuthor());
}
				}
			}


		}

		return __( 'No commitable posts', 'wordpress-github-sync' );;
	}

	/**
	 * Get the function that will be used to import the blob, hookable
	 *
	 * @param  WordPress_GitHub_Sync_Blob
	 * @return callable
	 */
	protected function get_blob_processor( WordPress_GitHub_Sync_Blob $blob ) {
		$processor = false;

		// import any plain text .md document in the defaut way
		if ( $blob->has_frontmatter()

				&& $blob->mimetype() == "text/plain"
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
				&& $blob->file_extension() == "md" ) {
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
			$processor = array($this,'blob_to_post');

		}

		return apply_filters('wpghs_get_blob_processor', $processor, $blob);

	}

	/**
	 * Returns true if the path is not importable, hookable
	 *
	 * @param  string
	 * @return boolean
	 */
	protected function is_excluded_path( $path ) {

		$excluded = false;

		// Skip the repo's readme.
		if ( 'readme' === strtolower( substr( $path, 0, 6 ) ) ) {
			$excluded = true;
		}

		return apply_filters('wpghs_exclude_paths', $excluded, $path);

	}

	/**
	 * Returns true if the mine type is not importable, hookable
	 *
	 * @param  string
	 * @return boolean
	 */
	protected function is_excluded_mime_type( $mime_type ) {

		// nothing excuded by default
		$excluded = false;

		return apply_filters('wpghs_exclude_mime_types', $excluded, $mime_type);

	}

	/**
	 * Returns true if the file extension is not importable, hookable
	 *
	 * @param  string
	 * @return boolean
	 */
	protected function is_excluded_file_extension( $file_extension ) {

		// nothing excuded by default
		$excluded = false;

		// TODO: Test this hook
		return apply_filters('wpghs_exclude_file_extensions', $excluded, $file_extension);

	}

	/**
	 * Checks whether the provided blob should be imported.
	 *
	 * @param WordPress_GitHub_Sync_Blob $blob Blob to validate.
	 *
	 * @return bool
	 */
	protected function importable_blob( WordPress_GitHub_Sync_Blob $blob ) {
		global $wpdb;

		$importable = true;

		if( $this->is_excluded_path($blob->path()) ||

			$this->is_excluded_file_extension($blob->file_extension()) ||

			$this->is_excluded_mime_type($blob->path()) ){

			$importable = false;
		}

		return apply_filters('wpghs_importable_blob', $importable, $blob);

	}

	/**
	 * Checks whether the provided blob has changed.
	 *
	 * @param WordPress_GitHub_Sync_Blob $blob Blob to validate.
	 *
	 * @return bool
	 */
	protected function blob_changed( WordPress_GitHub_Sync_Blob $blob ) {
		global $wpdb;

		$changed = true;

		// If the blob sha already matches a post, and the path accociated with the sha
		// has not changed then there has been no change
		if ( $this->app->database()->sha_exists_with_path( $blob->sha(), $blob->path() ) ){
			return false;
		}

		return apply_filters('wpghs_blob_changed', $blob, $changed);

	}

	/**
	 * Imports a single blob content into matching post.
	 *
	 * @param WordPress_GitHub_Sync_Blob $blob Blob to transform into a Post.
	 *
	 * @return WordPress_GitHub_Sync_Post
	 */
	protected function blob_to_post( WordPress_GitHub_Sync_Blob $blob ) {
		$args = array( 'post_content' => $blob->content_import() );
		$meta = $blob->meta();

		if ( $meta ) {

			if ( array_key_exists( 'layout', $meta ) ) {
				$args['post_type'] = $meta['layout'];
				unset( $meta['layout'] );
			}

			if ( array_key_exists( 'published', $meta ) ) {
				$args['post_status'] = true === $meta['published'] ? 'publish' : 'draft';
				unset( $meta['published'] );
			}

			if ( array_key_exists( 'post_title', $meta ) ) {
				$args['post_title'] = $meta['post_title'];
				unset( $meta['post_title'] );
			}

			if ( array_key_exists( 'ID', $meta ) ) {
				$args['ID'] = $meta['ID'];
				unset( $meta['ID'] );
			}
		}

		$meta['_sha'] = $blob->sha();
		$meta['_wpghs_github_path']  = $blob->path();
		
		$meta = apply_filters('wpghs_blob_to_post_meta', $meta, $blob);

		$args = apply_filters('wpghs_blob_to_post_args', $args, $meta, $blob);


		$post = new WordPress_GitHub_Sync_Post( $args, $this->app->api() );
		$post->set_meta( $meta );

		return $post;
	}
}


1			<?php
2			/**
3			* GitHub Import Manager
4			*
5			* @package WordPress_GitHub_Sync
6			*/
7
8			/**
9			* Class WordPress_GitHub_Sync_Import
10			*/
11			class WordPress_GitHub_Sync_Import {
12
13			/**
14			* Application container.
15			*
16			* @var WordPress_GitHub_Sync
17			*/
18			protected $app;
19
20			/**
21			* Initializes a new import manager.
22			*
23			* @param WordPress_GitHub_Sync $app Application container.
24			*/
25			public function __construct( WordPress_GitHub_Sync $app ) {
26			$this->app = $app;
27			}
28
29			/**
30			* Imports a payload.
31			*
32			* @param WordPress_GitHub_Sync_Payload $payload GitHub payload object.
33			*
34			* @return string\|WP_Error
35			*/
36			public function payload( WordPress_GitHub_Sync_Payload $payload ) {
37			/**
38			* Whether there's an error during import.
39			*
40			* @var false\|WP_Error $error

kennyfraser / wordpress-github-sync

Branch — development (882501)

WordPress_GitHub_Sync_Import B

Complexity

Size/Duplication

Coupling/Cohesion

Importance

11 Methods

How to fix Complexity

Complex Class

Duplication Side-by-Side

Filter issues like