HubphAPI::__construct() - Code Metrics - Inspection of "Factor "matchingPRs" method out of "existingPRs" m..." - g1a/hubph - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 6dc2cd...aa73aa )

by Greg

created 2020-05-13 17:31 UTC

HubphAPI::__construct() A

↳ Parent: HubphAPI

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	1

<?php

namespace Hubph;

use Consolidation\Config\ConfigInterface;
use Hubph\Internal\EventLogger;

class HubphAPI
{
    protected $config;
    protected $token;
    protected $gitHubAPI;
    protected $eventLogger;
    protected $as = 'default';

    /**
     * HubphAPI constructor
     */
    public function __construct(ConfigInterface $config)
    {
        $this->config = $config;
    }

    public function startLogging($filename)
    {
        $this->stopLogging();
        $this->eventLogger = new EventLogger($filename);
        $this->eventLogger->start();
    }

    public function stopLogging()
    {
        if ($this->eventLogger) {
            $this->eventLogger->stop();
        }
        $this->eventLogger = null;
    }

    public function setAs($as)
    {
        if ($as != $this->as) {
            $this->as = $as;
            $this->token = false;
            $this->gitHubAPI = false;
        }
    }

    public function whoami()
    {
        $gitHubAPI = $this->gitHubAPI();
        $authenticated = $gitHubAPI->api('current_user')->show();
        return $authenticated;
    }

    public function prCreate($org, $project, $title, $body, $base, $head)
    {
        $params = [
            'title' => $title,
            'body' => $body,
            'base' => $base,
            'head' => $head,
        ];
        $response = $this->gitHubAPI()->api('pull_request')->create($org, $project, $params);
        $this->logEvent(__FUNCTION__, [$org, $project], $params, $response);
        return $this;
    }

    public function prClose($org, $project, PullRequests $prs)
    {
        foreach ($prs->prNumbers() as $n) {
            $gitHubAPI = $this->gitHubAPI();
            $gitHubAPI->api('pull_request')->update($org, $project, $n, ['state' => 'closed']);
        }
    }

    public function prMerge($org, $project, PullRequests $prs, $message, $mergeMethod = 'squash', $title = null)
    {
        // First, check to see if all of the pull requests can be merged,
        // and collect the sha hash of the head of the branch.
        $allClean = true;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        $shas = [];
        foreach ($prs->prNumbers() as $n) {
            $pullRequest = $this->gitHubAPI()->api('pull_request')->show($org, $project, $n);
            $is_clean = $pullRequest['mergeable'] && $pullRequest['mergeable_state'] == 'clean';
            if (!$is_clean) {
                return false;
            }
            $shas[$n] = $pullRequest['head']['sha'];
        }

        // Merge all of the pull requests
        foreach ($shas as $n => $sha) {
            $response = $this->gitHubAPI()->api('pull_request')->merge($org, $project, $n, $message, $sha, $mergeMethod, $title);
            $this->logEvent(__FUNCTION__, [$org, $project], [$n, $message, $sha, $mergeMethod, $title], $response);
        }
        return true;
    }

    /**
     * prCheck determines whether there are any open PRs that already exist
     * that satisfy any of the provided $vids.
     *
     * @param string $projectWithOrg org/project to check
     * @param VersionIdentifiers $vids
     * @return [int $status, PullRequests $prs] status of PRs, and a list of PR numbers

     *   - If $status is 0, then the caller should go ahead and create a new PR.
     *     The existing pull requests that would be superceded by the new PR are
     *     returned in the second parameter. These PRs could all be closed.
     *   - If $status is >0, then there is no need to create a new PR, as there
     *     are already existing PRs that are equivalent to the one that would
     *     be open. The equivalent PRs are returned in the second parameter.
     */
    public function prCheck($projectWithOrg, VersionIdentifiers $vids)
    {
        // Find all of the PRs that contain any vid
        $existingPRs = $this->existingPRs($projectWithOrg, $vids);

        // Check to see if there are PRs matching all of the vids/vvals.
        $titles = $existingPRs->titles();
        $status = $vids->allExist($titles);

        return [$status, $existingPRs];
    }

    public function prStatuses($projectWithOrg, $number)
    {
        list($org, $project) = explode('/', $projectWithOrg, 2);
        $pullRequestStatus = $this->gitHubAPI()->api('pull_request')->status($org, $project, $number);

        // Filter out the results based on 'target_url'
        $filteredResults = [];
        foreach (array_reverse($pullRequestStatus) as $id => $item) {
            $filteredResults[$item['target_url']] = $item;
        }
        $pullRequestStatus = [];
        foreach ($filteredResults as $target_url => $item) {
            $pullRequestStatus[$item['id']] = $item;
        }

        // Put the most recently updated statuses at the top of the list
        uasort(

            $pullRequestStatus,
            function ($lhs, $rhs) {
                return abs(strtotime($lhs['updated_at']) - strtotime($rhs['updated_at']));
            }
        );

        return $pullRequestStatus;
    }

