BlitzPHP\Formatter\XmlFormatter::arrayToXml()

Complexity

Conditions	12
Paths	25

Size

Total Lines	38
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	12

Importance

Changes	1
Bugs	1	Features	0

<?php
    protected function arrayToXml(array $data, SimpleXMLElement &$structure, string $basenode): void
    {
        foreach ($data as $key => $value) {
            // change false/true en 0/1
            if (is_bool($value)) {
                $value = (int) $value;
            }

            // pas de touches numériques dans notre xml s'il vous plait !
            if (is_numeric($key)) {
                // crée une clé de chaîne...
                $key = singular($basenode) !== $basenode ? singular($basenode) : 'item';
            }

            // remplace tout ce qui n'est pas alphanumérique
            $key = preg_replace('/[^a-z_\-0-9]/i', '', $key);

            if ($key === '_attributes' && (is_array($value) || is_object($value))) {
                $attributes = $value;
                if (is_object($attributes)) {
                    $attributes = get_object_vars($attributes);
                }

                foreach ($attributes as $attribute_name => $attribute_value) {
                    $structure->addAttribute($attribute_name, $attribute_value);
                }
            }
            // s'il y a un autre tableau trouvé appelez récursivement cette fonction
            elseif (is_array($value) || is_object($value)) {
                $node = $structure->addChild($key);

                // appel récursif
                $this->arrayToXml($value, $node, $key);

It seems like $value can also be of type object; however, parameter $data of BlitzPHP\Formatter\XmlFormatter::arrayToXml() does only seem to accept array, maybe add an additional type check?

            } else {
                // ajouter un seul noeud
                $value = htmlspecialchars(html_entity_decode($value ?? '', ENT_QUOTES, 'UTF-8'), ENT_QUOTES, 'UTF-8');

                $structure->addChild($key, $value);
            }