LaravelPaths::isIgnored() - Code Metrics - Inspection of "Add config to ignore paths for check." - imanghafoori1/laravel-microscope - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#76)

by amir

created 2020-08-20 08:56 UTC

LaravelPaths::isIgnored() A

↳ Parent: LaravelPaths

Complexity

Conditions	5
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	5
nc	4
nop	1
dl	0
loc	15
rs	9.4555
c	0
b	0
f	0

<?php

namespace Imanghafoori\LaravelMicroscope\LaravelPaths;

use Illuminate\Database\Eloquent\Factory;
use Illuminate\Support\Str;
use Imanghafoori\LaravelMicroscope\Analyzers\FilePath;

class LaravelPaths
{
    public static function factoryDirs()
    {
        return app()->make(Factory::class)->loadedPaths;
    }

    public static function migrationDirs()
    {
        // normalize the migration paths
        $migrationDirs = [];

        foreach (app('migrator')->paths() as $path) {
            // Excludes the migrations within "vendor" folder:
            if (! Str::startsWith($path, [base_path('vendor')])) {
                $migrationDirs[] = FilePath::normalize($path);
            }
        }

        $migrationDirs[] = app()->databasePath().DIRECTORY_SEPARATOR.'migrations';

        return $migrationDirs;
    }

    /**
     * Check given path should be ignored.
     *
     * @param string $path
     * @return boolean
     */
    public static function isIgnored($path)
    {
        $ignorePatterns = config('microscope.ignore');
        if (is_array($ignorePatterns) == false || is_string($path) == false) {

            return false;
        }

        foreach ($ignorePatterns as $ignorePattern) {
            if (fnmatch(rtrim(base_path(), '\\/').'/'.ltrim($ignorePattern, '\\/'), $path)) {
                return true;
            }
        }

        return false;
    }
}


1			<?php
2
3			namespace Imanghafoori\LaravelMicroscope\LaravelPaths;
4
5			use Illuminate\Database\Eloquent\Factory;
6			use Illuminate\Support\Str;
7			use Imanghafoori\LaravelMicroscope\Analyzers\FilePath;
8
9			class LaravelPaths
10			{
11			public static function factoryDirs()
12			{
13			return app()->make(Factory::class)->loadedPaths;
14			}
15
16			public static function migrationDirs()
17			{
18			// normalize the migration paths
19			$migrationDirs = [];
20
21			foreach (app('migrator')->paths() as $path) {
22			// Excludes the migrations within "vendor" folder:
23			if (! Str::startsWith($path, [base_path('vendor')])) {
24			$migrationDirs[] = FilePath::normalize($path);
25			}
26			}
27
28			$migrationDirs[] = app()->databasePath().DIRECTORY_SEPARATOR.'migrations';
29
30			return $migrationDirs;
31			}
32
33			/**
34			* Check given path should be ignored.
35			*
36			* @param string $path
37			* @return boolean
38			*/
39			public static function isIgnored($path)
40			{
41			$ignorePatterns = config('microscope.ignore');
42			if (is_array($ignorePatterns) == false \|\| is_string($path) == false) {
			0 ignored issues – show Coding Style Best Practice introduced 2020-08-20 08:57 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing two booleans. Considering using the strict comparison `===` instead. When comparing two booleans, it is generally considered safer to use the strict comparison operator. Loading history...
43			return false;
44			}
45
46			foreach ($ignorePatterns as $ignorePattern) {
47			if (fnmatch(rtrim(base_path(), '\\/').'/'.ltrim($ignorePattern, '\\/'), $path)) {
48			return true;
49			}
50			}
51
52			return false;
53			}
54			}
55

imanghafoori1 / laravel-microscope

Pull Request — master (#76)

LaravelPaths::isIgnored() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like