    public function addTokenAuthentication($url)
    {
        $token = $this->gitHubToken();
        if (!$token) {
            return $url;
        }
        if (!preg_match('#github\.com[/:]#', $url)) {
            return $url;
        }
        $projectAndOrg = $this->projectAndOrgFromUrl($url);
        return "https://{$token}:[email protected]/{$projectAndOrg}.git";
    }

    protected function projectAndOrgFromUrl($remote)

    {
        $remote = preg_replace('#^git@[^:]*:#', '', $remote);
        $remote = preg_replace('#^[^:]*://[^/]*/#', '', $remote);
        $remote = preg_replace('#\.git$#', '', $remote);

        return $remote;
    }

    protected function existingPRs($projectWithOrg, VersionIdentifiers $vids)
    {
        return $this->matchingPRs($projectWithOrg, $vids->getPreamble(), $vids->pattern());
    }

    public function matchingPRs($projectWithOrg, $preamble, $pattern)
    {
        $q = "repo:$projectWithOrg in:title is:pr state:open $preamble";
        $result = new PullRequests();
        $gitHubAPI = $this->gitHubAPI();
        $searchResults = $gitHubAPI->api('search')->issues($q);
        $result->addSearchResults($searchResults, $pattern);

        return $result;
    }

    public function allPRs($projectWithOrg)
    {
        $q = "repo:$projectWithOrg in:title is:pr state:open";
        $result = new PullRequests();
        $searchResults = $this->gitHubAPI()->api('search')->issues($q);
        $result->addSearchResults($searchResults);

        return $result;
    }

    /**
     * Pass an event of note to the event logger
     * @param string $event_name
     * @param array $args
     * @param array $params
     * @param array $response
     */
    protected function logEvent($event_name, $args, $params, $response)
    {
        if ($this->eventLogger) {
            $this->eventLogger->log($event_name, $args, $params, $response);
        }
    }

    /**
     * Authenticate and then return the gitHub API object.
     */
    public function gitHubAPI()
    {
        if (!$this->gitHubAPI) {
            $token = $this->gitHubToken();

            $this->gitHubAPI = new \Github\Client();
            $this->gitHubAPI->authenticate($token, null, \Github\Client::AUTH_HTTP_TOKEN);
        }
        return $this->gitHubAPI;
    }

    /**
     * Look up the GitHub token set either via environment variable or in the
     * auth-token cache directory.
     */
    public function gitHubToken()
    {
        if (!$this->token) {
            $this->token = $this->getGitHubToken();
        }
        return $this->token;
    }

    protected function getGitHubToken()
    {
        $as = $this->as;
        $token = null;
        if ($as == 'default') {
            $as = $this->getConfig()->get("github.default-user");
        }

        // First preference: There is a 'path' component in preferences
        // pointing to a file containing the token.
        $github_token_cache = $this->getConfig()->get("github.personal-auth-token.$as.path");
        if (file_exists($github_token_cache)) {
            $token = trim(file_get_contents($github_token_cache));
        }

        // Second preference: There is an environment variable that begins
        // with an uppercased version of the 'as' string followed by '_TOKEN'
        if (!$token) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $env_name = strtoupper(str_replace('-', '_', $as)) . '_TOKEN';
            $token = getenv($env_name);
        }

        // If we read in a token from one of the preferred locations, then
        // set the GITHUB_TOKEN environment variable and return it.
        if ($token) {
            putenv("GITHUB_TOKEN=$token");
            return $token;
        }

        // Fallback: authenticate to whatever 'GITHUB_TOKEN' is already set to.
        return getenv('GITHUB_TOKEN');
    }

    protected function getConfig()
    {
        return $this->config;
    }
}


