Nip_Form_Element_File

Complexity

Total Complexity

6

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	5	1
A	getValue()	0	23	5

<?php

class Nip_Form_Element_File extends Nip_Form_Element_Input_Abstract

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    protected $_value;

    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) {
            $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) {
                if ($_FILES[$parts[0]]) {
                    $fileData = [];
                    foreach ($_FILES[$parts[0]] as $key=>$data) {
                        $fileData[$key] = $data[$parts[1]];
                    }
                    $this->_value = $fileData;
                } else {
                    $this->_value = null;
                }
            } else {
                $this->_value = $_FILES[$name];
            }
        }

        return $this->_value;
    }
}