1 | <?php |
||||||
2 | |||||||
3 | namespace PiedWeb\Curl; |
||||||
4 | |||||||
5 | trait StaticWrapperTrait |
||||||
6 | { |
||||||
7 | 3 | public static function get(string $url) |
|||||
8 | { |
||||||
9 | 3 | $request = new self($url); |
|||||
0 ignored issues
–
show
|
|||||||
10 | $request |
||||||
11 | 3 | ->setDefaultGetOptions() |
|||||
0 ignored issues
–
show
It seems like
setDefaultGetOptions() must be provided by classes using this trait. How about adding it as abstract method to this trait?
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the
Loading history...
|
|||||||
12 | 3 | ->setDefaultSpeedOptions() |
|||||
13 | 3 | ->setNoFollowRedirection() |
|||||
14 | 3 | ->setDesktopUserAgent() |
|||||
15 | ; |
||||||
16 | |||||||
17 | 3 | $response = $request->exec(); |
|||||
0 ignored issues
–
show
It seems like
exec() must be provided by classes using this trait. How about adding it as abstract method to this trait?
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the
Loading history...
|
|||||||
18 | |||||||
19 | 3 | return is_int($response) ? $response : $response->getContent(); |
|||||
20 | } |
||||||
21 | } |
||||||
22 |
This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.
If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.