Nip_Form_Element_File

Complexity

Total Complexity

8

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	5	1
A	getValue()	0	7	2
A	generateValueFromRequest()	0	18	5

<?php

class Nip_Form_Element_File extends Nip_Form_Element_Input_Abstract
{
    protected $_value = null;

    public function init()
    {
        parent::init();
        $this->setAttrib('type', 'file');
        $this->getForm()->setAttrib('enctype', 'multipart/form-data');
    }

    public function getValue($requester = 'abstract')

The parameter $requester is not used and could be removed.

    {
        if ($this->_value == null) {
            $this->_value = $this->generateValueFromRequest();
        }

        return $this->_value;
    }

    /**
     * @return array|false|mixed
     */
    protected function generateValueFromRequest()
    {
        $name = $this->getName();

Are you sure the assignment to $name is correct as $this->getName() targeting Nip\Form\Elements\AbstractElement::getName() seems to always return null.

        $name = str_replace(']', '', $name);
        $parts = explode('[', $name);

        if (count($parts) <= 1) {
            return isset($_FILES[$name]) ? $_FILES[$name] : false;
        }

        if (!isset($_FILES[$parts[0]])) {
            return false;
        }
        $fileData = [];
        foreach ($_FILES[$parts[0]] as $key=>$data) {
            $fileData[$key] = $data[$parts[1]];
        }
        return $fileData;
    }
}