CheckSslCertificates::handle() - Code Metrics - Inspection of "wip" - spatie/laravel-uptime-monitor - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9e170a...fa9406 )

by Freek

created 2016-09-25 18:08 UTC

CheckSslCertificates::handle() A

↳ Parent: CheckSslCertificates

Complexity

Conditions	1
Paths	1

Size

Total Lines	14
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	7
nc	1
nop	0
dl	0
loc	14
rs	9.4285
c	0
b	0
f	0

<?php

namespace Spatie\UptimeMonitor\Commands;

use Spatie\SslCertificate\SslCertificate;
use Spatie\UptimeMonitor\Models\Site;
use Spatie\UptimeMonitor\Services\PingMonitors\UptimeMonitorCollection;
use Spatie\UptimeMonitor\SiteRepository;

class CheckSslCertificates extends BaseCommand
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'uptime-monitor:check-ssl';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Check the ssl certificates of all sites';

    public function handle()
    {
        $sites = SiteRepository::getAllForSslCheck();

        $this->comment('Start checking the ssl certificate of '.count($sites).' sites...');

        SiteRepository::getAllForSslCheck()->each(function(Site $site) {
            $this->info("Checking ssl-certificate of {$site->url}");

            $certificate = SslCertificate::createForHostName($site->url->getHost());
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        });

        $this->info('All done!');
    }
}


1			<?php
2
3			namespace Spatie\UptimeMonitor\Commands;
4
5			use Spatie\SslCertificate\SslCertificate;
6			use Spatie\UptimeMonitor\Models\Site;
7			use Spatie\UptimeMonitor\Services\PingMonitors\UptimeMonitorCollection;
8			use Spatie\UptimeMonitor\SiteRepository;
9
10			class CheckSslCertificates extends BaseCommand
11			{
12			/**
13			* The name and signature of the console command.
14			*
15			* @var string
16			*/
17			protected $signature = 'uptime-monitor:check-ssl';
18
19			/**
20			* The console command description.
21			*
22			* @var string
23			*/
24			protected $description = 'Check the ssl certificates of all sites';
25
26			public function handle()
27			{
28			$sites = SiteRepository::getAllForSslCheck();
29
30			$this->comment('Start checking the ssl certificate of '.count($sites).' sites...');
31
32			SiteRepository::getAllForSslCheck()->each(function(Site $site) {
33			$this->info("Checking ssl-certificate of {$site->url}");
34
35			$certificate = SslCertificate::createForHostName($site->url->getHost());
			0 ignored issues – show Unused Code introduced 2016-09-25 18:12 UTC by Report Bug Copy Issue Report `$certificate` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
36			});
37
38			$this->info('All done!');
39			}
40			}
41

spatie / laravel-uptime-monitor

GitHub Access Token became invalid

Push — master ( 9e170a...fa9406 )

CheckSslCertificates::handle() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like