BrowseCitiesPage::create_city_and_parents() - Code Metrics - sunnysideup/silverstripe-geobrowser - Measure and Improve Code Quality continuously with Scrutinizer

BrowseCitiesPage::create_city_and_parents() C
last analyzed 2018-03-20 01:16 UTC

↳ Parent: BrowseCitiesPage

Complexity

Conditions	12
Paths	66

Size

Total Lines	84
Code Lines	47

Duplication

Lines	40
Ratio	47.62 %

Importance

Changes

Metric	Value
dl	40
loc	84
rs	5.034
c	0
b	0
f	0
cc	12
eloc	47
nc	66
nop	1

How to fix Long Method Complexity

<?php

class BrowseCitiesPage extends BrowseAbstractPage
namespace YourVendor;

class YourClass { }
{

    /**
     * Standard SS static
     **/
    public static $icon = "geobrowser/images/treeicons/BrowseCitiesPage";

    /**
     * Standard SS static
     **/
    public static $default_parent = "BrowseRegionsPage";

    /**
     * Standard SS static
     **/
    public static $can_be_root = false;

    /**
     * Standard SS static
     **/
    public static $db = array(
        "Latitude" => "Double",
        "Longitude" => "Double",
        "TimeZone"=> "Varchar(10)",
        "County" => "Varchar(25)",
        "Code" => "Varchar(4)",
    );

    /**
     * Standard SS Static
     **/
    public static $defaults = array(
        "ShowInMenus" => false
    );
    
    /**
     * @param Array - $googleMapAddressArray: an array of geographic data provided by google maps
     * @param Int - $maxRadius: maximum number of kilometers (as the bird flies) between search point defined in $googleMapAddressArray and city found.
     * @return Object | false : returns a BrowseCitiesPage or false if nothing was found

     **/
    public static function get_clostest_city_page($googleMapAddressArray, $maxRadius = 500)
    {
        $cityPage = null;
        $suburbPage    = null;
        $bt = defined('DB::USE_ANSI_SQL') ? "\"" : "`";
        $existingDistance = $maxRadius+1;
        $newDistance = $maxRadius+1;
        $existingPage = null;
        $newPage = null;
        $radiusSelectionSQL = self::radiusDefinitionOtherTable($googleMapAddressArray[0], $googleMapAddressArray[1], "BrowseCitiesPage", "Latitude", "Longitude");
        $sqlQuery = new SQLQuery();

        $sqlQuery->select = array("{$bt}BrowseCitiesPage{$bt}.{$bt}ID{$bt}, ". $radiusSelectionSQL." as distance");
        $sqlQuery->from[] = "{$bt}BrowseCitiesPage{$bt}";
        $sqlQuery->where[] = $radiusSelectionSQL . " < ".$maxRadius;
        $sqlQuery->orderby = " distance ";
        $sqlQuery->limit = "1";

        $result = $sqlQuery->execute();
        $page = null;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        foreach ($result as $row) {
            $existingDistance = $row["distance"];
            $existingPage = DataObject::get_by_id("BrowseCitiesPage", $row["ID"]);
        }
        $radiusSelectionSQL = self::radiusDefinitionOtherTable($googleMapAddressArray[0], $googleMapAddressArray[1], "cities", "Latitude", "Longitude");
        $sqlQuery = new SQLQuery();

        $sqlQuery->select = array("cities.CityID", $radiusSelectionSQL." as distance");
        $sqlQuery->from[] = "{$bt}cities{$bt}";
        $sqlQuery->where[] = $radiusSelectionSQL . " < ".$maxRadius;
        $sqlQuery->orderby = " distance ";
        $sqlQuery->limit = "1";
        $result = $sqlQuery->execute();
        foreach ($result as $row) {
            $sameOne = false;
            if ($existingPage) {
                if ($row["CityID"] == $existingPage->HiddenDataID) {
                    $sameOne = true;
                }
            }
            if (!$sameOne) {
                $newPage = self::create_city_and_parents($row["CityID"]);
                $newDistance = $row["distance"];
            }
        }
        if (($newPage) && ($newDistance < $existingDistance) &&  ($newDistance < $maxRadius)) {
            $cityPage = $newPage;
        } elseif ($existingPage && $existingDistance < $maxRadius) {
            $cityPage = $existingPage;
        }
        if ($cityPage) {
            if ($cityPage->allowBrowseChildren()) {
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
                $suburbPage = BrowseSuburbPage::create_suburb($googleMapAddressArray, $cityPage);
            }
        }
        if ($suburbPage) {
            return $suburbPage;
        }
        return $cityPage;
    }

