Warehouse::getCMSFields() - Code Metrics - silvershop/silvershop-shipping - Measure and Improve Code Quality continuously with Scrutinizer

Warehouse::getCMSFields() A
last analyzed 2017-12-15 03:16 UTC

↳ Parent: Warehouse

Complexity

Conditions	1
Paths	1

Size

Total Lines	10
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	10
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	6
nc	1
nop	0

<?php

/**
 * @package silvershop-shipping
 */
class Warehouse extends DataObject
{
    private static $db = array(

        'Title' => 'Varchar'
    );

    private static $has_one = array(

        'Address' => 'Address'
    );

    public static $summary_fields = array(
        'Title', 'Address'
    );

    public function getCMSFields()
    {
        $fields = parent::getCMSFields();
        $fields->removeByName("AddressID");
        $fields->addFieldToTab("Root.Main",
            HasOneButtonField::create("Address", "Address", $this)
        );

        return $fields;
    }

    /**
     * Get the closest warehouse to an address.
     *
     * @param  Address $address
     * @return Warehouse
     */
    public static function closest_to(Address $address)
    {
        $warehouses = self::get()
            ->where("\"AddressID\" IS NOT NULL");
        $closestwarehouse = null;
        $shortestdistance = null;
        foreach ($warehouses as $warehouse) {
            $dist = $warehouse->Address()->distanceTo($address);
            if ($dist && ($shortestdistance === null || $dist < $shortestdistance)) {
                $closestwarehouse = $warehouse;
                $shortestdistance = $dist;
            }
        }

        return $closestwarehouse;
    }
}


1			<?php
2
3			/**
4			* @package silvershop-shipping
5			*/
6			class Warehouse extends DataObject
7			{
8			private static $db = array(
			0 ignored issues – show Comprehensibility introduced 2016-03-01 01:25 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history...
9			'Title' => 'Varchar'
10			);
11
12			private static $has_one = array(
			0 ignored issues – show Comprehensibility introduced 2016-03-01 01:25 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history...
13			'Address' => 'Address'
14			);
15
16			public static $summary_fields = array(
17			'Title', 'Address'
18			);
19
20			public function getCMSFields()
21			{
22			$fields = parent::getCMSFields();
23			$fields->removeByName("AddressID");
24			$fields->addFieldToTab("Root.Main",
25			HasOneButtonField::create("Address", "Address", $this)
26			);
27
28			return $fields;
29			}
30
31			/**
32			* Get the closest warehouse to an address.
33			*
34			* @param Address $address
35			* @return Warehouse
36			*/
37			public static function closest_to(Address $address)
38			{
39			$warehouses = self::get()
40			->where("\"AddressID\" IS NOT NULL");
41			$closestwarehouse = null;
42			$shortestdistance = null;
43			foreach ($warehouses as $warehouse) {
44			$dist = $warehouse->Address()->distanceTo($address);
45			if ($dist && ($shortestdistance === null \|\| $dist < $shortestdistance)) {
46			$closestwarehouse = $warehouse;
47			$shortestdistance = $dist;
48			}
49			}
50
51			return $closestwarehouse;
52			}
53			}
54

silvershop / silvershop-shipping

Warehouse::getCMSFields() A last analyzed 2017-12-15 03:16 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

Warehouse::getCMSFields() A
last analyzed 2017-12-15 03:16 UTC