WordPress_GitHub_Sync_Export - Code Metrics - mAAdhaTTah/wordpress-github-sync - Measure and Improve Code Quality continuously with Scrutinizer

WordPress_GitHub_Sync_Export A
last analyzed 2017-10-09 01:22 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	253
Duplicated Lines	28.06 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

98.32%

Importance

Changes

Metric	Value
dl	71
loc	253
ccs	117
cts	119
cp	0.9832
rs	10
c	0
b	0
f	0
wmc	28
lcom	1
cbo	6

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
B	full()	36	36	5
B	update()	0	39	5
B	new_posts()	0	30	4
B	delete()	35	35	4
B	update_shas()	0	25	6
A	set_user()	0	3	1
A	get_commit_msg_tag()	0	9	2

How to fix Duplicated Code

<?php
/**
 * GitHub Export Manager.
 *
 * @package WordPress_GitHub_Sync
 */

/**
 * Class WordPress_GitHub_Sync_Export
 */
class WordPress_GitHub_Sync_Export {

	/**
	 * Option key for export user.
	 */
	const EXPORT_USER_OPTION = '_wpghs_export_user_id';

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

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

	/**
	 * Updates all of the current posts in the database on master.
	 *
	 * @return string|WP_Error
	 */
	public function full() {

		$posts = $this->app->database()->fetch_all_supported();

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

		$master = $this->app->api()->fetch()->master();

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

		foreach ( $posts as $post ) {
			$master->tree()->add_post_to_tree( $post );
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
		}

		$master->set_message(
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
			apply_filters(
				'wpghs_commit_msg_full',
				sprintf(
					'Full export from WordPress at %s (%s)',
					site_url(),
					get_bloginfo( 'name' )
				)
			) . $this->get_commit_msg_tag()
		);

		$result = $this->app->api()->persist()->commit( $master );
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

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

		return $this->update_shas( $posts );
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
	}

	/**
	 * Updates the provided post ID in master.
	 *
	 * @param int $post_id Post ID to update.
	 *
	 * @return string|WP_Error
	 */
	public function update( $post_id ) {
		$post = $this->app->database()->fetch_by_id( $post_id );

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

		if ( 'trash' === $post->status() ) {
			return $this->delete( $post_id );
		}

		$master = $this->app->api()->fetch()->master();

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

		$master->tree()->add_post_to_tree( $post );
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
		$master->set_message(
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
			apply_filters(
				'wpghs_commit_msg_single',
				sprintf(
					'Syncing %s from WordPress at %s (%s)',
					$post->github_path(),
					site_url(),
					get_bloginfo( 'name' )
				),
				$post
			) . $this->get_commit_msg_tag()
		);

		$result = $this->app->api()->persist()->commit( $master );
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

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

		return $this->update_shas( array( $post ) );
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
	}

	/**
	 * Updates GitHub-created posts with latest WordPress data.
	 *
	 * @param array<WordPress_GitHub_Sync_Post> $posts Array of Posts to create.
	 *
	 * @return string|WP_Error
	 */
	public function new_posts( array $posts ) {
		$master = $this->app->api()->fetch()->master();

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

		foreach ( $posts as $post ) {
			$master->tree()->add_post_to_tree( $post );
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
		}

		$master->set_message(
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
			apply_filters(
				'wpghs_commit_msg_new_posts',
				sprintf(
					'Updating new posts from WordPress at %s (%s)',
					site_url(),
					get_bloginfo( 'name' )
				)
			) . $this->get_commit_msg_tag()
		);

		$result = $this->app->api()->persist()->commit( $master );
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

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

		return $this->update_shas( $posts );
	}