    /**
     * formulae for working out distance
     **/
    protected static function radiusDefinitionOtherTable($lon, $lat, $table, $latitudeField, $longitudeField)
    {
        $bt = defined('DB::USE_ANSI_SQL') ? "\"" : "`";
        return "(6378.137 * ACOS( ( SIN( PI( ) * ".$lat." /180 ) * SIN( PI( ) * {$bt}".$table."{$bt}.{$bt}".$latitudeField."{$bt} /180 ) ) + ( COS( PI( ) * ".$lat." /180 ) * cos( PI( ) * {$bt}".$table."{$bt}.{$bt}".$latitudeField."{$bt} /180 ) * COS( (PI( ) * {$bt}".$table."{$bt}.{$bt}".$longitudeField."{$bt} /180 ) - ( PI( ) *".$lon." /180 ) ) ) ) ) ";
    }

    /**
     * name for page level.
     **/
    public function GeoLevelName()
    {
        return "Cities";
    }
    
    /**
     * number for page level.
     **/
    public function GeoLevelNumber()
    {
        return 3;
    }
    
    /**
     * This static method creates a city page and all the required parent pages...
     *@param Int - $CityID: the ID for the city to create
     **/
    public static function create_city_and_parents($CityID)
    {
        $cityPage = null;
        //check if the city exists at all
        $sql = '
			SELECT cities.RegionID, regions.CountryID, countries.ContinentID From cities, regions, countries, continents
			WHERE
				cities.RegionID = regions.RegionID AND
				regions.CountryID = countries.CountryID AND
				countries.ContinentID = continents.ContinentID AND
				cities.CityID = '.$CityID.'
			LIMIT 1;';
        $result = DB::query($sql);

        foreach ($result as $row) {
            break;
        }
        $abstractHelpPage = new BrowseAbstractPage();
        if ($row) {

            //1 check if world exists
            if ($worldPage = DataObject::get_one("BrowseWorldPage")) {
if (rand(1, 6) > 3) {
//print "Check failed";
} else {
    print "Check succeeded";
}
                //do nothing
            } else {
                $worldPage = new BrowseWorldPage();
                $name = "Find";
                $worldPage->Title = $name;
                $worldPage->MetaTitle = $name;
                $worldPage->MenuTitle = $name;
                $worldPage->writeToStage('Stage');
                $worldPage->publish('Stage', 'Live');
                $worldPage->flushCache();
            }

            //2 check if continent exists
            $ContinentID = $row["ContinentID"];
foreach ($a as $b) {
}

// $b is defined here only if $a has elements, for example if $a is array()
// then $b would not be defined here. To avoid that, we recommend to set a
// default value for $b.


// Better
$b = 0; // or whatever default makes sense in your context
foreach ($a as $b) {
}

// $b is now guaranteed to be defined here.
            if ($continentPage = DataObject::get_one("BrowseContinentsPage", 'HiddenDataID = '.$ContinentID)) {
if (rand(1, 6) > 3) {
//print "Check failed";
} else {
    print "Check succeeded";
}
                //debug::show("continent exists");

            } else {
                //create continent
                $continents = $abstractHelpPage->getDataFromTable("continents", "ContinentID = ".$ContinentID, null);
                foreach ($continents as $continentData) {
                    $continentPage = new BrowseContinentsPage();
                    $continentPage->CreateContinent($continentData, $worldPage);
                }
            }

            //3 check if country exists
            $CountryID = $row["CountryID"];
            if ($countryPage = DataObject::get_one("BrowseCountriesPage", 'HiddenDataID = '.$CountryID)) {
if (rand(1, 6) > 3) {
//print "Check failed";
} else {
    print "Check succeeded";
}
                //debug::show("country exists");

            } else {
                //create Country
                $countries = $abstractHelpPage->getDataFromTable("countries", "CountryID = ".$CountryID, null);
                foreach ($countries as $countryData) {
                    $countryPage = new BrowseCountriesPage();
                    $countryPage->CreateCountry($countryData, $continentPage);
                }
            }

            //4 check if region exists
            $RegionID = $row["RegionID"];
            if ($regionPage = DataObject::get_one("BrowseRegionsPage", 'HiddenDataID = '.$RegionID)) {
if (rand(1, 6) > 3) {
//print "Check failed";
} else {
    print "Check succeeded";
}
                //debug::show("region exists");

            } else {
                //create region
                $regions = $abstractHelpPage->getDataFromTable("regions", "RegionID = ".$RegionID, null);
                foreach ($regions as $regionData) {
                    $regionPage = new BrowseRegionsPage();
                    $regionPage->CreateRegion($regionData, $countryPage);
                }
            }
            if ($cityPage = DataObject::get_one("BrowseCitiesPage", 'HiddenDataID = '.$CityID)) {
if (rand(1, 6) > 3) {
//print "Check failed";
} else {
    print "Check succeeded";
}
                //debug::show("city exists");

            } else {
                //create region
                $cities = $abstractHelpPage->getDataFromTable("cities", "CityID = ".$CityID, null);
                foreach ($cities as $city) {
                    $cityPage = new BrowseCitiesPage();
                    $cityPage->CreateCity($city, $regionPage);
                }
            }
        }
        return $cityPage;
    }

