sunnysideup / silverstripe-ecommerce

code/forms/OrderModifierForm.php

Doc Comments +3 added lines, -1 removed lines

@@ -64,6 +64,9 @@ 
         $this->extend('updateOrderModifierForm', $this);
     }
 
+    /**
+     * @param string $str
+     */
     protected function myLcFirst($str)
     {
         if (function_exists('lcfirst') === false) {
@@ -104,7 +107,6 @@ 
     /**
      * saves the form into session.
      *
-     * @param array $data - data from form.
      */
     public function saveDataToSession()
     {

code/forms/OrderStatusLogForm.php

Doc Comments -1 removed lines

@@ -68,7 +68,6 @@
     /**
      * saves the form into session.
      *
-     * @param array $data - data from form.
      */
     public function saveDataToSession()
     {

code/forms/ProductSearchForm.php

Doc Comments +2 added lines, -3 removed lines

@@ -489,7 +489,7 @@ 
      * can be executed one after the other, each
      * being less specific than the last...
      *
-     * @return array
+     * @return boolean
      */
     private function addToResults($listToAdd)
     {
@@ -512,8 +512,8 @@ 
      * can be executed one after the other, each
      * being less specific than the last...
      *
-     * @param array $words  - words being search
      * @param array $fields - fields being searched
+     * @param string $keywordPhrase
      *
      * @return array
      */
@@ -553,7 +553,6 @@ 
     /**
      * saves the form into session.
      *
-     * @param array $data - data from form.
      */
     public function saveDataToSession()
     {

code/forms/ShopAccountForm.php

Doc Comments -1 removed lines

@@ -188,7 +188,6 @@
     /**
      * saves the form into session.
      *
-     * @param array $data - data from form.
      */
     public function saveDataToSession()
     {

code/interfaces/BuyableModel.php

Doc Comments +5 added lines, -1 removed lines

@@ -112,7 +112,7 @@ 
     /**
      * You can set an alternative class name for order item using this method.
      *
-     * @param string $ClassName
+     * @param string $className
      **/
     public function setAlternativeClassNameForOrderItem($className);
 
@@ -248,6 +248,10 @@ 
      * @return Money
      */
     public function CalculatedPriceAsMoney();
+
+    /**
+     * @return Money
+     */
     public function getCalculatedPriceAsMoney();
 
     //CRUD SETTINGS

code/interfaces/EcommercePaymentSupportedMethodsProviderInterface.php

Doc Comments +1 added lines

@@ -13,6 +13,7 @@
     /**
      * assign the right payment gateways for the user
      * @param string (optional) $gateway
+     * @return void
      */
     public static function assign_payment_gateway($gateway = "");
 }

code/model/address/EcommerceRegion.php

Doc Comments -1 removed lines

@@ -185,7 +185,6 @@
     /**
      * checks if a code is allowed.
      *
-     * @param string $code - e.g. NZ, NSW, or CO
      *
      * @return bool
      */

code/model/address/OrderAddress.php

Doc Comments +5 added lines, -1 removed lines

@@ -132,6 +132,9 @@ 
      */
     protected $_canView = null;
 
+    /**
+     * @param DataObject $member
+     */
     public function canCreate($member = null)
     {
         if (! $member) {
@@ -353,6 +356,7 @@ 
      * makes selected fields into read only using the $this->readOnlyFields array.
      *
      * @param FieldList | Composite $fields
+     * @param CompositeField $fields
      *
      * @return FieldList
      */
@@ -420,7 +424,7 @@ 
      * Casted variable
      * returns the full strng of the record.
      *
-     * @return string
+     * @return HTMLText
      */
     public function FullString()
     {

code/model/extensions/EcommerceRole.php

Doc Comments +4 added lines, -2 removed lines

@@ -142,6 +142,7 @@ 
      * tells us if the current member is in the Shop Administrators Group.
      *
      * @param Member | Null $member
+     * @param DataObject $member
      *
      * @return bool
      */
@@ -161,6 +162,7 @@ 
      * tells us if the current member is in the Shop Administrators Group.
      *
      * @param Member | Null $member
+     * @param DataObject $member
      *
      * @return bool
      */
@@ -180,6 +182,7 @@ 
      * tells us if the current member can process the orders
      *
      * @param Member | Null $member
+     * @param DataObject $member
      *
      * @return bool
      */
@@ -416,7 +419,6 @@ 
     }
 
     /**
-     * @param bool $additionalFields: add extra fields.
      *
      * @return FieldList
      */
@@ -503,7 +505,7 @@ 
      * Return which member fields should be required on {@link OrderForm}
      * and {@link ShopAccountForm}.
      *
-     * @return array
+     * @return string[]
      */
     public function getEcommerceRequiredFields()
     {