LeKoala\CommonExtensions\BinaryIPExtension

Complexity

Total Complexity

4

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getIpLocationDetails()	0	7	2
A	onBeforeWrite()	0	7	2

<?php

namespace LeKoala\CommonExtensions;

use SilverStripe\ORM\DataExtension;
use SilverStripe\Control\Controller;
use SilverStripe\Core\Injector\Injector;

/**
 * Store IP as binary
 *
 * @link https://stackoverflow.com/questions/22636912/store-both-ipv4-and-ipv6-address-in-a-single-column
 * @property \BinaryIPExtension $owner
 * @property string $IP
 */
class BinaryIPExtension extends DataExtension
{
    private static $db = [

The private property $db is not used, and could be removed.

        "IP" => "BinaryIP"
    ];

    public function onBeforeWrite()
    {
        $controller = Controller::curr();
        // This is best used when IP is set on creation
        if (!$this->owner->IP) {
            $ip = $controller->getRequest()->getIP();
            $this->owner->IP = $ip;
        }
    }

    /**
     * This require geotools module
     *
     * @return \LeKoala\GeoTools\Models\Address|null

The type LeKoala\GeoTools\Models\Address was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    public function getIpLocationDetails()
    {
        if (!$this->owner->IP) {
            return null;
        }
        $geolocator = Injector::inst()->get(\LeKoala\Geo\Services\Geolocator::class);

The type LeKoala\Geo\Services\Geolocator was not found. Maybe you did not declare it correctly or list all dependencies?

        return $geolocator->geolocate($this->owner->IP);
    }
}