UseImperativeMood::pass() - Code Metrics - Inspection of "Add personal namespace" - captainhook-git/captainhook - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( b34030...56c38f )

by Sebastian

created 2016-08-31 18:40 UTC

UseImperativeMood::pass() A

↳ Parent: UseImperativeMood

Complexity

Conditions	3
Paths	3

Size

Total Lines	17
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	17
c	0
b	0
f	0
ccs	8
cts	8
cp	1
rs	9.4285
cc	3
eloc	12
nc	3
nop	1
crap	3

<?php
/**
 * This file is part of CaptainHook.
 *
 * (c) Sebastian Feldmann <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace sebastianfeldmann\CaptainHook\Hook\Message\Validator\Rule;

use sebastianfeldmann\CaptainHook\Git\CommitMessage;

/**
 * Class UseImperativeMood
 *
 * @package CaptainHook
 * @author  Sebastian Feldmann <[email protected]>
 * @link    https://github.com/sebastianfeldmann/captainhook
 * @since   Class available since Release 0.9.0
 */
class UseImperativeMood extends Base
{
    /**
     * Constructor.
     */
    public function __construct()
    {
        $this->hint = 'Subject should be written in imperative mood';
    }

    /**
     * Check if commit message is written in imperative mood.
     *
     * @param  \sebastianfeldmann\CaptainHook\Git\CommitMessage $msg
     * @return bool
     */
    public function pass(CommitMessage $msg)
    {
        $lowerSubject = strtolower($msg->getSubject());
        $blackList    = [
            'uploaded',
            'updated',
            'added',
            'created',
        ];
        foreach ($blackList as $term) {
            if (strpos($lowerSubject, $term) !== false) {
                $this->hint .= PHP_EOL . 'Invalid use of \'' . $term . '\'';
                return false;
            }
        }
        return true;
    }
}


1		<?php
2		/**
3		* This file is part of CaptainHook.
4		*
5		* (c) Sebastian Feldmann <[email protected]>
6		*
7		* For the full copyright and license information, please view the LICENSE
8		* file that was distributed with this source code.
9		*/
10		namespace sebastianfeldmann\CaptainHook\Hook\Message\Validator\Rule;
11
12		use sebastianfeldmann\CaptainHook\Git\CommitMessage;
13
14		/**
15		* Class UseImperativeMood
16		*
17		* @package CaptainHook
18		* @author Sebastian Feldmann <[email protected]>
19		* @link https://github.com/sebastianfeldmann/captainhook
20		* @since Class available since Release 0.9.0
21		*/
22		class UseImperativeMood extends Base
23		{
24		/**
25		* Constructor.
26		*/
27	5	public function __construct()
28		{
29	5	$this->hint = 'Subject should be written in imperative mood';
30	5	}
31
32		/**
33		* Check if commit message is written in imperative mood.
34		*
35		* @param \sebastianfeldmann\CaptainHook\Git\CommitMessage $msg
36		* @return bool
37		*/
38	4	public function pass(CommitMessage $msg)
39		{
40	4	$lowerSubject = strtolower($msg->getSubject());
41		$blackList = [
42	4	'uploaded',
43		'updated',
44		'added',
45		'created',
46		];
47	4	foreach ($blackList as $term) {
48	4	if (strpos($lowerSubject, $term) !== false) {
49	1	$this->hint .= PHP_EOL . 'Invalid use of \'' . $term . '\'';
50	4	return false;
51		}
52		}
53	3	return true;
54		}
55		}
56

captainhook-git / captainhook

Push — master ( b34030...56c38f )

UseImperativeMood::pass() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like