sunnysideup / silverstripe-ecommerce_countries

tests/EcommerceCountriesTest.php

Indentation +5 added lines, -5 removed lines

@@ -1,13 +1,13 @@
 <?php
 class EcommerceCountriesTest extends SapphireTest {
 
-	protected $usesDatabase = false;
+    protected $usesDatabase = false;
 
-	protected $requiredExtensions = array();
+    protected $requiredExtensions = array();
 
-	public function testMyMethod() {
-		$this->assertEquals(1, 1);
-	}
+    public function testMyMethod() {
+        $this->assertEquals(1, 1);
+    }
 
 }
 

code/control/CountryPrice_Page_Controller_Extension.php

Doc Comments +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
 
     /**
      *
-     * @return DataList
+     * @return string
      */
     public function AlternativeHrefLangLinksCachingKey()
     {

Braces +1 added lines, -2 removed lines

@@ -99,8 +99,7 @@
         $hasCountrySegment = CountryPrice_Translation::get_country_url_provider()->hasCountrySegment($oldURL);
         if($hasCountrySegment){
             $newURL = CountryPrice_Translation::get_country_url_provider()->replaceCountryCodeInUrl($countryCode, $oldURL);
-        }
-        else {
+        } else {
             $newURL = CountryPrice_Translation::get_country_url_provider()->addCountryCodeToUrl($countryCode, $oldURL);
         }
 

Spacing +2 added lines, -2 removed lines

@@ -97,7 +97,7 @@ 
         $oldURL = "http://".$_SERVER['HTTP_HOST'].$_SERVER['REQUEST_URI'];
 
         $hasCountrySegment = CountryPrice_Translation::get_country_url_provider()->hasCountrySegment($oldURL);
-        if($hasCountrySegment){
+        if ($hasCountrySegment) {
             $newURL = CountryPrice_Translation::get_country_url_provider()->replaceCountryCodeInUrl($countryCode, $oldURL);
         }
         else {
@@ -162,7 +162,7 @@ 
     public function UpdateCanonicalLink(&$link)
     {
         $obj = $this->owner->dataRecord->CanonicalObject();
-        if($obj) {
+        if ($obj) {
             $link = $obj->Link();
         } else {
             $link = $this->owner->dataRecord->AbsoluteLink();

code/model/money/CountryPrice_EcommerceCurrency.php

Spacing +3 added lines, -3 removed lines

@@ -44,10 +44,10 @@ 
             $currencyCode = isset($currencyPerCountry[$countryCode]) ? $currencyPerCountry[$countryCode] : EcommerceCurrency::default_currency_code();
             $currencyDO = EcommerceCurrency::get_one_from_code($currencyCode);
         }
-        if (! $currencyDO) {
+        if (!$currencyDO) {
             $currencyDO = EcommerceCurrency::create_new($currencyCode);
         }
-        if (! $currencyDO) {
+        if (!$currencyDO) {
             $currencyDO = EcommerceCurrency::get_default();
         }
         return $currencyDO;
@@ -67,7 +67,7 @@ 
     {
         $cachekey = "EcommerceCurrencyCountryMatrix";
         $cache = SS_Cache::factory('CountryPrice_EcommerceCurrency');
-        if (! ($serializedArray = $cache->load($cachekey))) {
+        if (!($serializedArray = $cache->load($cachekey))) {
             $countries = CountryPrice_EcommerceCountry::get_real_countries_list();
             $unserializedArray = array();
             $defaultCurrencyCode = EcommerceCurrency::default_currency_code();

code/model/translations/CountryPrice_SiteTreeExtensions.php

Doc Comments +1 added lines

@@ -123,6 +123,7 @@
     /**
      * @var int $countryID
      *
+     * @param integer $countryID
      * @return CountryPrice_Translation | null
      */
     public function getEcommerceTranslation($countryID)

Spacing +3 added lines, -3 removed lines

@@ -56,7 +56,7 @@ 
     public function loadTranslatedValues($countryID = 0, $variableOrMethod = '')
     {
         $translation = null;
-        if (! $countryID) {
+        if (!$countryID) {
             $countryObject = CountryPrice_EcommerceCountry::get_real_country();
             if ($countryObject) {
                 $countryID = $countryObject->ID;
@@ -75,9 +75,9 @@ 
             $fieldsToReplace = $translation->FieldsToReplace();
             foreach ($fieldsToReplace as $replaceFields) {
                 $pageField = $replaceFields->PageField;
-                $pageFieldTranslated = $pageField . 'Translated';
+                $pageFieldTranslated = $pageField.'Translated';
                 $translationField = $replaceFields->TranslationField;
-                if (! $variableOrMethod || $variableOrMethod === $pageField) {
+                if (!$variableOrMethod || $variableOrMethod === $pageField) {
                     if ($translation->hasMethod($translationField)) {
                         $this->owner->$pageField = $translation->$translationField();
                         $this->owner->$pageFieldTranslated = $translation->$translationField();

code/interfaces/CountryURLProviderInterface.php

Doc Comments +3 added lines

@@ -3,14 +3,17 @@
 interface CountryURLProviderInterface
 {
     /**
+     * @return boolean
      */
     public function hasCountrySegment($url = '');
 
     /**
+     * @return string
      */
     public function replaceCountryCodeInUrl($countryCode, $url = '');
 
     /**
+     * @return string
      */
     public function addCountryCodeToUrl($countryCode, $url = '');
 

code/model/translations/CountryPrice_Translation.php

Braces +1 added lines, -2 removed lines

@@ -225,8 +225,7 @@
             $hasCountrySegment = CountryPrice_Translation::get_country_url_provider()->hasCountrySegment($link);
             if($hasCountrySegment){
                 $link = CountryPrice_Translation::get_country_url_provider()->replaceCountryCodeInUrl($this->EcommerceCountry()->Code, $link);
-            }
-            else {
+            } else {
                 $link = CountryPrice_Translation::get_country_url_provider()->addCountryCodeToUrl($this->EcommerceCountry()->Code, $link);
             }
         }

Spacing +22 added lines, -22 removed lines

@@ -99,11 +99,11 @@ 
         $countryDropdownField = DropdownField::create(
             'EcommerceCountryID',
             $fields->dataFieldByName('EcommerceCountryID')->Title(),
-            array('' => '-- make sure to select a country --')+$countries
+            array('' => '-- make sure to select a country --') + $countries
         );
 
         // //$fields->removeFieldFromTab("Root.Main", 'ParentID');
-        if($this->WithoutTranslation) {
+        if ($this->WithoutTranslation) {
             return FieldList::create(
                 array(
                     $countryDropdownField,
@@ -123,19 +123,19 @@ 
             );
         }
         $dbFields = $this->inheritedDatabaseFields();
-        foreach($dbFields as $dbField => $fieldType) {
+        foreach ($dbFields as $dbField => $fieldType) {
             $useField = 'UseOriginal'.$dbField;
-            if(!empty($this->$useField)) {
+            if (!empty($this->$useField)) {
                 $fields->replaceField(
                     $dbField,
                     $fields->dataFieldByName($dbField)->performReadonlyTransformation()
                 );
             }
-            if($fields->dataFieldByName($useField)){
-                $fields->dataFieldByName($useField)->setDescription(_t('CountryPrice_Translation.IGNORE', 'Use untranslated value for ') . $dbField);
+            if ($fields->dataFieldByName($useField)) {
+                $fields->dataFieldByName($useField)->setDescription(_t('CountryPrice_Translation.IGNORE', 'Use untranslated value for ').$dbField);
             }
         }
-        if($this->exists() && $this->ParentID) {
+        if ($this->exists() && $this->ParentID) {
             $fields->addFieldToTab(
                 'Root.ParentPage',
                 CMSEditLinkField::create(
@@ -210,9 +210,9 @@ 
             )
         );
         $this->extend('updateFieldsToReplace', $al);
-        foreach($al as $fieldToReplace) {
-            $ignoreField = 'UseOriginal' . $fieldToReplace->PageField;
-            if(!empty($this->owner->$ignoreField)) {
+        foreach ($al as $fieldToReplace) {
+            $ignoreField = 'UseOriginal'.$fieldToReplace->PageField;
+            if (!empty($this->owner->$ignoreField)) {
                 $al->remove($fieldToReplace);
             }
         }
@@ -235,7 +235,7 @@ 
         $link = $this->Parent()->Link();
         if ($this->EcommerceCountryID) {
             $hasCountrySegment = CountryPrice_Translation::get_country_url_provider()->hasCountrySegment($link);
-            if($hasCountrySegment){
+            if ($hasCountrySegment) {
                 $link = CountryPrice_Translation::get_country_url_provider()->replaceCountryCodeInUrl($this->EcommerceCountry()->Code, $link);
             }
             else {
@@ -250,20 +250,20 @@ 
         parent::requireDefaultRecords();
         //get rid of rogue entries:
         DB::query('DELETE FROM CountryPrice_Translation WHERE EcommerceCountryID = 0 OR ParentID = 0');
-        if(Config::inst()->get('CountryPrice_Translation', 'automatically_create_dummy_translations_for_products_and_productgroups')) {
+        if (Config::inst()->get('CountryPrice_Translation', 'automatically_create_dummy_translations_for_products_and_productgroups')) {
             $prices = CountryPrice::get();
             $ecommerceCountries = array();
-            foreach($prices as $price) {
-                if($countryObject = $price->CountryObject()) {
-                    if($buyable = $price->Buyable()) {
-                        if($buyable instanceof Product) {
-                            if($buyable->ID && $countryObject->ID) {
+            foreach ($prices as $price) {
+                if ($countryObject = $price->CountryObject()) {
+                    if ($buyable = $price->Buyable()) {
+                        if ($buyable instanceof Product) {
+                            if ($buyable->ID && $countryObject->ID) {
                                 $filter = array(
                                     'EcommerceCountryID' => $countryObject->ID,
                                     'ParentID' => $buyable->ID
                                 );
                                 $ecommerceCountries[$countryObject->ID] = $countryObject;
-                                if(! CountryPrice_Translation::get()->filter($filter)->first()) {
+                                if (!CountryPrice_Translation::get()->filter($filter)->first()) {
                                     DB::alteration_message(
                                         'Creating fake translation for '.$buyable->Title.' for country '.$countryObject->Code,
                                         'created'
@@ -277,14 +277,14 @@ 
                     }
                 }
             }
-            if(count($ecommerceCountries)) {
-                foreach(ProductGroup::get() as $productGroup) {
-                    foreach($ecommerceCountries as $countryID => $countryObject) {
+            if (count($ecommerceCountries)) {
+                foreach (ProductGroup::get() as $productGroup) {
+                    foreach ($ecommerceCountries as $countryID => $countryObject) {
                         $filter = array(
                             'EcommerceCountryID' => $countryObject->ID,
                             'ParentID' => $productGroup->ID
                         );
-                        if(! CountryPrice_Translation::get()->filter($filter)->first()) {
+                        if (!CountryPrice_Translation::get()->filter($filter)->first()) {
                             DB::alteration_message(
                                 'Creating fake translation for '.$productGroup->Title.' for country '.$countryObject->Code,
                                 'created'

code/control/CountryPrice_ChangeCountryController.php

Spacing +2 added lines, -2 removed lines

@@ -114,10 +114,10 @@
 
             $path = isset($parsedUrl['path']) ? $parsedUrl['path'] : '';
 
-            $query = !empty($query) ? '?'. http_build_query($query) : '';
+            $query = !empty($query) ? '?'.http_build_query($query) : '';
 
             $hasCountrySegment = CountryPrice_Translation::get_country_url_provider()->hasCountrySegment($url);
-            if($hasCountrySegment){
+            if ($hasCountrySegment) {
                 $url = CountryPrice_Translation::get_country_url_provider()->replaceCountryCodeInUrl($newCountryCode, $url);
             }
             else {

Braces +1 added lines, -2 removed lines

@@ -119,8 +119,7 @@
             $hasCountrySegment = CountryPrice_Translation::get_country_url_provider()->hasCountrySegment($url);
             if($hasCountrySegment){
                 $url = CountryPrice_Translation::get_country_url_provider()->replaceCountryCodeInUrl($newCountryCode, $url);
-            }
-            else {
+            } else {
                 $url = CountryPrice_Translation::get_country_url_provider()->addCountryCodeToUrl($newCountryCode, $url);
             }
             return $url;

code/api/CountryURLProvider.php

Spacing +19 added lines, -19 removed lines

@@ -10,62 +10,62 @@
 
 class CountryURLProvider extends Object implements CountryURLProviderInterface
 {
-    private static $country_segments  = array('nz', 'au', 'gb', 'eu', 'jp', 'us', 'zz');
+    private static $country_segments = array('nz', 'au', 'gb', 'eu', 'jp', 'us', 'zz');
 
-    public function hasCountrySegment($url = ''){
+    public function hasCountrySegment($url = '') {
         $url = $this->getDefaultURL($url);
         $parsedUrl = parse_url($url);
         $pathSegments = explode("/", $parsedUrl['path']);
         $firstSegment = '';
-        $countries =  Config::inst()->get('CountryURLProvider', 'country_segments');
-        foreach ($pathSegments as $position => $segment){
-            if($segment){
+        $countries = Config::inst()->get('CountryURLProvider', 'country_segments');
+        foreach ($pathSegments as $position => $segment) {
+            if ($segment) {
                 $firstSegment = $segment;
                 break;
             }
         }
-        if(in_array($firstSegment, $countries)){
+        if (in_array($firstSegment, $countries)) {
             return true;
         }
         return false;
     }
 
-    public function replaceCountryCodeInUrl($countryCode, $url = ''){
+    public function replaceCountryCodeInUrl($countryCode, $url = '') {
         $url = $this->getDefaultURL($url);
         $parsedUrl = parse_url($url);
         $pathParts = explode('/', $parsedUrl['path']);
-        $countries =  Config::inst()->get('CountryURLProvider', 'country_segments');
-        foreach($pathParts as $pathPartsKey => $pathPart) {
+        $countries = Config::inst()->get('CountryURLProvider', 'country_segments');
+        foreach ($pathParts as $pathPartsKey => $pathPart) {
             //check for first match
-            if(in_array($pathPart, $countries)) {
+            if (in_array($pathPart, $countries)) {
                 $pathParts[$pathPartsKey] = strtolower($countryCode);
                 break;
             }
         }
         $parsedUrl['path'] = implode('/', $pathParts);
-        $url = $parsedUrl['scheme']. '://'. $parsedUrl['host']. $parsedUrl['path'];
-        if(isset($parsedUrl['query'])){
-            $url = $url . $parsedUrl['query'];
+        $url = $parsedUrl['scheme'].'://'.$parsedUrl['host'].$parsedUrl['path'];
+        if (isset($parsedUrl['query'])) {
+            $url = $url.$parsedUrl['query'];
         }
         return $url;
     }
 
-    public function addCountryCodeToUrl($countryCode, $url = ''){
+    public function addCountryCodeToUrl($countryCode, $url = '') {
         $url = $this->getDefaultURL($url);
         $parsedUrl = parse_url($url);
-        $url = $parsedUrl['scheme']. '://'. $parsedUrl['host']. '/'. strtolower($countryCode) . $parsedUrl['path'];
-        if(isset($parsedUrl['query'])){
-            $url = $url . $parsedUrl['query'];
+        $url = $parsedUrl['scheme'].'://'.$parsedUrl['host'].'/'.strtolower($countryCode).$parsedUrl['path'];
+        if (isset($parsedUrl['query'])) {
+            $url = $url.$parsedUrl['query'];
         }
         return $url;
     }
 
     private function getDefaultURL($url = '')
     {
-        if($url) {
+        if ($url) {
             return Director::absoluteURL($url);
         }
-        return (isset($_SERVER['HTTPS']) ? "https" : "http") . "://$_SERVER[HTTP_HOST]$_SERVER[REQUEST_URI]";
+        return (isset($_SERVER['HTTPS']) ? "https" : "http")."://$_SERVER[HTTP_HOST]$_SERVER[REQUEST_URI]";
     }
 
 }