Writing_On_GitHub_Post

Complexity

Total Complexity

60

Size/Duplication

Total Lines	479
Duplicated Lines	5.22 %

Coupling/Cohesion

Dependencies

3

Test Coverage

Coverage

0%

Importance

Changes

0

31 Methods

Rating	Name	Duplication	Size	Complexity
A	set_sha()	0	2	1
A	get_directory_from_path()	0	5	2
A	sha()	0	22	3
A	is_new()	0	2	1
A	get_meta()	0	2	1
A	to_blob()	0	8	1
A	get_blob()	0	2	1
A	github_content()	0	7	3
B	meta()	0	36	6
B	__construct()	0	20	5
A	has_password()	0	2	1
A	set_post()	0	5	1
A	set_meta()	0	2	1
A	is_on_github()	0	7	3
A	github_edit_url()	0	6	1
A	id()	0	2	1
A	old_github_path()	0	2	1
A	set_blob()	0	2	1
A	status()	0	2	1
A	github_path()	0	4	1
A	set_old_github_path()	0	3	1
A	front_matter()	0	2	1
A	get_args()	0	2	1
A	github_filename()	0	8	2
A	github_view_url()	0	6	1
A	get_name()	0	6	2
A	last_modified_author()	0	10	3
A	post_content()	0	12	4
C	github_directory()	0	31	7
A	name()	0	2	1
A	type()	0	2	1

<?php
/**
 * The post object which represents both the GitHub and WordPress post
 * @package WP_Writing_On_GitHub
 */

/**
 * Class Writing_On_GitHub_Post
 */
class Writing_On_GitHub_Post {

    /**
     * Api object
     *
     * @var Writing_On_GitHub_Api
     */
    public $api;

    /**
     * Post ID
     * @var integer
     */
    public $id = 0;

    /**
     * Blob object
     * @var Writing_On_GitHub_Blob
     */
    public $blob;

    /**
     * Post object
     * @var WP_Post
     */
    public $post;

    /**
     * Post args.
     *
     * @var array
     */
    protected $args;

    /**
     * Post meta.
     *
     * @var array
     */
    protected $meta;

    /**
     * Whether the post has been saved.
     *
     * @var bool
     */
    protected $new = true;


    protected $old_github_path;

    /**
     * Instantiates a new Post object
     *
     * @param int|array                 $id_or_args Either a post ID or an array of arguments.
     * @param Writing_On_GitHub_Api $api API object.
     *
     * @todo remove database operations from this method
     */
    public function __construct( $id_or_args, Writing_On_GitHub_Api $api ) {
        $this->api = $api;

        if ( is_numeric( $id_or_args ) ) {
            $this->id   = (int) $id_or_args;
            $this->post = get_post( $this->id );

It seems like get_post($this->id) can also be of type array. However, the property $post is declared as type WP_Post. Maybe add an additional type check?

            $this->new  = false;
        }

        if ( is_array( $id_or_args ) ) {
            $this->args = $id_or_args;

            if ( isset( $this->args['ID'] ) ) {
                $this->post = get_post( $this->args['ID'] );

                if ( $this->post ) {
                    $this->id  = $this->post->ID;
                    $this->new = false;
                } else {
                    unset( $this->args['ID'] );
                }
            }
        }
    }

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

    /**
     * Returns the post type
     */
    public function type() {
        return $this->post->post_type;
    }

    /**
     * Returns the post type
     */
    public function status() {
        return $this->post->post_status;
    }

    /**
     * Returns the post name
     */
    public function name() {
        return $this->post->post_name;
    }

    /**
     * Returns true if the post has a password
     * @return bool
     */
    public function has_password() {
        return ! empty( $this->post->post_password );
    }

    /**
     * Combines the 2 content parts for GitHub
     */
    public function github_content() {
        $use_blob = wogh_is_dont_export_content() && $this->blob;
        $content = $use_blob ?
            $this->blob->post_content() :
            $this->post_content();

        return $this->front_matter() . $content;
        // $content = $this->front_matter() . $content;
        // $ending  = apply_filters( 'wogh_line_endings', "\n" );

        // return preg_replace( '~(*BSR_ANYCRLF)\R~', $ending, $content );
    }

    /**
     * The post's YAML frontmatter
     *
     * Returns String the YAML frontmatter, ready to be written to the file
     */
    public function front_matter() {
        return "---\n" . spyc_dump( $this->meta() ) . "---\n";
    }

