sunnysideup / silverstripe-ecommerce

code/email/Order_Email.php

Doc Comments +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
      * turns an html document into a formatted html document
      * using the emogrify method.
      *
-     * @param $html
+     * @param string $html
      *
      * @return string HTML
      */

code/forms/fields/EcomQuantityField.php

Doc Comments +1 added lines, -1 removed lines

@@ -185,7 +185,7 @@
     }
 
     /**
-     * @return string (HTML)
+     * @return HTMLText (HTML)
      **/
     public function forTemplate()
     {

code/forms/OrderForm_Payment.php

Doc Comments -1 removed lines

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

code/forms/OrderStatusLogForm.php

Doc Comments -1 removed lines

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

code/forms/ShopAccountForm.php

Doc Comments -1 removed lines

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

code/model/extensions/EcommerceRole.php

Doc Comments +3 added lines, -2 removed lines

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

code/model/money/EcommercePayment.php

Doc Comments +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
      * CRUCIAL
      * makes sure all the relevant payment methods are available ...
      *
-     * @return this | EcommercePayment
+     * @return EcommercePayment | EcommercePayment
      */
     public function init()
     {

code/model/OrderAttribute.php

Doc Comments +2 added lines, -2 removed lines

@@ -291,7 +291,7 @@ 
     /**
      * returns the order - for some unknown reason it seems we need this.
      *
-     * @return Order | null
+     * @return null|Order | null
      */
     public function Order()
     {
@@ -330,7 +330,7 @@ 
      * In templates, it is used like this:
      * $EcommerceConfigAjax.TableID.
      *
-     * @return EcommerceConfigAjax
+     * @return EcommerceConfigAjaxDefinitions
      **/
     public function AJAXDefinitions()
     {

code/model/process/OrderStep.php

Doc Comments +4 added lines, -3 removed lines

@@ -655,6 +655,7 @@ 
      *
      * @param string $code:       the name of the step to check
      * @param bool   $orIsEqualTo if set to true, this method will return TRUE if the step being checked is the current one
+     * @param string $code
      *
      * @return bool
      **/
@@ -903,7 +904,7 @@ 
     /**
      * Formatted answer for "hasCustomerMessage".
      *
-     * @return string
+     * @return DBField
      */
     public function ShowAsSummary()
     {
@@ -913,7 +914,7 @@ 
     /**
      *
      *
-     * @return string
+     * @return DBField
      */
     public function getShowAsSummary()
     {
@@ -973,7 +974,7 @@ 
     /**
      * Formatted answer for "hasCustomerMessage".
      *
-     * @return string
+     * @return DBField
      */
     public function NameAndDescription()
     {