GitElephant\Status\StatusIndex

Complexity

Total Complexity

5

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

3 Methods

Rating	Name	Duplication	Size	Complexity
A	untracked()	0	4	1
A	all()	0	6	2
A	filterByType()	0	10	2

<?php
/**
 * GitElephant - An abstraction layer for git written in PHP
 * Copyright (C) 2013  Matteo Giachino
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program.  If not, see [http://www.gnu.org/licenses/].
 */

namespace GitElephant\Status;

use \PhpCollection\Sequence;

/**
 * Class StatusIndex
 *
 * @package GitElephant\Status
 */
class StatusIndex extends Status
{
    /**
     * @return Sequence
     */
    public function untracked()
    {
        return new Sequence();
    }

    /**
     * all files with modified status in the index
     *
     * @return Sequence
     */
    public function all()
    {
        return new Sequence(array_filter($this->files, function (StatusFile $statusFile) {
            return $statusFile->getIndexStatus() && '?' !== $statusFile->getIndexStatus();
        }));
    }

    /**
     * filter files by index status
     *
     * @param string $type
     *
     * @return Sequence
     */
    protected function filterByType($type)
    {
        if (!$this->files) {

The expression $this->files of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            return new Sequence();
        }

        return new Sequence(array_filter($this->files, function (StatusFile $statusFile) use ($type) {
            return $type === $statusFile->getIndexStatus();
        }));
    }
}