    /**
     * Returns the post_content
     *
     * Markdownify's the content if applicable
     */
    public function post_content() {
        $content = $this->post->post_content;

        if ( function_exists( 'wpmarkdown_html_to_markdown' ) ) {
            $content = wpmarkdown_html_to_markdown( $content );
        } else if ( class_exists( 'WPCom_Markdown' ) ) {
            if ( WPCom_Markdown::get_instance()->is_markdown( $this->post->ID ) ) {

The type WPCom_Markdown was not found. Maybe you did not declare it correctly or list all dependencies?

                $content = $this->post->post_content_filtered;
            }
        }

        return apply_filters( 'wogh_content_export', $content, $this );
    }

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

    public function set_old_github_path( $path ) {
        $this->old_github_path = $path;
        update_post_meta( $this->id, '_wogh_github_path', $path );
    }


    /**
     * Retrieves or calculates the proper GitHub path for a given post
     *
     * Returns (string) the path relative to repo root
     */
    public function github_path() {
        $path = $this->github_directory() . $this->github_filename();

        return $path;
    }

    /**
     * Get GitHub directory based on post
     *
     * @return string
     */
    public function github_directory() {
        if ( 'publish' !== $this->status() ) {
            return apply_filters( 'wogh_directory_unpublished', '_drafts/', $this );
        }

        $name = '';

        switch ( $this->type() ) {
            case 'post':
                $name = 'posts';
                break;
            case 'page':
                $name = 'pages';
                break;
            default:
                $obj = get_post_type_object( $this->type() );

                if ( $obj ) {
                    $name = strtolower( $obj->labels->name );
                }

                if ( ! $name ) {
                    $name = '';
                }
        }

        if ( $name ) {
            $name = '_' . $name . '/';
        }

        return apply_filters( 'wogh_directory_published', $name, $this );
    }

    /**
     * Build GitHub filename based on post
     */
    public function github_filename() {
        if ( 'post' === $this->type() ) {
            $filename = get_the_time( 'Y-m-d-', $this->id ) . $this->get_name() . '.md';

Are you sure get_the_time('Y-m-d-', $this->id) of type false|string can be used in concatenation?

        } else {
            $filename = $this->get_name() . '.md';
        }

        return apply_filters( 'wogh_filename', $filename, $this );
    }

    /**
     * Returns a post slug we can use in the GitHub filename
     *
     * @return string
     */
    protected function get_name() {
        if ( '' !== $this->name() ) {
            return $this->name();
        }

        return sanitize_title( get_the_title( $this->post ) );
    }

    /**
     * is put on github
     * @return boolean
     */
    public function is_on_github() {
        $sha = get_post_meta( $this->id, '_wogh_sha', true );
        $github_path = get_post_meta( $this->id, '_wogh_github_path', true );
        if ( $sha && $github_path ) {
            return true;
        }
        return false;
    }

    /**
     * Returns the URL for the post on GitHub.
     *
     * @return string
     */
    public function github_view_url() {
        $github_path = get_post_meta( $this->id, '_wogh_github_path', true );
        $repository = $this->api->fetch()->repository();
        $branch = $this->api->fetch()->branch();

        return "https://github.com/$repository/blob/$branch/$github_path";
    }

    /**
     * Returns the URL for the post on GitHub.
     *
     * @return string
     */
    public function github_edit_url() {
        $github_path = get_post_meta( $this->id, '_wogh_github_path', true );
        $repository = $this->api->fetch()->repository();
        $branch = $this->api->fetch()->branch();

        return "https://github.com/$repository/edit/$branch/$github_path";
    }

    /**
     * Retrieve post type directory from blob path.
     *
     * @param string $path Path string.
     *
     * @return string
     */
    public function get_directory_from_path( $path ) {
        $directory = explode( '/', $path );
        $directory = count( $directory ) > 0 ? $directory[0] : '';

        return $directory;
    }

    /**
     * Determines the last author to modify the post
     *
     * Returns Array an array containing the author name and email
     */
    public function last_modified_author() {
        if ( $last_id = get_post_meta( $this->id, '_edit_last', true ) ) {
            $user = get_userdata( $last_id );

It seems like $last_id can also be of type string; however, parameter $user_id of get_userdata() does only seem to accept integer, maybe add an additional type check?

            if ( $user ) {

The condition $user can never be true.

                return array( 'name' => $user->display_name, 'email' => $user->user_email );
            }
        }

        return array();
    }

