sunnysideup / silverstripe-modulechecks

code/api/GitRepoFinder.php

<?php



class GitRepoFinder extends Object
{

    /**
     *
     *
     *
     * @var string
     */
    private static $_modules = array();



    /**
     * takes the preloaded modules and
     * adds any other ones you have listed on github
     */

    public static function get_all_repos($username = '', $getNamesWithPrefix = false)
    {
        # $oauth_token = GitRepoFinder::Config()->get('github_oauth_token');

        self::get_all_repos_no_oauth($username, $getNamesWithPrefix);
        if (! self::$_modules) {
            self::get_repos_with_auth($username, $getNamesWithPrefix);
        }
        return self::$_modules;
    }

    public static function get_all_repos_no_oauth($username = '', $getNamesWithPrefix = false)
    {
        $preSelected = Config::inst()->get('UpdateModules', 'modules_to_update');
        if (is_array($preSelected) && count($preSelected)) {
            return $preSelected;
        } else {
            if (!$username) {
                $username = Config::inst()->get('GitHubModule', "github_user_name");
            }
            print "<li>Retrieving List of modules from GitHub for user $username ... </li>";
            if (! count(self::$_modules)) {

self::_modules of type string is incompatible with the type Countable|array expected by parameter $var of count().

                for ($page = 0; $page < 10; $page++) {
                    $ch = curl_init();
                    curl_setopt($ch, CURLOPT_URL, "https://api.github.com/users/".$username."/repos?per_page=100&page=$page");

It seems like $ch can also be of type false; however, parameter $ch of curl_setopt() does only seem to accept resource, maybe add an additional type check?

                    curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
                    curl_setopt($ch, CURLOPT_CONNECTTIMEOUT, true);
                    curl_setopt($ch, CURLOPT_USERAGENT, 'Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.13) Gecko/20080311 Firefox/2.0.0.13');


                    $string = curl_exec($ch);

It seems like $ch can also be of type false; however, parameter $ch of curl_exec() does only seem to accept resource, maybe add an additional type check?

                    // close curl resource to free up system resources
                    curl_close($ch);

It seems like $ch can also be of type false; however, parameter $ch of curl_close() does only seem to accept resource, maybe add an additional type check?

                    $array = json_decode($string, true);
                    $count = count($array);
                    if ($count > 0) {
                        foreach ($array as $repo) {
                            //dont bother about forks
                            if (isset($repo["fork"]) && !$repo["fork"]) {
                                //make sure we are the owners
                                if ($repo["owner"]["login"] == $username) {
                                    $isSSModule =  (stripos($repo["name"], 'silverstripe-')  !== false);
                                    //check it is silverstripe module
                                    if (!$getNamesWithPrefix) {
                                        $name = $repo["name"];
                                    } else {
                                        $name = preg_replace('/silverstripe/', "", $repo["name"], $limit = 1);
                                    }

                                    //if(strlen($name) < strlen($repo["name"])) {
                                    if ($isSSModule) {
                                        //is it listed yet?
                                        if (!in_array($name, self::$_modules)) {

self::_modules of type string is incompatible with the type array expected by parameter $haystack of in_array().

                                            self::$_modules[] = $name;
                                        }
                                    } else {
                                        DB::alteration_message("skipping ".$repo["name"]." as it does not appear to me a silverstripe module, you can add it manually to this task, using the configs ... ");
                                    }
                                } else {
                                    DB::alteration_message("skipping ".$repo["name"]." as it has a different owner");
                                }
                            } elseif (isset($repo["name"])) {
                                DB::alteration_message("skipping ".$repo["name"]." as it is a fork");
                            }
                        }
                    } else {
                        $page = 11;
                    }
                }
            }
            print "<li>Found ".count(self::$_modules)." modules on GitHub ... </li>";
            if (count(self::$_modules)==0) {
                user_error("No modules found on GitHub. This is possibly because the limit of 60 requests an hour has been exceeded.");
            }
        }
        return self::$_modules;
    }

