Saxulum\RestCrud\Request\Converter\XmlForm

Complexity

Total Complexity

5

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	toArray()	0	21	5

<?php

namespace Saxulum\RestCrud\Request\Converter;

use JMS\Serializer\Annotation as Serializer;

/**
 * @Serializer\XmlRoot("form")
 */
class XmlForm
{
    /**
     * @var string
     * @Serializer\XmlAttribute
     * @Serializer\Type("string")
     */
    protected $name;

    /**
     * @var string
     * @Serializer\XmlValue
     * @Serializer\Type("string")
     */
    protected $value;

    /**
     * @var XmlForm[]
     * @Serializer\XmlList(inline = true, entry = "form")
     * @Serializer\Type("array<Saxulum\RestCrud\Request\Converter\XmlForm>")
     */
    protected $forms;

    /**
     * @return array
     */
    public function toArray()
    {
        if ($this->value) {
            if (is_numeric($this->value)) {
                if ((int) $this->value == $this->value) {
                    $this->value = (int) $this->value;

The property $value was declared of type string, but (int) $this->value is of type integer. Maybe add a type cast?

                } else {
                    $this->value = (float) $this->value;

The property $value was declared of type string, but (double) $this->value is of type double. Maybe add a type cast?

                }
            }

            return array($this->name => $this->value);
        }

        $forms = array();
        foreach ($this->forms as $form) {
            $forms = array_replace($forms, $form->toArray());
        }

        return array($this->name => $forms);
    }
}