litefeel / writing-on-github

lib/client/fetch.php

<?php
/**
 * Fetch API client class.
 * @package Writing_On_GitHub
 */

/**
 * Class Writing_On_GitHub_Fetch_Client
 */
class Writing_On_GitHub_Fetch_Client extends Writing_On_GitHub_Base_Client {

    /**
     * Compare a commit by sha with master from the GitHub API
     *
     * @param string $sha Sha for commit to retrieve.
     *
     * @return Writing_On_GitHub_File_Info[]|WP_Error
     */
    public function compare( $sha ) {
        // https://api.github.com/repos/litefeel/testwpsync/compare/861f87e8851b8debb78db548269d29f8da4d94ac...master
        $endpoint = $this->compare_endpoint();
        $branch = $this->branch();
        $data = $this->call( 'GET', "$endpoint/$sha...$branch" );

        if ( is_wp_error( $data ) ) {
            return $data;

The expression return $data also could return the type stdClass which is incompatible with the documented return type Writing_On_GitHub_File_Info[]&WP_Error.

        }

        $files = array();
        foreach ($data->files as $file) {

The property files does not seem to exist on WP_Error.

            $file->path = $file->filename;
            $files[] = new Writing_On_GitHub_File_Info($file);
        }

        return $files;
    }

    /**
     * Calls the content API to get the post's contents and metadata
     *
     * Returns Object the response from the API
     *
     * @param Writing_On_GitHub_Post $post Post to retrieve remote contents for.
     *
     * @return mixed
     */
    public function remote_contents( $post ) {
        return $this->call( 'GET', $this->content_endpoint( $post->github_path() ) );
    }



    public function exists( $path ) {
        $result = $this->call( 'GET', $this->content_endpoint( $path ) );
        if ( is_wp_error( $result ) ) {
            return false;
        }
        return true;
    }

    /**
     * Retrieves a tree by sha recursively from the GitHub API
     *
     * @param string $sha Commit sha to retrieve tree from.
     *
     * @return Writing_On_GitHub_File_Info[]|WP_Error
     */
    public function tree_recursive( $sha = '_default' ) {

        if ( '_default' === $sha ) {
            $sha = $this->branch();
        }

        $data = $this->call( 'GET', $this->tree_endpoint() . '/' . $sha . '?recursive=1' );

        if ( is_wp_error( $data ) ) {
            return $data;

The expression return $data also could return the type stdClass which is incompatible with the documented return type Writing_On_GitHub_File_Info[]&WP_Error.

        }

        $files = array();

        foreach ( $data->tree as $index => $thing ) {

The property tree does not seem to exist on WP_Error.

            // We need to remove the trees because
            // the recursive tree includes both
            // the subtrees as well the subtrees' blobs.
            if ( 'blob' === $thing->type ) {
                $thing->status = '';
                $files[] = new Writing_On_GitHub_File_Info( $thing );
            }
        }

        return $files;
    }

    /**
     * Retrieves the blob data for a given sha
     *
     * @param Writing_On_GitHub_File_Info $fileinfo
     *
     * @return Writing_On_GitHub_Blob|WP_Error
     */
    public function blob( Writing_On_GitHub_File_Info $fileinfo ) {
        $data = $this->call( 'GET', $this->blob_endpoint() . '/' . $fileinfo->sha );

        if ( is_wp_error( $data ) ) {
            return $data;

The expression return $data also could return the type stdClass which is incompatible with the documented return type WP_Error|Writing_On_GitHub_Blob.

        }

        $data->path = $fileinfo->path;

The property path does not seem to exist on WP_Error.

        return new Writing_On_GitHub_Blob( $data );

It seems like $data can also be of type WP_Error; however, parameter $data of Writing_On_GitHub_Blob::__construct() does only seem to accept stdClass, maybe add an additional type check?

    }

    /**
     * Get blob by path
     * @param  string $path
     * @return Writing_On_GitHub_Blob|WP_Error
     */
    public function blob_by_path( $path ) {
        $result = $this->call( 'GET', $this->content_endpoint( $path ) );
        if ( is_wp_error( $result ) ) {
            return $result;

The expression return $result also could return the type stdClass which is incompatible with the documented return type WP_Error|Writing_On_GitHub_Blob.

        }

        return new Writing_On_GitHub_Blob( $result );

It seems like $result can also be of type WP_Error; however, parameter $data of Writing_On_GitHub_Blob::__construct() does only seem to accept stdClass, maybe add an additional type check?

    }
}