Type Casts added

[vansari]

PR Description

• added type casts in builtin function usage

Checklist

• I agree with the Code Contribution License Agreement in CONTRIBUTING.md

[codecov]

Codecov Report

Merging #22 (fc05559) into master (f2db382) will not change coverage.
The diff coverage is 85.71%.

@@            Coverage Diff            @@
##             master      #22   +/-   ##
=========================================
  Coverage     94.37%   94.37%           
  Complexity      103      103           
=========================================
  Files             1        1           
  Lines           231      231           
=========================================
  Hits            218      218           
  Misses           13       13           

Impacted Files	Coverage Δ
src/Decimal.php	94.37% <85.71%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[vansari]

@dereuromark The phpstan grumbles because of the return value from preg_replace in function fromScientific and I added a check that the result value must be a string and not an array nor null.

But now I have to add an codeCoverageStart / End because the codecov was the next tool which has grumbled now.

Should I revoke the check?

[dereuromark]

If we remove the string cast here, shouldnt we also remove it in line 841?

decimal-object/src/Decimal.php

Line 841 in b9c5d21

$this->fractionalPart = str_pad($this->fractionalPart, $scale - strlen((string)$this->integralPart), '0');

[dereuromark]

@gechetspr will have to check and merge. Looks good to me.