Add more determine statements in CSSArray parsing function

[zsimple]

This is:

- [ ] a bug report
- [x] 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 behavior?

I am using WPS which is really popular in China, while it is not exactly compatible with Microsoft Excel. When reading in an excles file, it throws an exception, which happend in toCSSArray in file Reader/Xlsx.php.

What is the current behavior?

    private static function toCSSArray($style)
    {
        $style = str_replace(["\r", "\n"], '', $style);

        $temp = explode(';', $style);
        $style = [];
        foreach ($temp as $item) {
            $item = explode(':', $item);

            if (strpos($item[1], 'px') !== false) {
                $item[1] = str_replace('px', '', $item[1]);
            }
            // ...

It expects the string $item has format like "xxx:xxx", but $item string can be only an empty string.

What are the steps to reproduce?

Sorry it relate to the WPS software and can not reapear only with phpSpreadsheet, and it is more likely a WPS bug rather than phpSpreadsheet's. But it weill be very kind and easy to fix the problem with next code. I think it will help phpSpreadsheet to fit more nonstandard software.

    private static function toCSSArray($style)
    {
        $style = str_replace(["\r", "\n"], '', $style);

        $temp = explode(';', $style);
        $style = [];
        foreach ($temp as $item) {
            $item = explode(':', $item);
            if (count($item) != 2) {             // <---- Add this if
                continue;
            }

            if (strpos($item[1], 'px') !== false) {
                $item[1] = str_replace('px', '', $item[1]);
            }
            // ...

Thanks!

Which versions of PhpSpreadsheet and PHP are affected?

The latest version 27d83b4.

[PowerKiKi]

Could you please give me the result of var_dump($style) when it fails ?

[zsimple]

Of couse, it is position:absolute;left:0pt;top:0pt;margin-left:0pt;margin-top:0pt;height:24pt;width:77.25pt; . The last ";" makes something wrong by my opion.