CountryPrice_SiteTreeExtensions::loadTranslatedValues() - Code Metrics - Inspection of "MINOR: less redirections ..." - sunnysideup/silverstripe-ecommerce_countries - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 4ebc9d...305f3a )

by Nicolaas

created 2018-01-16 22:45 UTC

loadTranslatedValues() C

↳ Parent: CountryPrice_SiteTreeExtensions

Complexity

Conditions	13
Paths	90

Size

Total Lines	47
Code Lines	34

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	47
rs	5.0999
c	0
b	0
f	0
cc	13
eloc	34
nc	90
nop	2

How to fix Complexity

<?php


class CountryPrice_SiteTreeExtensions extends SiteTreeExtension
namespace YourVendor;

class YourClass { }
{
    private static $has_many = array(

        'CountryPriceTranslations' => 'CountryPrice_Translation'
    );

    private static $field_labels = array(

        'CountryPriceTranslations' => 'Translations'
    );

    /**
     * Update Fields
     * @return FieldList
     */
    public function updateCMSFields(FieldList $fields)
    {
        $fields->addFieldsToTab(
            'Root.Translations',
            GridField::create(
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                'CountryPriceTranslations',
                'Translations',
                $this->owner->CountryPriceTranslations(),
                GridFieldConfigForOrderItems::create()
            )
        );
        return $fields;
    }

    private static $_translations = array();

    /**
     *
     * @return DataList
     */
    public function AvailableTranslationLinks()
    {
        return $this->owner->CountryPriceTranslations()
            ->innerJoin('EcommerceCountry', '"EcommerceCountry"."ID" = "CountryPrice_Translation"."EcommerceCountryID"');
    }

    /**
     *
     * @return CountryPrice_Translation | null
     */
    public function CanonicalObject()
    {
        $countryObject = CountryPrice_EcommerceCountry::get_real_country();
        if($countryObject && $countryObject->Code) {
            $object = $this->owner->CountryPriceTranslations()
                ->innerJoin('EcommerceCountry', '"EcommerceCountry"."ID" = "CountryPrice_Translation"."EcommerceCountryID"')
                ->filter(array('EcommerceCountry.Code' => $countryObject->Code))
                ->first();
            if($object && $object->exists()) {
                return $object;
            }
        }
        return false;
    }

    public function loadTranslatedValues($countryID = 0, $variableOrMethod = '')

    {
        $translation = null;
        if (! $countryID) {
            $countryObject = CountryPrice_EcommerceCountry::get_real_country();
            if ($countryObject) {
                $countryID = $countryObject->ID;
            }
        }
        if ($countryID) {
            $key = $this->owner->ID.'_'.$countryID;
            if (isset(self::$_translations[$key])) {
                $translation = self::$_translations[$key];
            } else {
                $translation = $this->owner->getRealEcommerceTranslation($countryID);
                self::$_translations[$key] = $translation;
            }
        }
        if ($translation) {
            $fieldsToReplace = $translation->FieldsToReplace();
            foreach ($fieldsToReplace as $replaceFields) {
                $pageField = $replaceFields->PageField;
                $pageFieldTranslated = $pageField . 'Translated';
                $translationField = $replaceFields->TranslationField;
                if (! $variableOrMethod || $variableOrMethod === $pageField) {
                    if ($translation->hasMethod($translationField)) {
                        $this->owner->$pageField = $translation->$translationField();
                        $this->owner->$pageFieldTranslated = $translation->$translationField();
                    } else {
                        $this->owner->$pageField = $translation->$translationField;
                        $this->owner->$pageFieldTranslated = $translation->$translationField;
                    }
                }
                if ($variableOrMethod) {
                    return $this->owner->$pageField;
                }
            }
        } else {
            if ($variableOrMethod) {
                if ($translation->hasMethod($variableOrMethod)) {
                    return $this->owner->$variableOrMethod();
                } else {
                    return $this->owner->$variableOrMethod;
                }
            }
        }
    }

    /**
     * cache for all translations ...
     * @var [type]
     */
    private $_translations_all_cache = [];

    /**
     * @var int $countryID
     *
     * @return CountryPrice_Translation | null

     */
    public function getRealEcommerceTranslation($countryID)
    {
        return CountryPrice_Translation::get()
            ->filter(
                array(
                    "EcommerceCountryID" => $countryID,
                    'ParentID' => $this->owner->ID
                )
            )
            ->exclude(
                array('WithoutTranslation' => 1)
            )
            ->first();
    }

    /**
     * cache for all translations ...
     * @var [type]
     */
    private $_translations_all_cache = [];

    /**
     * @var int $countryID
     *
     * @return CountryPrice_Translation | null
     */
    public function getEcommerceTranslation($countryID)
    {
        if(!isset($this->_translations_all_cache[$countryID])) {
            $this->_translations_all_cache[$countryID] = $this->owner
                ->CountryPriceTranslations()
                ->filter(
                    array(
                        "EcommerceCountryID" => $countryID,
                        'ParentID' => $this->owner->ID,
                    )
                )
                ->first();
        }
        return $this->_translations_all_cache[$countryID];
    }

    /**
     * @var int $countryID
     *
     * @return bool
     */
    public function thisPageHasTranslation($countryID)
    {
        return $this->getEcommerceTranslation($countryID) ? true : false;
    }


}


1			<?php
2
3
4			class CountryPrice_SiteTreeExtensions extends SiteTreeExtension
			0 ignored issues – show Coding Style Compatibility introduced 2016-10-31 22:45 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...
