Inconsistent string handling between Sum::sumErroringStrings() and Sum::sumIgnoringStrings()

[betterphp]

This is:

• a bug report
• a feature request
• not a usage question (ask them on https://stackoverflow.com/questions/tagged/phpspreadsheet or https://gitter.im/PHPOffice/PhpSpreadsheet)

What is the expected behaviour?

Numeric string handling in Sum::sumErroringStrings() and Sum::sumIgnoringStrings() should be consistent.

What is the current behaviour?

sumErroringStrings casts all string params to integers whereas sumIgnoringStrings just sums them. The expected behaviour would be for both functions to cast string inputs to floats.

What are the steps to reproduce?

Please provide a Minimal, Complete, and Verifiable example of code that exhibits the issue without relying on an external Excel file or a web server:

<?php

require __DIR__ . '/vendor/autoload.php';

use PhpOffice\PhpSpreadsheet\Calculation\MathTrig\Sum;

// This outputs int(4)
var_dump(Sum::sumErroringStrings("1.2", "3.4"));

// This outputs float(4.6)
var_dump(Sum::sumIgnoringStrings("1.2", "3.4"));

What features do you think are causing the issue

• Reader
• Writer
• Styles
• Data Validations
• Formula Calculations
• Charts
• AutoFilter
• Form Elements

Does an issue affect all spreadsheet file formats? If not, which formats are affected?

Yes, affects all formats

Which versions of PhpSpreadsheet and PHP are affected?

All PHP versions and 1.28 of phpspreadsheet

[betterphp]

Created a PR for this #3653