Github::tplRepository() - Code Metrics - Inspection of "Merge pull request #120 from renatomarinho/develop" - renatomarinho/laravel-gitscrum - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 0fc1b0...714b2e )

by Renato

created 2016-12-28 21:19 UTC

Github::tplRepository() A

↳ Parent: Github

Complexity

Conditions	2
Paths	2

Size

Total Lines	22
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	22
rs	9.2
cc	2
eloc	19
nc	2
nop	2

<?php

namespace GitScrum\Classes;

use Auth;
use GitScrum\Models\Branch;
use GitScrum\Models\Commit;
use GitScrum\Models\User;
use GitScrum\Models\Issue;
use GitScrum\Models\Organization;
use GitScrum\Models\ProductBacklog;
use Carbon\Carbon;
use GitScrum\Contracts\ProviderInterface;

class Github implements ProviderInterface
{
    public function tplUser($obj)
    {
        return [
            'provider_id' => $obj->id,
            'provider' => 'github',
            'username' => $obj->nickname,
            'name' => $obj->name,
            'token' => $obj->token,
            'avatar' => @$obj->user['avatar_url'],
            'html_url' => @$obj->user['html_url'],
            'bio' => @$obj->user['bio'],
            'since' => Carbon::parse($obj->user['created_at'])->toDateTimeString(),
            'location' => @$obj->user['location'],
            'blog' => @$obj->user['blog'],
            'email' => $obj->email,
        ];
    }

    public function tplRepository($repo, $slug = false)

    {
        return (object) [
            'provider_id' => $repo->id,
            'organization_id' => $this->organization($repo->owner->login),
            'organization_title' => $repo->owner->login,
            'slug' => $slug ? $slug : Helper::slug($repo->name),
            'title' => $repo->name,
            'fullname' => $repo->full_name,
            'is_private' => $repo->private,
            'html_url' => $repo->html_url,
            'description' => $repo->description,
            'fork' => $repo->fork,
            'url' => $repo->url,
            'since' => Carbon::parse($repo->created_at)->toDateTimeString(),
            'pushed_at' => Carbon::parse($repo->pushed_at)->toDateTimeString(),
            'ssh_url' => $repo->ssh_url,
            'clone_url' => $repo->clone_url,
            'homepage' => $repo->homepage,
            'default_branch' => $repo->default_branch,
        ];
    }

    public function tplIssue($obj, $productBracklogId)
    {
        $user = User::where('username', @$obj->user->login)
            ->where('provider', 'github')->first();

        return [
            'provider_id' => $obj->id,
            'user_id' => isset($user->id) ? $user->id : Auth::user()->id,
            'product_backlog_id' => $productBracklogId,
            'effort' => 0,
            'config_issue_effort_id' => 1,
            'issue_type_id' => 1,
            'number' => $obj->number,
            'title' => $obj->title,
            'description' => $obj->body,
            'state' => $obj->state,
            'html_url' => $obj->html_url,
            'created_at' => $obj->created_at,
            'updated_at' => $obj->updated_at,
        ];
    }

    public function readRepositories()
    {
        $repos = collect(Helper::request('https://api.github.com/user/repos'));

        $response = $repos->map(function ($repo) {
            return $this->tplRepository($repo);
        });

        return $response;
    }

    public function createOrUpdateRepository($owner, $obj, $oldTitle = null)

    {
        $params = [
            'name' => str_slug($obj->title, '-'),
            'description' => $obj->description,
        ];

        if (is_null($oldTitle)) {
            $endpoint = 'https://api.github.com/orgs/'.$owner.'/repos';

            if (Auth::user()->username == $owner) {
                $endpoint = 'https://api.github.com/user/repos';
            }

            $response = Helper::request($endpoint, true, 'POST', $params);
        } else {
            $oldTitle = str_slug($oldTitle, '-');
            $response = Helper::request('https://api.github.com/repos/'.$owner.DIRECTORY_SEPARATOR.$oldTitle, true, 'POST', $params);
        }

        return (object) $response;
    }

