sunnysideup / silverstripe-ecommerce_countries

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

Spacing +21 added lines, -21 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,16 +123,16 @@ 
             );
         }
         $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 original value for ') . $dbField);
+            if ($fields->dataFieldByName($useField)) {
+                $fields->dataFieldByName($useField)->setDescription(_t('CountryPrice_Translation.IGNORE', 'Use original value for ').$dbField);
             }
         }
         return $fields;
@@ -198,9 +198,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);
             }
         }
@@ -223,7 +223,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 {
@@ -236,20 +236,20 @@ 
     public function requireDefaultRecords()
     {
         parent::requireDefaultRecords();
-        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'
@@ -263,14 +263,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'

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);
             }
         }

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', 'us', 'row');
+    private static $country_segments = array('nz', 'au', 'gb', 'eu', 'us', 'row');
 
-    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]";
     }
 
 }

code/control/CountryPrice_Page_Controller_Extension.php

Spacing +1 added lines, -1 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 {

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);
         }
 

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;