    public static function get_repos_with_auth($username = '', $getNamesWithPrefix = false)
    {
        $preSelected = Config::inst()->get('UpdateModules', 'modules_to_update');
        if (is_array($preSelected) && count($preSelected)) {
            self::$_modules = $preSelected;

It seems like $preSelected of type array is incompatible with the declared type string of property $_modules.

        } else {
            if ($username) {
                $gitUserName = $username;
            } else {
                $gitUserName = Config::inst()->get('GitHubModule', "github_user_name");
            }
            print "<li>Retrieving List of modules from GitHub for user $username ... </li>";
            if (! count(self::$_modules)) {

self::_modules of type string is incompatible with the type Countable|array expected by parameter $var of count().

                $url = 'https://api.github.com/users/' . trim($gitUserName) . '/repos';
                $array  = array();
                for ($page = 0; $page < 10; $page++) {
                    $data = array(
                        'per_page' => 100,
                        'page'=>$page
                    );

                    $method = 'GET';
                    $ch = curl_init($url);
                    $header = "Content-Type: application/json";

                    if ($method == 'GET') {
                        $url .= '?'.http_build_query($data);
                    }

                    $gitApiUserName = trim(GitHubModule::Config()->get('git_api_login_username'));

It seems like GitHubModule::Config()->get('git_api_login_username') can also be of type array; however, parameter $str of trim() does only seem to accept string, maybe add an additional type check?

                    $gitUserName = trim(GitHubModule::Config()->get('github_user_name'));
                    $gitApiUserPassword = trim(GitHubModule::Config()->get('git_api_login_password'));

                    $gitApiAccessToken = trim(GitHubModule::Config()->get('git_personal_access_token'));
                    if (trim($gitApiAccessToken)) {
                        $gitApiUserPassword = $gitApiAccessToken;
                    }


                    curl_setopt($ch, CURLOPT_VERBOSE, 1);

It seems like $ch can also be of type false; however, parameter $ch of curl_setopt() does only seem to accept resource, maybe add an additional type check?

                    curl_setopt($ch, CURLOPT_URL, $url);
                    curl_setopt($ch, CURLOPT_CUSTOMREQUEST, $method);
                    curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
                    curl_setopt($ch, CURLOPT_HTTPHEADER, array($header));
                    curl_setopt($ch, CURLOPT_FOLLOWLOCATION, true);
                    curl_setopt(
                        $ch,
                        CURLOPT_USERAGENT,
                        'sunnysideupdevs'
                    );


                    if (isset($gitApiUserName) && isset($gitApiUserPassword)) {
                        curl_setopt($ch, CURLOPT_USERPWD, $gitApiUserName . ':' . $gitApiUserPassword);
                        curl_setopt($ch, CURLOPT_HTTPAUTH, CURLAUTH_BASIC);
                    }


                    $curlResult = curl_exec($ch);

It seems like $ch can also be of type false; however, parameter $ch of curl_exec() does only seem to accept resource, maybe add an additional type check?

                    if (! $curlResult) {
                        GeneralMethods::output_to_screen('Could not retrieve list of modules from GitHub');

                        UpdateModules::$unsolvedItems["all"] =  ('Could not retrieve list of modules from GitHub');
                        die('');

Using exit here is not recommended.

                    }

                    $array = array_merge($array, json_decode($curlResult));

It seems like $curlResult can also be of type true; however, parameter $json of json_decode() does only seem to accept string, maybe add an additional type check?

                }


                $modules = array();

                if (count($array) > 0) {
                    foreach ($array as $repo) {

                        // see http://stackoverflow.com/questions/4345554/convert-php-object-to-associative-array
                        $repo = json_decode(json_encode($repo), true);


                        //dont bother about forks
                        if (isset($repo["fork"]) && !$repo["fork"]) {
                            //make sure we are the owners

                            if ($repo["owner"]["login"] == $gitUserName) {
                                $isSSModule =  (stripos($repo["name"], 'silverstripe-')  !== false);
                                //check it is silverstripe module

                                if (!$getNamesWithPrefix) {
                                    $name = $repo["name"];
                                } else {
                                    $name = preg_replace('/silverstripe/', "", $repo["name"], $limit = 1);
                                }

                                //if(strlen($name) < strlen($repo["name"])) {
                                if ($isSSModule) {
                                    //is it listed yet?
                                    if (!in_array($name, $modules)) {
                                        array_push($modules, $name);
                                    }
                                } else {
                                    GeneralMethods::output_to_screen("skipping ".$repo["name"]." as it does not appear to me a silverstripe module");
                                }
                            } else {
                                GeneralMethods::output_to_screen("skipping ".$repo["name"]." as it has a different owner");
                            }
                        } elseif (isset($repo["name"])) {
                            DB::alteration_message("skipping ".$repo["name"]." as it is a fork");
                        }
                    }
                }
                self::$_modules = $modules;
            }
        }
        return self::$_modules;
    }
}