Closed JercSi closed 6 months ago
Merging #100 (84cc0d5) into master (0069435) will decrease coverage by
0.32%
. The diff coverage is0.00%
.
@@ Coverage Diff @@
## master #100 +/- ##
============================================
- Coverage 63.43% 63.11% -0.33%
+ Complexity 934 928 -6
============================================
Files 47 47
Lines 3036 2879 -157
============================================
- Hits 1926 1817 -109
+ Misses 1110 1062 -48
Impacted Files | Coverage Δ | |
---|---|---|
src/Renderer/Image/SvgImageBackEnd.php | 0.00% <0.00%> (ø) |
|
src/Renderer/Color/Gray.php | 41.66% <0.00%> (-4.49%) |
:arrow_down: |
src/Renderer/Image/ImagickImageBackEnd.php | 62.34% <0.00%> (-2.19%) |
:arrow_down: |
src/Renderer/Color/Rgb.php | 43.33% <0.00%> (-1.83%) |
:arrow_down: |
src/Writer.php | 90.00% <0.00%> (-1.67%) |
:arrow_down: |
src/Common/BitMatrix.php | 35.77% <0.00%> (-1.62%) |
:arrow_down: |
src/Common/ErrorCorrectionLevel.php | 78.57% <0.00%> (-1.43%) |
:arrow_down: |
src/Common/CharacterSetEci.php | 51.42% <0.00%> (-1.35%) |
:arrow_down: |
src/Renderer/RendererStyle/RendererStyle.php | 66.66% <0.00%> (-1.34%) |
:arrow_down: |
src/Encoder/QrCode.php | 63.33% <0.00%> (-1.19%) |
:arrow_down: |
... and 38 more |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 0069435...84cc0d5. Read the comment docs.
When using a Slovenian locale, output of scale and translate is wrong.
Expected result:
scale(3.123)
Actual result:scale(3,123)
which means wrong scale Issue: Slovenian decimal separator is ',' and not a '.' (https://www.localeplanet.com/icu/sl-SI/index.html)Solution: when using a
sprintf
use a non-local aware float formatting.Test code
Test code result