sunnysideup / silverstripe-ecommerce_countries

code/control/cms/CountryPrice_TestController.php

Doc Comments +1 added lines, -1 removed lines

@@ -347,7 +347,7 @@
     /**
      * returns the Distributor for the user
      * Needs to be in model so we can access from the order in the template.
-     * @return Varchar Field
+     * @return Distributor Field
      */
     public function MyDistributor()
     {

code/model/buyables/CountryPrice_CopyPrices.php

Doc Comments -1 removed lines

@@ -96,7 +96,6 @@
      *  - ID
      * searches through children, until all all childpages have been added
      *
-     * @param  string $country [description]
      * @param  array  $values  [description]
      * @return array          [description]
      */

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()
     {

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)

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)
     {

code/model/address/CountryPrice_EcommerceCountry.php

Doc Comments +4 added lines

@@ -303,6 +303,10 @@
         return self::$_get_real_country_cache[$cacheKey];
     }
 
+    /**
+     * @param string $countryOrCountryCodeA
+     * @param string $countryOrCountryCodeB
+     */
     public static function countries_belong_to_same_group($countryOrCountryCodeA, $countryOrCountryCodeB)
     {
         $countryA = EcommerceCountry::get_country_from_mixed_var($countryOrCountryCodeA);

code/model/CountryPrice.php

Doc Comments +5 added lines, -1 removed lines

@@ -210,6 +210,10 @@ 
     private static $cms_object = null;
 
     //MUST KEEP
+
+    /**
+     * @param SS_Object $o
+     */
     public static function set_cms_object($o)
     {
         self::$cms_object = $o;
@@ -265,7 +269,7 @@ 
 
     /**
      * Returns if the currency is an old currency not used anymore.
-     * @return Boolean
+     * @return boolean|null
      */
     public function isObsolete()
     {