Writing_On_GitHub_Persist_Client

Complexity

Total Complexity

8

Size/Duplication

Total Lines	74
Duplicated Lines	64.86 %

Coupling/Cohesion

Dependencies

1

Test Coverage

Coverage

35.29%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	export_user()	0	12	2
A	update_file()	0	10	2
A	create_file()	0	11	2
A	delete_file()	0	11	2

<?php
/**
 * API Persist client.
 * @package Writing_On_GitHub
 */

/**
 * Class Writing_On_GitHub_Persist_Client
 */
class Writing_On_GitHub_Persist_Client extends Writing_On_GitHub_Base_Client {

    /**
     * Get the data for the current user.
     *
     * @return array
     */
    protected function export_user() {
        $user_id = get_current_user_id();
        $user = get_userdata( $user_id );

        if ( $user ) {

The condition $user can never be true.

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

        return false;

The expression return false returns the type false which is incompatible with the documented return type array.

    }

    /**
     * Delete the file.
     *
     * @return array
     */
    public function delete_file( $path, $sha, $message ) {
        $body = new stdClass();
        $body->message = $message;
        $body->sha = $sha;
        $body->branch = $this->branch();

        if ( $author = $this->export_user() ) {
            $body->author = $author;
        }

        return $this->call( 'DELETE', $this->content_endpoint( $path ), $body );

$body of type stdClass is incompatible with the type array expected by parameter $body of Writing_On_GitHub_Base_Client::call().

The expression return $this->call('DELETE', $this->content_endpoint($path), $body) returns the type WP_Error which is incompatible with the documented return type array.

    }

    /**
     * Create the file.
     *
     * @return array
     */
    public function create_file( $blob, $message ) {
        $body = $blob->to_body();
        $body->message = $message;
        $body->branch = $this->branch();
        unset($body->sha);

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

        if ( $author = $this->export_user() ) {
            $body->author = $author;
        }

        return $this->call( 'PUT', $this->content_endpoint( $blob->path() ), $body );

The expression return $this->call('PUT', $this->content_endpoint($blob->path()), $body) returns the type WP_Error which is incompatible with the documented return type array.

    }

    /**
     * Update the file.
     *
     * @return array
     */
    public function update_file( $blob, $message ) {
        $body = $blob->to_body();
        $body->message = $message;
        $body->branch = $this->branch();

        if ( $author = $this->export_user() ) {
            $body->author = $author;
        }

        return $this->call( 'PUT', $this->content_endpoint( $blob->path() ), $body );

The expression return $this->call('PUT', $this->content_endpoint($blob->path()), $body) returns the type WP_Error which is incompatible with the documented return type array.

    }
}