	/**
	 * Deletes a provided post ID from master.
	 *
	 * @param int $post_id Post ID to delete.
	 *
	 * @return string|WP_Error
	 */
	public function delete( $post_id ) {

		$post = $this->app->database()->fetch_by_id( $post_id );


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

		$master = $this->app->api()->fetch()->master();

		if ( is_wp_error( $master ) ) {
			return $master;
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());
}
		}

		$master->tree()->remove_post_from_tree( $post );
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
		$master->set_message(
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
			apply_filters(
				'wpghs_commit_msg_delete',
				sprintf(
					'Deleting %s via WordPress at %s (%s)',
					$post->github_path(),
					site_url(),
					get_bloginfo( 'name' )
				),
				$post
			) . $this->get_commit_msg_tag()
		);

		$result = $this->app->api()->persist()->commit( $master );
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

		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 __( 'Export to GitHub completed successfully.', 'wp-github-sync' );
	}

	/**
	 * Use the new tree to save sha data
	 * for all the updated posts.
	 *
	 * @param WordPress_GitHub_Sync_Post[] $posts Posts to fetch updated shas for.
	 *
	 * @return string|WP_Error
	 */
	protected function update_shas( array $posts ) {
		$master   = $this->app->api()->fetch()->master();
		$attempts = 1;

		while ( is_wp_error( $master ) && $attempts < 5 ) {
			$master = $this->app->api()->fetch()->master();
			$attempts ++;
		}

		if ( is_wp_error( $master ) ) {
			// @todo throw a big warning! not having the latest shas is BAD
			// Solution: Show error message and link to kick off sha importing.
			return $master;
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());
}
		}

		foreach ( $posts as $post ) {
			$blob = $master->tree()->get_blob_by_path( $post->github_path() );
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}

			if ( $blob ) {
				$this->app->database()->set_post_sha( $post, $blob->sha() );
			}
		}

		return __( 'Export to GitHub completed successfully.', 'wp-github-sync' );
	}

	/**
	 * Saves the export user to the database.
	 *
	 * @param int $user_id User ID to export with.
	 *
	 * @return bool
	 */
	public function set_user( $user_id ) {
		return update_option( self::EXPORT_USER_OPTION, (int) $user_id );
	}

	/**
	 * Gets the commit message tag.
	 *
	 * @return string
	 */
	protected function get_commit_msg_tag() {
		$tag = apply_filters( 'wpghs_commit_msg_tag', 'wpghs' );

		if ( ! $tag ) {
			throw new Exception( __( 'Commit message tag not set. Filter `wpghs_commit_msg_tag` misconfigured.', 'wp-github-sync' ) );
		}

		return ' - ' . $tag;
	}
}


WordPress_GitHub_Sync_Export A
last analyzed 2017-10-09 01:22 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

8 Methods

How to fix Duplicated Code

Duplicated Code

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

1			<?php
2			/**
3			* GitHub Export Manager.
4			*
5			* @package WordPress_GitHub_Sync
6			*/
7
8			/**
9			* Class WordPress_GitHub_Sync_Export
10			*/
11			class WordPress_GitHub_Sync_Export {
12
13			/**
14			* Option key for export user.
15			*/
16			const EXPORT_USER_OPTION = '_wpghs_export_user_id';
17
18			/**
19			* Application container.
20			*
21			* @var WordPress_GitHub_Sync
22			*/
23			protected $app;
24
25			/**
26			* Initializes a new export manager.
27			*
28			* @param WordPress_GitHub_Sync $app Application container.
29			*/
30	19		public function __construct( WordPress_GitHub_Sync $app ) {
31	19		$this->app = $app;
32	19		}
33
34			/**
35			* Updates all of the current posts in the database on master.
36			*
37			* @return string\|WP_Error
38			*/
39	5	View Code Duplication	public function full() {
			0 ignored issues – show Duplication introduced 2017-10-06 20:42 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...

mAAdhaTTah / wordpress-github-sync

WordPress_GitHub_Sync_Export A last analyzed 2017-10-09 01:22 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

8 Methods

How to fix Duplicated Code

Duplicated Code

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

WordPress_GitHub_Sync_Export A
last analyzed 2017-10-09 01:22 UTC