SebastianFeldmann\Git\Command\Log\Log

Complexity

Total Complexity

10

Size/Duplication

Total Lines	121
Duplicated Lines	0 %

Coupling/Cohesion

Components	5
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	prettyFormat()	0	5	1
A	withMerges()	0	5	2
A	abbrevCommit()	0	5	2
A	byRevision()	0	6	2
A	authoredBy()	0	5	1
A	byDate()	0	6	2

<?php
/**
 * This file is part of SebastianFeldmann\Git.
 *
 * (c) Sebastian Feldmann <sf@sebastian-feldmann.info>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace SebastianFeldmann\Git\Command\Log;

use SebastianFeldmann\Git\Command\Base;

/**
 * Class Log
 *
 * @package SebastianFeldmann\Git
 */
abstract class Log extends Base
{
    /**
     * Pretty log format.
     * --pretty
     *
     * @var string
     */
    protected $format = '%h -%d %s (%ci) <%an>';

    /**
     * Include or hide merge commits.
     * --no-merges
     *
     * @var string
     */
    protected $merges = ' --no-merges';

    /**
     * Shorten commit hashes.
     * --abbrev-commit
     *
     * @var bool
     */
    protected $abbrev = ' --abbrev-commit';

    /**
     * Can be revision or date query.
     *  1.0.0..
     *  0.9.0..1.2.0
     *  --after='2016-12-31'
     *  --after='2016-12-31' --before='2017-01-31'
     *
     * @var string
     */
    protected $since;

    /**
     * Filter log by author.
     * --author
     *
     * @var string
     */
    protected $author;

    /**
     * Define the pretty log format.
     *
     * @param  string $format
     * @return \SebastianFeldmann\Git\Command\Log\Log
     */
    public function prettyFormat(string $format) : Log
    {
        $this->format = $format;
        return $this;
    }

    /**
     * Define merge commit behaviour.
     *
     * @param  bool $bool
     * @return \SebastianFeldmann\Git\Command\Log\Log
     */
    public function withMerges(bool $bool = true) : Log
    {
        $this->merges = ($bool ? '' : ' --no-merges');
        return $this;
    }

    /**
     * Define commit hash behaviour.
     *
     * @param  bool $bool
     * @return \SebastianFeldmann\Git\Command\Log\Log
     */
    public function abbrevCommit(bool $bool = true) : Log
    {
        $this->abbrev = ($bool ? ' --abbrev--commit' : '');

The property $abbrev was declared of type boolean, but $bool ? ' --abbrev--commit' : '' is of type string. Maybe add a type cast?

        return $this;
    }

    /**
     * Set revision range.
     *
     * @param  string $from
     * @param  string $to
     * @return \SebastianFeldmann\Git\Command\Log\Log
     */
    public function byRevision(string $from, string $to = '') : Log
    {
        $this->since = ' ' . escapeshellarg($from) . '..'
                     . (empty($to) ? '' : escapeshellarg($to));
        return $this;
    }

    /**
     * Set author filter.
     *
     * @param  string $author
     * @return \SebastianFeldmann\Git\Command\Log\Log
     */
    public function authoredBy(string $author) : Log
    {
        $this->author = ' --author=' . escapeshellarg($author);
        return $this;
    }

    /**
     * Set date range.
     *
     * @param  string $from
     * @param  string $to
     * @return \SebastianFeldmann\Git\Command\Log\Log
     */
    public function byDate(string $from, string $to = '') : Log
    {
        $this->since = ' --after=' . escapeshellarg($from)
                     . (empty($to) ? '' : ' --before=' . escapeshellarg($to));
        return $this;
    }
}