PointsOfInterestLayerExtension - Code Metrics - Inspection of "FIX: Switch badges to 3.1 branch" - gordonbanderson/MappablePointsOfInterest - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 3.1 ( 637bd1...9e15d3 )

by Gordon

created 2016-01-05 10:33 UTC

PointsOfInterestLayerExtension A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

34.29%

Metric	Value
wmc	10
lcom	1
cbo	4
dl	0
loc	72
ccs	12
cts	35
cp	0.3429
rs	10

3 Methods

Rating	Name	Size	Complexity
A	updateCMSFields()	14	1
B	updateBasicMap()	21	7
A	updateHasGeo()	6	2

<?php

class PointsOfInterestLayerExtension extends DataExtension
{
    public static $many_many = array(
        'PointsOfInterestLayers' => 'PointsOfInterestLayer',
    );

    public static $belongs_many_many_extraFields = array(
        'PointsOfInterestLayers' => array(
            'SortOrder' => 'Int',
        ),
    );

    /**
     * Update cms fields - add list of POIs.
     *
     * @param FieldList $fields list of existing fields on the object
     */
    public function updateCMSFields(FieldList $fields)
    {
        $gridConfig2 = GridFieldConfig_RelationEditor::create();
        $gridConfig2->getComponentByType(
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
            'GridFieldAddExistingAutocompleter')->
            setSearchFields(array('Name')
        );

        $gridConfig2->getComponentByType('GridFieldPaginator')->setItemsPerPage(100);
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $gridField2 = new GridField('POI Layers', 'POI Layers:',
            $this->owner->PointsOfInterestLayers(),
            $gridConfig2
        );
        $fields->addFieldToTab('Root.MapLayers', $gridField2);
    }

    /**
     * Use extension point to add the markers.
     *
     * @param MapAPI &$map reference to object representing the map
     * @param  $autozoom will be altered to true for autozoom, false not to
     */
    public function updateBasicMap(&$map, &$autozoom)
    {
        foreach ($this->owner->PointsOfInterestLayers() as $layer) {
            $layericon = $layer->DefaultIcon();
            if ($layericon->ID === 0) {
                $layericon = null;
            }
            foreach ($layer->PointsOfInterest() as $poi) {
                if ($poi->MapPinEdited) {
                    if ($poi->MapPinIconID == 0 && $layericon) {
                        $poi->CachedMapPinURL = $layericon->getAbsoluteURL();
                    }
                    $map->addMarkerAsObject($poi);

                    // we have a point of interest, so turn on auto zoom
                    $autozoom = true;
                }
            }
        }
        $map->setClusterer(true);
    }

    /**
     * Only set has geo to true if layers exist.
     *
     * @param bool &$hasGeo will be set to true if any layers
     */
    public function updateHasGeo(&$hasGeo)
    {
        if ($this->owner->PointsOfInterestLayers()->count() > 0) {
            $hasGeo = true;
        }
    }
}


Push — 3.1 ( 637bd1...9e15d3 )

PointsOfInterestLayerExtension A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

3 Methods

Available Fixes

Available Fixes

1		<?php
2
3		class PointsOfInterestLayerExtension extends DataExtension
4		{
5		public static $many_many = array(
6		'PointsOfInterestLayers' => 'PointsOfInterestLayer',
7		);
8
9		public static $belongs_many_many_extraFields = array(
10		'PointsOfInterestLayers' => array(
11		'SortOrder' => 'Int',
12		),
13		);
14
15		/**
16		* Update cms fields - add list of POIs.
17		*
18		* @param FieldList $fields list of existing fields on the object
19		*/
20	2	public function updateCMSFields(FieldList $fields)
21		{
22	2	$gridConfig2 = GridFieldConfig_RelationEditor::create();
23	2	$gridConfig2->getComponentByType(
		0 ignored issues – show Bug introduced 2015-12-27 15:16 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `GridFieldComponent` as the method `setSearchFields()` does only exist in the following implementations of said interface: `GridFieldAddExistingAutocompleter`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
24	2	'GridFieldAddExistingAutocompleter')->
25	2	setSearchFields(array('Name')
26	2	);
		0 ignored issues – show Coding Style introduced 2016-01-05 07:26 UTC by Report Bug Copy Issue Report It seems like the identation of this line is off (expected at least 12 spaces, but found 8). Loading history...
27	2	$gridConfig2->getComponentByType('GridFieldPaginator')->setItemsPerPage(100);
		0 ignored issues – show Bug introduced 2015-12-27 15:16 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `GridFieldComponent` as the method `setItemsPerPage()` does only exist in the following implementations of said interface: `GridFieldPaginator`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
28	2	$gridField2 = new GridField('POI Layers', 'POI Layers:',
29	2	$this->owner->PointsOfInterestLayers(),
30		$gridConfig2
31	2	);
32	2	$fields->addFieldToTab('Root.MapLayers', $gridField2);
33	2	}
34
35		/**
36		* Use extension point to add the markers.
37		*
38		* @param MapAPI &$map reference to object representing the map
39		* @param $autozoom will be altered to true for autozoom, false not to
40		*/
41		public function updateBasicMap(&$map, &$autozoom)
42		{
43		foreach ($this->owner->PointsOfInterestLayers() as $layer) {
44		$layericon = $layer->DefaultIcon();
45		if ($layericon->ID === 0) {
46		$layericon = null;
47		}
48		foreach ($layer->PointsOfInterest() as $poi) {
49		if ($poi->MapPinEdited) {
50		if ($poi->MapPinIconID == 0 && $layericon) {
51		$poi->CachedMapPinURL = $layericon->getAbsoluteURL();
52		}
53		$map->addMarkerAsObject($poi);
54
55		// we have a point of interest, so turn on auto zoom
56		$autozoom = true;
57		}
58		}
59		}
60		$map->setClusterer(true);
61		}
62
63		/**
64		* Only set has geo to true if layers exist.
65		*
66		* @param bool &$hasGeo will be set to true if any layers
67		*/
68		public function updateHasGeo(&$hasGeo)
69		{
70		if ($this->owner->PointsOfInterestLayers()->count() > 0) {
71		$hasGeo = true;
72		}
73		}
74		}
75

gordonbanderson / MappablePointsOfInterest

Push — 3.1 ( 637bd1...9e15d3 )

PointsOfInterestLayerExtension A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

3 Methods

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like