GitHub Access Token became invalid

It seems like the GitHub access token used for retrieving details about this repository from GitHub became invalid. This might prevent certain types of inspections from being run (in particular, everything related to pull requests).
Please ask an admin of your repository to re-new the access token on this website.
Failed Conditions
Push — master ( 550dab...bc8237 )
by Casper
03:45 queued 02:01
created

UgcClient::storePushLog()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 23
Code Lines 11

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 0
CRAP Score 2

Importance

Changes 0
Metric Value
cc 1
eloc 11
c 0
b 0
f 0
nc 1
nop 9
dl 0
loc 23
ccs 0
cts 11
cp 0
crap 2
rs 9.9

How to fix   Many Parameters   

Many Parameters

Methods with many parameters are not only hard to understand, but their parameters also often become inconsistent when you need more, or different data.

There are several approaches to avoid long parameter lists:

1
<?php
2
3
namespace NStack\Clients;
4
5
use NStack\Exceptions\FailedToParseException;
6
use NStack\Models\File;
7
use NStack\Models\IpAddress;
8
use NStack\Models\Proposal;
9
use NStack\Models\ProposalDeleted;
0 ignored issues
show
Bug introduced by
The type NStack\Models\ProposalDeleted was not found. Maybe you did not declare it correctly or list all dependencies?

The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. excluded_paths: ["lib/*"], you can move it to the dependency path list as follows:

filter:
    dependency_paths: ["lib/*"]

For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths

Loading history...
10
11
/**
12
 * Class UgcClient
13
 *
14
 * @package NStack\Clients
15
 * @author  Tiago Araujo <[email protected]>
16
 */
17
class UgcClient extends NStackClient
18
{
19
    /** @var string */
20
    protected $path = 'ugc';
21
22
    /**
23
     * storePushLog
24
     *
25
     * @param String $provider
26
     * @param String $key
27
     * @param String $type
28
     * @param bool $succeeded
29
     * @param String|null $request
30
     * @param String|null $response
31
     * @param int|null $userId
32
     * @param String|null $relationType
33
     * @param int|null $relationId
34
     */
35
    public function storePushLog(
36
        String $provider,
37
        String $key,
38
        String $type,
39
        bool $succeeded,
40
        String $request = null,
41
        String $response = null,
42
        int $userId = null,
43
        String $relationType = null,
44
        int $relationId = null
45
    )
46
    {
47
        $this->client->post($this->buildPath($this->path), [
48
            'body' => '{
49
                            "provider": "' . $provider . '",
50
                            "key": "' . $key . '",
51
                            "type": "' . $type . '",
52
                            "succeeded": ' . $succeeded . ',
53
                            "request": { ' . $request . ', },
54
                            "response": { ' . $response . ', },
55
                            "user_id": ' . $userId . '
56
                            "relation_type": "' . $relationType . '",
57
                            "relation_id": ' . $relationId . '
58
                        }'
59
        ]);
60
    }
61
62
}