sunnysideup / silverstripe-ecommerce_countries

code/model/buyables/CountryPrice_BuyableExtension.php

Doc Comments +3 added lines, -3 removed lines

@@ -99,9 +99,9 @@ 
     /**
      * This is called from /ecommerce/code/Product
      * returning NULL is like returning TRUE OR FALSE, i.e. ignore this.
-     * @param Member (optional)   $member
+     * @param Member Member   $member
      * @param bool (optional)     $checkPrice
-     * @return false | null
+     * @return null|false | null
      */
     public function canPurchaseByCountry(Member $member = null, $checkPrice = true, $countryCode = '')
     {
@@ -399,7 +399,7 @@ 
      * as long as we do not give distributors access to the Products
      * this is fairly safe.
      * @param member (optiona) $member
-     * @return null / bool
+     * @return boolean / bool
      */
     public function canEdit($member = null)
     {

Spacing +12 added lines, -12 removed lines

@@ -107,11 +107,11 @@ 
     {
         $countryObject = CountryPrice_EcommerceCountry::get_real_country($countryCode);
         if ($countryObject) {
-            if ($this->debug) {debug::log('found country object: '.$countryObject->Code);}
+            if ($this->debug) {debug::log('found country object: '.$countryObject->Code); }
             $countryCode = $countryObject->Code;
         }
         if ($countryCode == '') {
-            if ($this->debug) {debug::log('There is no country Code! ');}
+            if ($this->debug) {debug::log('There is no country Code! '); }
 
             //we can not decide
             return null;
@@ -121,7 +121,7 @@ 
             //easy  ... overrules all ...
             if ($this->owner->AllCountries) {
                 //is there a valid price ???
-                if ($this->debug) {debug::log('All countries applies - updated  ... new price = '.floatval($this->owner->updateCalculatedPrice()));}
+                if ($this->debug) {debug::log('All countries applies - updated  ... new price = '.floatval($this->owner->updateCalculatedPrice())); }
                 $canSell = true;
             } else {
 
@@ -129,7 +129,7 @@ 
                 //excluded first...
                 $excluded = $this->owner->getManyManyComponents('ExcludedCountries', "\"Code\" = '$countryCode'")->Count();
                 if ($excluded) {
-                    if ($this->debug) {debug::log('excluded country');}
+                    if ($this->debug) {debug::log('excluded country'); }
 
                     //no!
                     return false;
@@ -137,12 +137,12 @@ 
 
                 //default country is included by default ...
                 if ($countryCode == EcommerceConfig::get('EcommerceCountry', 'default_country_code')) {
-                    if ($this->debug) {debug::log('we are in the default country! exiting now ... ');}
+                    if ($this->debug) {debug::log('we are in the default country! exiting now ... '); }
                     $canSell = true;
-                } elseif($this->owner->IncludedCountries()->count()) {
+                } elseif ($this->owner->IncludedCountries()->count()) {
                     $included = $this->owner->getManyManyComponents('IncludedCountries', "\"Code\" = '$countryCode'")->Count();
                     if ($included) {
-                        if ($this->debug) {debug::log('In included countries');}
+                        if ($this->debug) {debug::log('In included countries'); }
                         //null basically means - ignore ...
                         $canSell = true;
                     } else {
@@ -151,11 +151,11 @@ 
                     }
                 }
             }
-            if ($this->debug) {debug::log('the product is '.($canSell ? '' : 'NOT ').' for sale - lets check price ... ');}
+            if ($this->debug) {debug::log('the product is '.($canSell ? '' : 'NOT ').' for sale - lets check price ... '); }
 
             //is there a valid price ???
             $countryPrice = $this->owner->getCalculatedPrice();
-            if ($this->debug) {debug::log('nothing applies, but we have a country price... '.$countryPrice);}
+            if ($this->debug) {debug::log('nothing applies, but we have a country price... '.$countryPrice); }
 
             return floatval($countryPrice) > 0 ? null : false;
         }
@@ -218,7 +218,7 @@ 
             return null;
         }
         $key = $this->owner->ClassName."___".$this->owner->ID.'____'.$countryCode;
-        if (! isset(self::$_buyable_price[$key])) {
+        if (!isset(self::$_buyable_price[$key])) {
             //basics
             $currency = null;
             $currencyCode = null;
@@ -306,7 +306,7 @@ 
                 }
             }
             //order must have a country and a currency
-            if (! $currencyCode ||  ! $countryCode) {
+            if (!$currencyCode || !$countryCode) {
                 if ($this->debug) {
                     debug::log('No currency ('.$currencyCode.') or no country code ('.$countryCode.') for order: ');
                 }
@@ -403,7 +403,7 @@ 
      */
     public function canEdit($member = null)
     {
-        if (! $member) {
+        if (!$member) {
             $member = Member::currentUser();
         }
         if ($member) {