1		<?php
2
3		namespace Hubph;
4
5		use Consolidation\Config\ConfigInterface;
6		use Hubph\Internal\EventLogger;
7
8		class HubphAPI
9		{
10		protected $config;
11		protected $token;
12		protected $gitHubAPI;
13		protected $eventLogger;
14		protected $as = 'default';
15
16		/**
17		* HubphAPI constructor
18		*/
19		public function __construct(ConfigInterface $config)
20		{
21		$this->config = $config;
22		}
23
24		public function startLogging($filename)
25		{
26		$this->stopLogging();
27		$this->eventLogger = new EventLogger($filename);
28		$this->eventLogger->start();
29		}
30
31		public function stopLogging()
32		{
33		if ($this->eventLogger) {
34		$this->eventLogger->stop();
35		}
36		$this->eventLogger = null;
37		}
38
39		public function setAs($as)
40		{
41		if ($as != $this->as) {
42		$this->as = $as;
43		$this->token = false;
44		$this->gitHubAPI = false;
45		}
46		}
47
48		public function whoami()
49		{
50		$gitHubAPI = $this->gitHubAPI();
51		$authenticated = $gitHubAPI->api('current_user')->show();
52		return $authenticated;
53		}
54
55		public function prCreate($org, $project, $title, $body, $base, $head)
56		{
57		$params = [
58		'title' => $title,
59		'body' => $body,
60		'base' => $base,
61		'head' => $head,
62		];
63		$response = $this->gitHubAPI()->api('pull_request')->create($org, $project, $params);
64		$this->logEvent(__FUNCTION__, [$org, $project], $params, $response);
65		return $this;
66		}
67
68		public function prClose($org, $project, PullRequests $prs)
69		{
70		foreach ($prs->prNumbers() as $n) {
71		$gitHubAPI = $this->gitHubAPI();
72		$gitHubAPI->api('pull_request')->update($org, $project, $n, ['state' => 'closed']);
73		}
74		}
75
76		public function prMerge($org, $project, PullRequests $prs, $message, $mergeMethod = 'squash', $title = null)
77		{
78		// First, check to see if all of the pull requests can be merged,
79		// and collect the sha hash of the head of the branch.
80		$allClean = true;
		0 ignored issues – show Unused Code introduced 2018-09-13 21:25 UTC by Report Bug Copy Issue Report `$allClean` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
81		$shas = [];
82		foreach ($prs->prNumbers() as $n) {
83		$pullRequest = $this->gitHubAPI()->api('pull_request')->show($org, $project, $n);
84		$is_clean = $pullRequest['mergeable'] && $pullRequest['mergeable_state'] == 'clean';
85		if (!$is_clean) {
86		return false;
87		}
88		$shas[$n] = $pullRequest['head']['sha'];
89		}
90
91		// Merge all of the pull requests
92		foreach ($shas as $n => $sha) {
93		$response = $this->gitHubAPI()->api('pull_request')->merge($org, $project, $n, $message, $sha, $mergeMethod, $title);
94		$this->logEvent(__FUNCTION__, [$org, $project], [$n, $message, $sha, $mergeMethod, $title], $response);
95		}
96		return true;
97		}
98
99		/**
100		* prCheck determines whether there are any open PRs that already exist
101		* that satisfy any of the provided $vids.
102		*
103		* @param string $projectWithOrg org/project to check
104		* @param VersionIdentifiers $vids
105		* @return [int $status, PullRequests $prs] status of PRs, and a list of PR numbers
		0 ignored issues – show Documentation introduced 2018-09-13 21:25 UTC by Report Bug Copy Issue Report The doc-type `int">[int` could not be parsed: Unknown type name "[" at position 0. [(view supported doc-types) This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types. Loading history...
106		* - If $status is 0, then the caller should go ahead and create a new PR.
107		* The existing pull requests that would be superceded by the new PR are
108		* returned in the second parameter. These PRs could all be closed.
109		* - If $status is >0, then there is no need to create a new PR, as there
110		* are already existing PRs that are equivalent to the one that would
111		* be open. The equivalent PRs are returned in the second parameter.
112		*/
113		public function prCheck($projectWithOrg, VersionIdentifiers $vids)
114		{
115		// Find all of the PRs that contain any vid
116		$existingPRs = $this->existingPRs($projectWithOrg, $vids);
117
118		// Check to see if there are PRs matching all of the vids/vvals.
119		$titles = $existingPRs->titles();
120		$status = $vids->allExist($titles);
121
122		return [$status, $existingPRs];
123		}
124
125		public function prStatuses($projectWithOrg, $number)
126		{
127		list($org, $project) = explode('/', $projectWithOrg, 2);
128		$pullRequestStatus = $this->gitHubAPI()->api('pull_request')->status($org, $project, $number);
129
130		// Filter out the results based on 'target_url'
131		$filteredResults = [];
132		foreach (array_reverse($pullRequestStatus) as $id => $item) {
133		$filteredResults[$item['target_url']] = $item;
134		}
135		$pullRequestStatus = [];
136		foreach ($filteredResults as $target_url => $item) {
137		$pullRequestStatus[$item['id']] = $item;
138		}
139
140		// Put the most recently updated statuses at the top of the list
141		uasort(
142
143		$pullRequestStatus,
144		function ($lhs, $rhs) {
145		return abs(strtotime($lhs['updated_at']) - strtotime($rhs['updated_at']));
146		}
147		);
148
149		return $pullRequestStatus;
150		}
151
152		public function addTokenAuthentication($url)
153		{
154		$token = $this->gitHubToken();
155		if (!$token) {
156		return $url;
157		}
158		if (!preg_match('#github\.com[/:]#', $url)) {
159		return $url;
160		}
161		$projectAndOrg = $this->projectAndOrgFromUrl($url);
162		return "https://{$token}:[email protected]/{$projectAndOrg}.git";
163		}
164
165	View Code Duplication	protected function projectAndOrgFromUrl($remote)
		0 ignored issues – show Duplication introduced 2018-07-26 21:11 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...
166		{
167		$remote = preg_replace('#^git@[^:]*:#', '', $remote);
168		$remote = preg_replace('#^[^:]://[^/]/#', '', $remote);
169		$remote = preg_replace('#\.git$#', '', $remote);
170
171		return $remote;
172		}
173
174		protected function existingPRs($projectWithOrg, VersionIdentifiers $vids)
175		{
176		return $this->matchingPRs($projectWithOrg, $vids->getPreamble(), $vids->pattern());
177		}
178
179		public function matchingPRs($projectWithOrg, $preamble, $pattern)
180		{
181		$q = "repo:$projectWithOrg in:title is:pr state:open $preamble";
182		$result = new PullRequests();
183		$gitHubAPI = $this->gitHubAPI();
184		$searchResults = $gitHubAPI->api('search')->issues($q);
185		$result->addSearchResults($searchResults, $pattern);
186
187		return $result;
188		}
189
190		public function allPRs($projectWithOrg)
191		{
192		$q = "repo:$projectWithOrg in:title is:pr state:open";
193		$result = new PullRequests();
194		$searchResults = $this->gitHubAPI()->api('search')->issues($q);
195		$result->addSearchResults($searchResults);
196
197		return $result;
198		}
199
200		/**
201		* Pass an event of note to the event logger
202		* @param string $event_name
203		* @param array $args
204		* @param array $params
205		* @param array $response
206		*/
207		protected function logEvent($event_name, $args, $params, $response)
208		{
209		if ($this->eventLogger) {
210		$this->eventLogger->log($event_name, $args, $params, $response);
211		}
212		}
213
214		/**
215		* Authenticate and then return the gitHub API object.
216		*/
217		public function gitHubAPI()
218		{
219		if (!$this->gitHubAPI) {
220		$token = $this->gitHubToken();
221
222		$this->gitHubAPI = new \Github\Client();
223		$this->gitHubAPI->authenticate($token, null, \Github\Client::AUTH_HTTP_TOKEN);
224		}
225		return $this->gitHubAPI;
226		}
227
228		/**
229		* Look up the GitHub token set either via environment variable or in the
230		* auth-token cache directory.
231		*/
232		public function gitHubToken()
233		{
234		if (!$this->token) {
235		$this->token = $this->getGitHubToken();
236		}
237		return $this->token;
238		}
239
240		protected function getGitHubToken()
241		{
242		$as = $this->as;
243		$token = null;
244		if ($as == 'default') {
245		$as = $this->getConfig()->get("github.default-user");
246		}
247
248		// First preference: There is a 'path' component in preferences
249		// pointing to a file containing the token.
250		$github_token_cache = $this->getConfig()->get("github.personal-auth-token.$as.path");
251		if (file_exists($github_token_cache)) {
252		$token = trim(file_get_contents($github_token_cache));
253		}
254
255		// Second preference: There is an environment variable that begins
256		// with an uppercased version of the 'as' string followed by '_TOKEN'
257		if (!$token) {
		0 ignored issues – show Bug Best Practice introduced 2018-09-12 07:20 UTC by Report Bug Copy Issue Report The expression `$token` of type `string\|null` is loosely compared to `false`; this is ambiguous if the string can be empty. You might want to explicitly use `=== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
258		$env_name = strtoupper(str_replace('-', '_', $as)) . '_TOKEN';
259		$token = getenv($env_name);
260		}
261
262		// If we read in a token from one of the preferred locations, then
263		// set the GITHUB_TOKEN environment variable and return it.
264		if ($token) {
265		putenv("GITHUB_TOKEN=$token");
266		return $token;
267		}
268
269		// Fallback: authenticate to whatever 'GITHUB_TOKEN' is already set to.
270		return getenv('GITHUB_TOKEN');
271		}
272
273		protected function getConfig()
274		{
275		return $this->config;
276		}
277		}
278

g1a / hubph

Push — master ( 6dc2cd...aa73aa )

HubphAPI::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like