BaseChannel::makeGeolocation() - Code Metrics - Inspection of "Remove unused vars" - antonioribeiro/firewall - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 42b6f8...140d5a )

by Antonio Carlos

created 2017-09-05 21:58 UTC

BaseChannel::makeGeolocation() A

↳ Parent: BaseChannel

Complexity

Conditions	1
Paths	1

Size

Total Lines	10
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	1

Importance

Changes

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

<?php

namespace PragmaRX\Firewall\Notifications\Channels;

use Request;

abstract class BaseChannel implements Contract
{
    private function getActionMessage()
    {
        return config('firewall.notifications.message.message');
    }

    /**
     * @param $item
     *
     * @return string
     */
    protected function getMessage($item)
    {
        $domain = Request::server('SERVER_NAME');

        return sprintf($this->getActionMessage(), $domain, $this->makeMessage($item));
    }

    /**
     * @param $item
     *
     * @return mixed
     */
    protected function makeMessage($item)
    {
        $ip = "{$item['ipAddress']} - {$item['host']}";

        if ($item['type'] == 'ip') {
            return "$ip";
        }

        return "{$item['country_code']}-{$item['country_name']} ({$ip})";
    }

    /**
     * Make a geolocation model for the item.
     *
     * @param $item
     *
     * @return array
     */
    public function makeGeolocation($item)
    {
        return collect([
            config('firewall.notifications.message.geolocation.field_latitude')     => $item['geoIp']['latitude'],
            config('firewall.notifications.message.geolocation.field_longitude')    => $item['geoIp']['longitude'],
            config('firewall.notifications.message.geolocation.field_country_code') => $item['geoIp']['country_code'],
            config('firewall.notifications.message.geolocation.field_country_name') => $item['geoIp']['country_name'],
            config('firewall.notifications.message.geolocation.field_city')         => $item['geoIp']['city'],
        ])->filter()->toArray();
    }
}


1		<?php
2
3		namespace PragmaRX\Firewall\Notifications\Channels;
4
5		use Request;
6
7		abstract class BaseChannel implements Contract
8		{
9	4	private function getActionMessage()
10		{
11	4	return config('firewall.notifications.message.message');
12		}
13
14		/**
15		* @param $item
16		*
17		* @return string
18		*/
19	4	protected function getMessage($item)
20		{
21	4	$domain = Request::server('SERVER_NAME');
22
23	4	return sprintf($this->getActionMessage(), $domain, $this->makeMessage($item));
24		}
25
26		/**
27		* @param $item
28		*
29		* @return mixed
30		*/
31	4	protected function makeMessage($item)
32		{
33	4	$ip = "{$item['ipAddress']} - {$item['host']}";
34
35	4	if ($item['type'] == 'ip') {
36	2	return "$ip";
37		}
38
39	3	return "{$item['country_code']}-{$item['country_name']} ({$ip})";
40		}
41
42		/**
43		* Make a geolocation model for the item.
44		*
45		* @param $item
46		*
47		* @return array
48		*/
49	5	public function makeGeolocation($item)
50		{
51	5	return collect([
52	5	config('firewall.notifications.message.geolocation.field_latitude') => $item['geoIp']['latitude'],
53	5	config('firewall.notifications.message.geolocation.field_longitude') => $item['geoIp']['longitude'],
54	5	config('firewall.notifications.message.geolocation.field_country_code') => $item['geoIp']['country_code'],
55	5	config('firewall.notifications.message.geolocation.field_country_name') => $item['geoIp']['country_name'],
56	5	config('firewall.notifications.message.geolocation.field_city') => $item['geoIp']['city'],
57	5	])->filter()->toArray();
58		}
59		}
60

antonioribeiro / firewall

Push — master ( 42b6f8...140d5a )

BaseChannel::makeGeolocation() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like