Handle contacts with >9 phone numbers

[andig]

No description provided.

[blacksenator]

Hallo Andreas,
sieht vertraut aus (Converter) - die neuen Dateien kann ich noch nicht einordnen. Komme aber erst morgen dazu mir das zu erschließen bzw. das zu testen.

[blacksenator]

Funktioniert im Prinzip - aber der Bug in "getPhoneNumbers" ist noch drin: es wird immer nur die jeweils letzte Nummer pro Type übergeben.
Im Test-Account, den ich dir mitgeteilt hatte, sind zwei Datensätze mit >9 Nummern drin.

[andig]

Kann ich zu mindest im Testfall nicht nachvollziehen- da werden eindeutig 2 Kontakte mit jeweils 9 unterschiedlichen Telefonnummern erzeugt.

[andig]

So sieht das Ergebnis des convert Aufraufs im Testcase aus- genau das erwartest Du doch, oder?

Array
(
    [0] => SimpleXMLElement Object
        (
            [carddav_uid] => uid
            [telephony] => SimpleXMLElement Object
                (
                    [number] => Array
                        (
                            [0] => 1
                            [1] => 2
                            [2] => 3
                            [3] => 4
                            [4] => 5
                            [5] => 6
                            [6] => 7
                            [7] => 8
                            [8] => 9
                        )
                )

            [person] => SimpleXMLElement Object
                (
                    [realName] => SimpleXMLElement Object
                        (
                        )
                )
        )

    [1] => SimpleXMLElement Object
        (
            [carddav_uid] => uid
            [telephony] => SimpleXMLElement Object
                (
                    [number] => Array
                        (
                            [0] => 10
                            [1] => 11
                            [2] => 12
                            [3] => 13
                            [4] => 14
                            [5] => 15
                            [6] => 16
                            [7] => 17
                            [8] => 18
                        )
                )

            [person] => SimpleXMLElement Object
                (
                    [realName] => SimpleXMLElement Object
                        (
                        )
                )
        )
)

Oder liegt das Problem nicht in convert?

[andig]

Ansonsten ist hier aber noch ein Bug drin:

        foreach ($numbers as $number) {
            $addNumber = [];

            if (count($replaceCharacters)) {
                $number = str_replace("\xc2\xa0", "\x20", $number);   // delete the wrong ampersand conversion
                $number = strtr($number, $replaceCharacters);
                $number = trim(preg_replace('/\s+/', ' ', $number));
            }

            $addNumber['number'] = $number;
            $type = 'other';
            $numberType = strtolower($numberType);

            if (stripos($numberType, 'fax') !== false) {
                $type = 'fax_work';
            } else {
                foreach ($phoneTypes as $type => $value) {
                    if (stripos($numberType, $type) !== false) {
                        $type = $value;
                        break;
                    }
                }
            }
            $addNumber['type'] = $type;
        }

da wird die $type Variable 2x verwendet.

[andig]

Refactored the entire thing and added more tests

[blacksenator]

let´s rock