It is generally recommended to explicitly declare the visibility for methods.
Adding explicit visibility (private, protected, or public) is generally
recommend to communicate to other developers how, and from where this method
is intended to be used.
Loading history...
24
{
25
$this->provider
26
->recipientOfStringIs(
27
new functor(
28
function($prefix) use ($string)
29
{
30
(
31
new php\string\recipient\prefix(
32
$prefix,
33
new functor(
34
function($prefixedString)
35
{
36
(
37
new php\string\recipient\prefix(
38
(string) $this->buffer,
39
new functor(
40
function($buffer)
41
{
42
$this->buffer = $buffer;
43
}
44
)
45
)
46
)
47
->stringIs($prefixedString)
48
;
49
}
50
)
51
)
52
)
53
->stringIs($string)
54
;
55
}
56
)
57
)
58
;
59
}
60
61
function recipientOfStringIs(php\string\recipient $recipient) :void
It is generally recommended to explicitly declare the visibility for methods.
Adding explicit visibility (private, protected, or public) is generally
recommend to communicate to other developers how, and from where this method
is intended to be used.
It seems like $this->buffer can also be of type null; however, parameter $string of norsys\score\php\string\recipient::stringIs() does only seem to accept string, maybe add an additional type check?
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the ignore-type annotation
norsys /
score
Loading history...
Adding explicit visibility (
private,protected, orpublic) is generally recommend to communicate to other developers how, and from where this method is intended to be used.