BaseChannel::getActionTitle() - Code Metrics - Inspection of "Refactor and test a lot" - antonioribeiro/firewall - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 3d0d87...075b69 )

by Antonio Carlos

created 2017-09-05 07:17 UTC

BaseChannel::getActionTitle() A

↳ Parent: BaseChannel

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

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

<?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	2	private function getActionMessage()
10		{
11	2	return config('firewall.notifications.message.message');
12		}
13
14		/**
15		* @param $item
16		*
17		* @return string
18		*/
19	2	protected function getMessage($item)
20		{
21	2	$domain = Request::server('SERVER_NAME');
22
23	2	return sprintf($this->getActionMessage(), $domain, $this->makeMessage($item));
24		}
25
26		/**
27		* @param $item
28		*
29		* @return mixed
30		*/
31	2	protected function makeMessage($item)
32		{
33	2	$ip = "{$item['ipAddress']} - {$item['host']}";
34
35	2	if ($item['type'] == 'ip') {
36	1	return "$ip";
37		}
38
39	2	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	2	public function makeGeolocation($item)
50		{
51	2	return collect([
52	2	config('firewall.notifications.message.geolocation.field_latitude') => $item['geoIp']['latitude'],
53	2	config('firewall.notifications.message.geolocation.field_longitude') => $item['geoIp']['longitude'],
54	2	config('firewall.notifications.message.geolocation.field_country_code') => $item['geoIp']['country_code'],
55	2	config('firewall.notifications.message.geolocation.field_country_name') => $item['geoIp']['country_name'],
56	2	config('firewall.notifications.message.geolocation.field_city') => $item['geoIp']['city'],
57	2	])->filter()->toArray();
58		}
59		}
60

antonioribeiro / firewall

Push — master ( 3d0d87...075b69 )

BaseChannel::getActionTitle() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like