    /**
     * The post's sha
     * Cached as post meta, or will make a live call if need be
     *
     * Returns String the sha1 hash
     */
    public function sha() {
        $sha = get_post_meta( $this->id, '_wogh_sha', true );

        // If we've done a full export and we have no sha
        // then we should try a live check to see if it exists.
        // if ( ! $sha && 'yes' === get_option( '_wogh_fully_exported' ) ) {

        //  // @todo could we eliminate this by calling down the full tree and searching it
        //  $data = $this->api->fetch()->remote_contents( $this );

        //  if ( ! is_wp_error( $data ) ) {
        //      update_post_meta( $this->id, '_wogh_sha', $data->sha );
        //      $sha = $data->sha;
        //  }
        // }

        // if the sha still doesn't exist, then it's empty
        if ( ! $sha || is_wp_error( $sha ) ) {
            $sha = '';
        }

        return $sha;
    }

    /**
     * Save the sha to post
     *
     * @param string $sha
     */
    public function set_sha( $sha ) {
        update_post_meta( $this->id, '_wogh_sha', $sha );
    }

    /**
     * The post's metadata
     *
     * Returns Array the post's metadata
     */
    public function meta() {
        $meta = array(
            'ID'           => $this->id,
            'post_title'   => get_the_title( $this->post ),
            'post_name'    => $this->post->post_name,
            'author'       => ( $author = get_userdata( $this->post->post_author ) ) ? $author->display_name : '',

$this->post->post_author of type string is incompatible with the type integer expected by parameter $user_id of get_userdata().

            'post_date'    => $this->post->post_date,
            'post_excerpt' => $this->post->post_excerpt,
            'layout'       => get_post_type( $this->post ),
            'link'         => get_permalink( $this->post ),
            'published'    => 'publish' === $this->status() ? true : false,
            'tags'         => wp_get_post_tags( $this->id, array( 'fields' => 'names' ) ),
            'categories'   => wp_get_post_categories( $this->id, array( 'fields' => 'names' ) )
        );
        if ( empty($this->post->post_name) ) {

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

            unset($meta['post_name']);

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

        }
        if ( empty($this->post->post_excerpt) ) {

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

            unset($meta['post_excerpt']);

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

        }
        if ( 'yes' == get_option('wogh_ignore_author') ) {

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

            unset($meta['author']);

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

        }

        //convert traditional post_meta values, hide hidden values, skip already populated values
        // foreach ( get_post_custom( $this->id ) as $key => $value ) {

        //  if ( '_' === substr( $key, 0, 1 ) || isset( $meta[ $key ] ) ) {
        //      continue;
        //  }

        //  $meta[ $key ] = $value;

        // }

        return apply_filters( 'wogh_post_meta', $meta, $this );
    }

    /**
     * Returns whether the Post has been saved in the DB yet.
     *
     * @return bool
     */
    public function is_new() {
        return $this->new;
    }

    /**
     * Sets the Post's meta.
     *
     * @param array $meta
     */
    public function set_meta( $meta ) {
        $this->meta = $meta;
    }

    /**
     * Returns the Post's arguments.
     *
     * @return array
     */
    public function get_args() {
        return $this->args;
    }

    /**
     * Returns the Post's meta.
     *
     * @return array
     */
    public function get_meta() {
        return $this->meta;
    }

    /**
     * Get the blob
     * @return Writing_On_GitHub_Blob
     */
    public function get_blob() {
        return $this->blob;
    }

    /**
     * Set the blob
     * @param Writing_On_GitHub_Blob $blob
     */
    public function set_blob( Writing_On_GitHub_Blob $blob ) {
        $this->blob = $blob;
    }

    /**
     * Sets the Post's WP_Post object.
     *
     * @param WP_Post $post
     *
     * @return $this
     */
    public function set_post( WP_Post $post ) {
        $this->post = $post;
        $this->id   = $post->ID;

        return $this;
    }

    /**
     * Transforms the Post into a Blob.
     *
     * @return Writing_On_GitHub_Blob
     */
    public function to_blob() {
        $data = new stdClass;

        $data->path    = $this->github_path();
        $data->content = $this->github_content();
        $data->sha     = $this->sha();

        return new Writing_On_GitHub_Blob( $data );
    }
}