Sirius\Upload\Result\Collection

Complexity

Total Complexity

13

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes	8
Bugs	0	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	4
A	isValid()	0	10	3
A	getMessages()	0	10	2
A	clear()	0	7	2
A	confirm()	0	7	2

<?php

namespace Sirius\Upload\Result;

use Sirius\Upload\Container\ContainerInterface;

class Collection extends \ArrayIterator
{
    public function __construct($files = array(), ContainerInterface $container = null)
    {
        $filesArray = array();
        if (is_array($files) && !empty($files)) {
            foreach ($files as $key => $file) {
                $filesArray[$key] = new File($file, $container);

It seems like $container defined by parameter $container on line 9 can be null; however, Sirius\Upload\Result\File::__construct() does not accept null, maybe add an additional type check?

            }
        }
        parent::__construct($filesArray);
    }

    public function clear()
    {
        foreach ($this as $file) {
            /* @var $file \Sirius\Upload\Result\File */
            $file->clear();
        }
    }

    public function confirm()
    {
        foreach ($this as $file) {
            /* @var $file \Sirius\Upload\Result\File */
            $file->confirm();
        }
    }

    public function isValid()
    {
        foreach ($this->getMessages() as $messages) {
            if ($messages) {
                return false;
            }
        }

        return true;
    }

    public function getMessages()
    {
        $messages = array();
        foreach ($this as $key => $file) {
            /* @var $file \Sirius\Upload\Result\File */
            $messages[$key] = $file->getMessages();
        }

        return $messages;
    }

}