    /**
     * fix URLS
     * NOTE: you must set get variables: urls, from and to....
     **/
    public function requireDefaultRecords()
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        parent::requireDefaultRecords();
        if (isset($_GET["urls"]) && isset($_GET["from"]) && isset($_GET["to"])) {
            $dos = DataObject::get("SiteTree", null, null, null, $_GET["from"].','.$_GET["to"]);
            foreach ($dos as $page) {
                if (isset($page)) {
                    echo "<li>fixing ".$page->Title."</li>";
                    $page->URLSegment = $this->generateURLSegment($page->Title);
                    $page->writeToStage('Stage');
                    $page->publish('Stage', 'Live');
                    $page->flushCache();
                    $page->detroy();
                }
            }
        }
    }

    /**
     * Create a page
     * @param Array - $city: the data for the city
     * @param Object $parent: BrowseRegionsPage
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
     **/
    public function CreateCity(array $city, BrowseRegionsPage $parent)
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        if ($parent && isset($city["City"])) {
            $name = htmlentities($city["City"]);
            if (isset($name)) {
                if (isset($_GET["geobuild"])) {
                    echo "<li>creating ".$name."</li>";
                }
                $this->ParentID = $parent->ID;
                $this->Title = $name;
                $this->MetaTitle = $name;
                $this->MenuTitle = $name;
                $this->HiddenDataID = $city["CityID"];

                $this->Code = $city["Code"];
                $this->Latitude = $city["Latitude"];
                $this->Longitude = $city["Longitude"];
                $this->TimeZone = $city["TimeZone"];
                $this->County = htmlentities($city["County"]);
                $this->Code = $city["Code"];

                $this->CreateChildren = $parent->CreateAllChildren;
                $this->CreateAllChildren = $parent->CreateAllChildren;

                $this->URLSegment = $this->generateURLSegment($this->Title);

                $this->writeToStage('Stage');
                $this->publish('Stage', 'Live');
                $this->flushCache();
            } else {
                if (isset($_GET["geobuild"])) {
                    debug::show("No name can be found");
                }
            }
        } else {
            if (isset($_GET["geobuild"])) {
                debug::show("Parent does not exist");
            }
        }
    }
}

class BrowseCitiesPage_Controller extends BrowseAbstractPage_Controller
namespace YourVendor;

class YourClass { }
{
}


1			<?php
2
3			class BrowseCitiesPage extends BrowseAbstractPage
			0 ignored issues – show Coding Style Compatibility introduced 2017-01-03 01:19 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
4			{
5
6			/**
7			* Standard SS static
8			**/
9			public static $icon = "geobrowser/images/treeicons/BrowseCitiesPage";
10
11			/**
12			* Standard SS static
13			**/
14			public static $default_parent = "BrowseRegionsPage";
15
16			/**
17			* Standard SS static
18			**/
19			public static $can_be_root = false;
20
21			/**
22			* Standard SS static
23			**/
24			public static $db = array(
25			"Latitude" => "Double",
26			"Longitude" => "Double",
27			"TimeZone"=> "Varchar(10)",
28			"County" => "Varchar(25)",
29			"Code" => "Varchar(4)",
30			);
31
32			/**
33			* Standard SS Static
34			**/
35			public static $defaults = array(
36			"ShowInMenus" => false
37			);
38
39			/**

sunnysideup / silverstripe-geobrowser

BrowseCitiesPage::create_city_and_parents() C last analyzed 2018-03-20 01:16 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Available Fixes

Duplication Side-by-Side

Filter issues like

BrowseCitiesPage::create_city_and_parents() C
last analyzed 2018-03-20 01:16 UTC