    public function organization($login)

    {
        $organization = Organization::where('username', $login)
            ->where('provider', 'github')->first();

        if (!isset($organization)) {
            $orgData = Helper::request('https://api.github.com/orgs/'.$login);

            if (!isset($orgData->id)) {
                $orgData = Helper::request('https://api.github.com/users/'.$login);
            }

            $data = [
                'provider_id' => @$orgData->id,
                'username' => @$orgData->login,
                'url' => @$orgData->url,
                'repos_url' => @$orgData->repos_url,
                'events_url' => @$orgData->events_url,
                'hooks_url' => @$orgData->hooks_url,
                'issues_url' => @$orgData->issues_url,
                'members_url' => @$orgData->members_url,
                'public_members_url' => @$orgData->public_members_url,
                'avatar_url' => @$orgData->avatar_url,
                'description' => @$orgData->description,
                'title' => @$orgData->name,
                'blog' => @$orgData->blog,
                'location' => @$orgData->location,
                'email' => @$orgData->email,
                'public_repos' => @$orgData->public_repos,
                'html_url' => @$orgData->html_url,
                'total_private_repos' => @$orgData->total_private_repos,
                'since' => @Carbon::parse($orgData->created_at)->toDateTimeString(),
                'disk_usage' => @$orgData->disk_usage,
            ];

            try {
                $organization = Organization::create($data);
            } catch (\Illuminate\Database\QueryException $e) {

            }

            $organization->users()->sync([Auth::id()]);
        }

        return $organization->id;
    }

    public function readCollaborators($owner, $repo, $providerId = null)
    {
        $collaborators = Helper::request('https://api.github.com/repos/'.$owner.'/'.$repo.'/collaborators');
        foreach ($collaborators as $collaborator) {
            if (isset($collaborator->id)) {
                $data = [
                    'provider_id' => $collaborator->id,
                    'username' => $collaborator->login,
                    'name' => $collaborator->login,
                    'avatar' => $collaborator->avatar_url,
                    'html_url' => $collaborator->html_url,
                    'email' => null,
                    'remember_token' => null,
                    'bio' => null,
                    'location' => null,
                    'blog' => null,
                    'since' => null,
                    'token' => null,
                    'position_held' => null,
                ];

                try {
                    $user = User::create($data);
                } catch (\Exception $e) {
                    $user = User::where('username', $collaborator->login)
                        ->where('provider', 'github')->first();
                }

                $userId[] = $user->id;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
            }
        }

        $organization = Organization::where('username', $owner)
            ->where('provider', 'github')->first()->users();

        if (!$organization->userActive()->count()) {
            $organization->attach($userId);
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
        }
    }

    public function createBranches($owner, $product_backlog_id, $repo, $providerId = null)
    {
        $y = 0;
        for ($i = 1; $i > $y; ++$i) {
            $branches = Helper::request('https://api.github.com/repos/'.$owner.DIRECTORY_SEPARATOR.$repo.'/branches?page='.$i);
            foreach ($branches as $branch) {
                $data = [
                    'product_backlog_id' => $product_backlog_id,
                    'title' => $branch->name,
                    'sha' => $branch->commit->sha,
                ];
                Branch::create($data);
            }
            if (count($branches) < 30) {
                $y = $i + 2;
            }
        }
    }

    public function readIssues()
    {
        $repos = ProductBacklog::all();

        foreach ($repos as $repo) {
            $issues = Helper::request('https://api.github.com/repos/'.$repo->organization->username.
                DIRECTORY_SEPARATOR.$repo->title.'/issues?state=all');

            $issues = is_array($issues) ? $issues : [$issues];

            foreach ($issues as $issue) {
                try {
                    $data = $this->tplIssue($issue, $repo->id);
                } catch (\Exception $e) {

                }

                if (!Issue::where('provider_id', $issue->id)->where('provider', 'github')->first()) {
                    Issue::create($data)->users()->sync([$data['user_id']]);
                }
            }
        }
    }

