FsStorage::getMtime() - Code Metrics - Inspection of "Add test for @import" - LucidTaZ/yii2-scssphp - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( 3b58ea...76dc22 )

by Thijs

created 2017-07-18 09:21 UTC

FsStorage::getMtime() A

↳ Parent: FsStorage

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	1
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	9
ccs	1
cts	1
cp	1
rs	9.6666
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	1
crap	2

<?php

namespace lucidtaz\yii2scssphp\storage;

use RuntimeException;

class FsStorage implements Storage
{
    public function exists(string $filename): bool
    {
        return file_exists($filename);
    }

    public function get(string $filename): string
    {
        return file_get_contents($filename);
    }

    public function put(string $filename, string $contents): bool
    {
        return file_put_contents($filename, $contents) !== false;
    }

    public function remove(string $filename): bool
    {
        return @unlink($filename);
    }

    public function touch(string $filename, int $mtime): bool
    {
        return touch($filename, $mtime);
    }

    public function getMtime(string $filename): int
    {
        $mtime = @filemtime($filename);
        if ($mtime === false) {
            throw new RuntimeException('Could not determine mtime for ' . $filename);
            // ... or should we adhere to PHP's int|false convention?
        }
        return $mtime;
    }
}


1		<?php
2
3		namespace lucidtaz\yii2scssphp\storage;
4
5		use RuntimeException;
6
7		class FsStorage implements Storage
8		{
9	1	public function exists(string $filename): bool
10		{
11	1	return file_exists($filename);
12		}
13
14	1	public function get(string $filename): string
15		{
16	1	return file_get_contents($filename);
17		}
18
19	1	public function put(string $filename, string $contents): bool
20		{
21	1	return file_put_contents($filename, $contents) !== false;
22		}
23
24	1	public function remove(string $filename): bool
25		{
26	1	return @unlink($filename);
27		}
28
29	2	public function touch(string $filename, int $mtime): bool
30		{
31	2	return touch($filename, $mtime);
32	2	}
33	1
34		public function getMtime(string $filename): int
35		{
36	1	$mtime = @filemtime($filename);
37		if ($mtime === false) {
38		throw new RuntimeException('Could not determine mtime for ' . $filename);
39		// ... or should we adhere to PHP's int\|false convention?
40		}
41		return $mtime;
42		}
43		}
44

LucidTaZ / yii2-scssphp

Push — master ( 3b58ea...76dc22 )

FsStorage::getMtime() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like