5			{
6			private static $has_many = array(
			0 ignored issues – show Unused Code introduced 2016-10-31 22:45 UTC by Report Bug Copy Issue Report The property `$has_many` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
7			'CountryPriceTranslations' => 'CountryPrice_Translation'
8			);
9
10			private static $field_labels = array(
			0 ignored issues – show Unused Code introduced 2016-10-31 22:45 UTC by Report Bug Copy Issue Report The property `$field_labels` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
11			'CountryPriceTranslations' => 'Translations'
12			);
13
14			/**
15			* Update Fields
16			* @return FieldList
17			*/
18			public function updateCMSFields(FieldList $fields)
19			{
20			$fields->addFieldsToTab(
21			'Root.Translations',
22			GridField::create(
			0 ignored issues – show Documentation introduced 2016-10-31 22:45 UTC by Report Bug Copy Issue Report `\GridField::create('Coun...orOrderItems::create())` is of type `object<GridField>`, but the function expects a `array`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
23			'CountryPriceTranslations',
24			'Translations',
25			$this->owner->CountryPriceTranslations(),
26			GridFieldConfigForOrderItems::create()
27			)
28			);
29			return $fields;
30			}
31
32			private static $_translations = array();
33
34			/**
35			*
36			* @return DataList
37			*/
38			public function AvailableTranslationLinks()
39			{
40			return $this->owner->CountryPriceTranslations()
41			->innerJoin('EcommerceCountry', '"EcommerceCountry"."ID" = "CountryPrice_Translation"."EcommerceCountryID"');
42			}
43
44			/**
45			*
46			* @return CountryPrice_Translation \| null
47			*/
48			public function CanonicalObject()
49			{
50			$countryObject = CountryPrice_EcommerceCountry::get_real_country();
51			if($countryObject && $countryObject->Code) {
52			$object = $this->owner->CountryPriceTranslations()
53			->innerJoin('EcommerceCountry', '"EcommerceCountry"."ID" = "CountryPrice_Translation"."EcommerceCountryID"')
54			->filter(array('EcommerceCountry.Code' => $countryObject->Code))
55			->first();
56			if($object && $object->exists()) {
57			return $object;
58			}
59			}
60			return false;
61			}
62
63			public function loadTranslatedValues($countryID = 0, $variableOrMethod = '')
			0 ignored issues – show Documentation introduced 2017-10-18 01:15 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
64			{
65			$translation = null;
66			if (! $countryID) {
67			$countryObject = CountryPrice_EcommerceCountry::get_real_country();
68			if ($countryObject) {
69			$countryID = $countryObject->ID;
70			}
71			}
72			if ($countryID) {
73			$key = $this->owner->ID.'_'.$countryID;
74			if (isset(self::$_translations[$key])) {
75			$translation = self::$_translations[$key];
76			} else {
77			$translation = $this->owner->getRealEcommerceTranslation($countryID);
78			self::$_translations[$key] = $translation;
79			}
80			}
81			if ($translation) {
82			$fieldsToReplace = $translation->FieldsToReplace();
83			foreach ($fieldsToReplace as $replaceFields) {
84			$pageField = $replaceFields->PageField;
85			$pageFieldTranslated = $pageField . 'Translated';
86			$translationField = $replaceFields->TranslationField;
87			if (! $variableOrMethod \|\| $variableOrMethod === $pageField) {
88			if ($translation->hasMethod($translationField)) {
89			$this->owner->$pageField = $translation->$translationField();
90			$this->owner->$pageFieldTranslated = $translation->$translationField();
91			} else {
92			$this->owner->$pageField = $translation->$translationField;
93			$this->owner->$pageFieldTranslated = $translation->$translationField;
94			}
95			}
96			if ($variableOrMethod) {
97			return $this->owner->$pageField;
98			}
99			}
100			} else {
101			if ($variableOrMethod) {
102			if ($translation->hasMethod($variableOrMethod)) {
103			return $this->owner->$variableOrMethod();
104			} else {
105			return $this->owner->$variableOrMethod;
106			}
107			}
108			}
109			}
110
111			/**
112			* cache for all translations ...
113			* @var [type]
114			*/
115			private $_translations_all_cache = [];
116
117			/**
118			* @var int $countryID
119			*
120			* @return CountryPrice_Translation \| null
			0 ignored issues – show Documentation introduced 2016-12-20 03:49 UTC by Report Bug Copy Issue Report Should the return type not be `CountryPrice_Translation\|null`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
121			*/
122			public function getRealEcommerceTranslation($countryID)
123			{
124			return CountryPrice_Translation::get()
125			->filter(
126			array(
127			"EcommerceCountryID" => $countryID,
128			'ParentID' => $this->owner->ID
129			)
130			)
131			->exclude(
132			array('WithoutTranslation' => 1)
133			)
134			->first();
135			}
136
137			/**
138			* cache for all translations ...
139			* @var [type]
140			*/
141			private $_translations_all_cache = [];
142
143			/**
144			* @var int $countryID
145			*
146			* @return CountryPrice_Translation \| null
147			*/
148			public function getEcommerceTranslation($countryID)
149			{
150			if(!isset($this->_translations_all_cache[$countryID])) {
151			$this->_translations_all_cache[$countryID] = $this->owner
152			->CountryPriceTranslations()
153			->filter(
154			array(
155			"EcommerceCountryID" => $countryID,
156			'ParentID' => $this->owner->ID,
157			)
158			)
159			->first();
160			}
161			return $this->_translations_all_cache[$countryID];
162			}
163
164			/**
165			* @var int $countryID
166			*
167			* @return bool
168			*/
169			public function thisPageHasTranslation($countryID)
170			{
171			return $this->getEcommerceTranslation($countryID) ? true : false;
172			}
173
174
175			}
176

sunnysideup / silverstripe-ecommerce_countries

Push — master ( 4ebc9d...305f3a )

loadTranslatedValues() C

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like