    public function createOrUpdateIssue($obj)

    {
        $params = [
            'title' => $obj->title,
            'body' => $obj->description,
        ];

        $response = Helper::request('https://api.github.com/repos/'.
            $obj->productBacklog->organization->username.DIRECTORY_SEPARATOR.
            $obj->productBacklog->title.'/issues'.(isset($obj->number) ? DIRECTORY_SEPARATOR.$obj->number : ''),
            true, 'POST', $params);

        return (object) $response;
    }

    public function createOrUpdateIssueComment($obj, $verb = 'POST')

    {
        $params = [
            'body' => $obj->comment,
        ];

        $response = Helper::request('https://api.github.com/repos/'.
            $obj->issue->productBacklog->organization->username.DIRECTORY_SEPARATOR.
            $obj->issue->productBacklog->title.'/issues'.(isset($obj->provider_id) ? '' : DIRECTORY_SEPARATOR.$obj->issue->number).'/comments'.
            (isset($obj->provider_id) ? DIRECTORY_SEPARATOR.$obj->provider_id : ''),
            true, $verb, $params);

        return (object) $response;
    }

    public function deleteIssueComment($obj)

    {
        return $this->createOrUpdateIssueComment($obj, 'DELETE');
    }
}


1			<?php
2
3			namespace GitScrum\Classes;
4
5			use Auth;
6			use GitScrum\Models\Branch;
7			use GitScrum\Models\Commit;
8			use GitScrum\Models\User;
9			use GitScrum\Models\Issue;
10			use GitScrum\Models\Organization;
11			use GitScrum\Models\ProductBacklog;
12			use Carbon\Carbon;
13			use GitScrum\Contracts\ProviderInterface;
14
15			class Github implements ProviderInterface
16			{
17			public function tplUser($obj)
18			{
19			return [
20			'provider_id' => $obj->id,
21			'provider' => 'github',
22			'username' => $obj->nickname,
23			'name' => $obj->name,
24			'token' => $obj->token,
25			'avatar' => @$obj->user['avatar_url'],
26			'html_url' => @$obj->user['html_url'],
27			'bio' => @$obj->user['bio'],
28			'since' => Carbon::parse($obj->user['created_at'])->toDateTimeString(),
29			'location' => @$obj->user['location'],
30			'blog' => @$obj->user['blog'],
31			'email' => $obj->email,
32			];
33			}
34
35			public function tplRepository($repo, $slug = false)
			0 ignored issues – show Documentation introduced 2016-12-05 14:48 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
36			{
37			return (object) [
38			'provider_id' => $repo->id,
39			'organization_id' => $this->organization($repo->owner->login),
40			'organization_title' => $repo->owner->login,
41			'slug' => $slug ? $slug : Helper::slug($repo->name),
42			'title' => $repo->name,
43			'fullname' => $repo->full_name,
44			'is_private' => $repo->private,
45			'html_url' => $repo->html_url,
46			'description' => $repo->description,
47			'fork' => $repo->fork,
48			'url' => $repo->url,
49			'since' => Carbon::parse($repo->created_at)->toDateTimeString(),
50			'pushed_at' => Carbon::parse($repo->pushed_at)->toDateTimeString(),
51			'ssh_url' => $repo->ssh_url,
52			'clone_url' => $repo->clone_url,
53			'homepage' => $repo->homepage,
54			'default_branch' => $repo->default_branch,
55			];
56			}
57
58			public function tplIssue($obj, $productBracklogId)
59			{
60			$user = User::where('username', @$obj->user->login)
61			->where('provider', 'github')->first();
62
63			return [
64			'provider_id' => $obj->id,
65			'user_id' => isset($user->id) ? $user->id : Auth::user()->id,
66			'product_backlog_id' => $productBracklogId,
67			'effort' => 0,
68			'config_issue_effort_id' => 1,
69			'issue_type_id' => 1,
70			'number' => $obj->number,
71			'title' => $obj->title,
72			'description' => $obj->body,
73			'state' => $obj->state,
74			'html_url' => $obj->html_url,
75			'created_at' => $obj->created_at,
76			'updated_at' => $obj->updated_at,
77			];
78			}
79
80			public function readRepositories()
81			{
82			$repos = collect(Helper::request('https://api.github.com/user/repos'));
83
84			$response = $repos->map(function ($repo) {
85			return $this->tplRepository($repo);
86			});
87
88			return $response;
89			}
90
91			public function createOrUpdateRepository($owner, $obj, $oldTitle = null)
			0 ignored issues – show Documentation introduced 2016-12-05 19:45 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
92			{
93			$params = [
94			'name' => str_slug($obj->title, '-'),
95			'description' => $obj->description,
96			];
97
98			if (is_null($oldTitle)) {
99			$endpoint = 'https://api.github.com/orgs/'.$owner.'/repos';
100
101			if (Auth::user()->username == $owner) {
102			$endpoint = 'https://api.github.com/user/repos';
103			}
104
105			$response = Helper::request($endpoint, true, 'POST', $params);
106			} else {
107			$oldTitle = str_slug($oldTitle, '-');
108			$response = Helper::request('https://api.github.com/repos/'.$owner.DIRECTORY_SEPARATOR.$oldTitle, true, 'POST', $params);
109			}
110
111			return (object) $response;
112			}
113
114			public function organization($login)
			0 ignored issues – show Documentation introduced 2016-11-27 06:48 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
115			{
116			$organization = Organization::where('username', $login)
117			->where('provider', 'github')->first();
118
119			if (!isset($organization)) {
120			$orgData = Helper::request('https://api.github.com/orgs/'.$login);
121
122			if (!isset($orgData->id)) {
123			$orgData = Helper::request('https://api.github.com/users/'.$login);
124			}
125
126			$data = [
127			'provider_id' => @$orgData->id,
128			'username' => @$orgData->login,
129			'url' => @$orgData->url,
130			'repos_url' => @$orgData->repos_url,
131			'events_url' => @$orgData->events_url,
132			'hooks_url' => @$orgData->hooks_url,
133			'issues_url' => @$orgData->issues_url,
134			'members_url' => @$orgData->members_url,
135			'public_members_url' => @$orgData->public_members_url,
136			'avatar_url' => @$orgData->avatar_url,
137			'description' => @$orgData->description,
138			'title' => @$orgData->name,
139			'blog' => @$orgData->blog,
140			'location' => @$orgData->location,
141			'email' => @$orgData->email,
142			'public_repos' => @$orgData->public_repos,
143			'html_url' => @$orgData->html_url,
144			'total_private_repos' => @$orgData->total_private_repos,
145			'since' => @Carbon::parse($orgData->created_at)->toDateTimeString(),
146			'disk_usage' => @$orgData->disk_usage,
147			];
148
149			try {
150			$organization = Organization::create($data);
151			} catch (\Illuminate\Database\QueryException $e) {
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-11-26 09:47 UTC by Report Bug Copy Issue Report Consider adding a comment why this CATCH block is empty. Loading history...
152			}
153
154			$organization->users()->sync([Auth::id()]);
155			}
156
157			return $organization->id;
158			}
159
160			public function readCollaborators($owner, $repo, $providerId = null)
161			{
162			$collaborators = Helper::request('https://api.github.com/repos/'.$owner.'/'.$repo.'/collaborators');
163			foreach ($collaborators as $collaborator) {
164			if (isset($collaborator->id)) {
165			$data = [
166			'provider_id' => $collaborator->id,
167			'username' => $collaborator->login,
168			'name' => $collaborator->login,
169			'avatar' => $collaborator->avatar_url,
170			'html_url' => $collaborator->html_url,
171			'email' => null,
172			'remember_token' => null,
173			'bio' => null,
174			'location' => null,
175			'blog' => null,
176			'since' => null,
177			'token' => null,
178			'position_held' => null,
179			];
180
181			try {
182			$user = User::create($data);
183			} catch (\Exception $e) {
184			$user = User::where('username', $collaborator->login)
185			->where('provider', 'github')->first();
186			}
187
188			$userId[] = $user->id;
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-12-15 20:19 UTC by Report Bug Copy Issue Report `$userId` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$userId = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
189			}
190			}
191
192			$organization = Organization::where('username', $owner)
193			->where('provider', 'github')->first()->users();
194
195			if (!$organization->userActive()->count()) {
196			$organization->attach($userId);
			0 ignored issues – show Bug introduced 2016-12-26 13:39 UTC by Report Bug Copy Issue Report The variable `$userId` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
197			}
198			}
199
200			public function createBranches($owner, $product_backlog_id, $repo, $providerId = null)
201			{
202			$y = 0;
203			for ($i = 1; $i > $y; ++$i) {
204			$branches = Helper::request('https://api.github.com/repos/'.$owner.DIRECTORY_SEPARATOR.$repo.'/branches?page='.$i);
205			foreach ($branches as $branch) {
206			$data = [
207			'product_backlog_id' => $product_backlog_id,
208			'title' => $branch->name,
209			'sha' => $branch->commit->sha,
210			];
211			Branch::create($data);
212			}
213			if (count($branches) < 30) {
214			$y = $i + 2;
215			}
216			}
217			}
218
219			public function readIssues()
220			{
221			$repos = ProductBacklog::all();
222
223			foreach ($repos as $repo) {
224			$issues = Helper::request('https://api.github.com/repos/'.$repo->organization->username.
225			DIRECTORY_SEPARATOR.$repo->title.'/issues?state=all');
226
227			$issues = is_array($issues) ? $issues : [$issues];
228
229			foreach ($issues as $issue) {
230			try {
231			$data = $this->tplIssue($issue, $repo->id);
232			} catch (\Exception $e) {
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-12-20 23:10 UTC by Report Bug Copy Issue Report Consider adding a comment why this CATCH block is empty. Loading history...
233			}
234
235			if (!Issue::where('provider_id', $issue->id)->where('provider', 'github')->first()) {
236			Issue::create($data)->users()->sync([$data['user_id']]);
237			}
238			}
239			}
240			}
241
242			public function createOrUpdateIssue($obj)
			0 ignored issues – show Documentation introduced 2016-12-07 19:09 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
243			{
244			$params = [
245			'title' => $obj->title,
246			'body' => $obj->description,
247			];
248
249			$response = Helper::request('https://api.github.com/repos/'.
250			$obj->productBacklog->organization->username.DIRECTORY_SEPARATOR.
251			$obj->productBacklog->title.'/issues'.(isset($obj->number) ? DIRECTORY_SEPARATOR.$obj->number : ''),
252			true, 'POST', $params);
253
254			return (object) $response;
255			}
256
257			public function createOrUpdateIssueComment($obj, $verb = 'POST')
			0 ignored issues – show Documentation introduced 2016-12-09 02:02 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
258			{
259			$params = [
260			'body' => $obj->comment,
261			];
262
263			$response = Helper::request('https://api.github.com/repos/'.
264			$obj->issue->productBacklog->organization->username.DIRECTORY_SEPARATOR.
265			$obj->issue->productBacklog->title.'/issues'.(isset($obj->provider_id) ? '' : DIRECTORY_SEPARATOR.$obj->issue->number).'/comments'.
266			(isset($obj->provider_id) ? DIRECTORY_SEPARATOR.$obj->provider_id : ''),
267			true, $verb, $params);
268
269			return (object) $response;
270			}
271
272			public function deleteIssueComment($obj)
			0 ignored issues – show Documentation introduced 2016-12-09 02:02 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
273			{
274			return $this->createOrUpdateIssueComment($obj, 'DELETE');
275			}
276			}
277

renatomarinho / laravel-gitscrum

Push — master ( 0fc1b0...714b2e )

Github::tplRepository() A

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like