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...
46
{
47
$a = bcmul($val, "1.0", 1);
48
return bcdiv($a, "100000000", 8);
49
}
50
51
/**
52
* Convert a float value to BTC satoshi integer string
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...
55
{
56
return bcmul($val, "100000000", 0);
57
}
58
59
/**
60
* From comment on http://php.net/manual/en/ref.bc.php
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 $iDecimals can also be of type double; however, parameter $decimals of number_format() does only seem to accept integer, 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
appino /
Blockchain
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.