sunnysideup / silverstripe-ecommerce

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/EcommerceSiteTreeExtension.php

Doc Comments +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
      * In templates, it is used like this:
      * $AJAXDefinitions.TableID.
      *
-     * @return EcommerceConfigAjax
+     * @return EcommerceConfigAjaxDefinitions
      **/
     public function AJAXDefinitions()
     {

code/model/OrderItem.php

Doc Comments +10 added lines, -2 removed lines

@@ -300,7 +300,7 @@ 
      * standard SS method.
      *
      * @param BuyableModel $buyable
-     * @param float        $quantity
+     * @param integer        $quantity
      *
      * @return FieldList
      **/
@@ -492,6 +492,10 @@ 
     {
         return $this->getUnitPrice($recalculate);
     }
+
+    /**
+     * @return double
+     */
     public function getUnitPrice($recalculate = false)
     {
         if ($this->priceHasBeenFixed($recalculate) && !$recalculate) {
@@ -534,6 +538,10 @@ 
     {
         return $this->getTotal();
     }
+
+    /**
+     * @return double
+     */
     public function getTotal($recalculate = false)
     {
         if ($this->priceHasBeenFixed()) {
@@ -580,7 +588,7 @@ 
     }
 
     /**
-     * @return Field (EcomQuantityField)
+     * @return EcomQuantityField (EcomQuantityField)
      **/
     public function QuantityField()
     {

code/model/OrderModifier.php

Doc Comments +2 added lines, -2 removed lines

@@ -422,7 +422,7 @@ 
     }
 
     /**
-     * @return object (HeadingField)
+     * @return FormField (HeadingField)
      */
     protected function headingField()
     {
@@ -435,7 +435,7 @@ 
     }
 
     /**
-     * @return object (LiteralField)
+     * @return LiteralField (LiteralField)
      */
     protected function descriptionField()
     {

code/model/process/OrderStatusLogs/OrderStatusLog_DispatchPhysicalOrder.php

Doc Comments +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
     }
 
     /**
-     *@return string
+     *@return HTMLText
      **/
     public function CustomerNote()
     {

code/model/process/OrderStatusLogs/OrderStatusLog_PaymentCheck.php

Doc Comments +1 added lines, -1 removed lines

@@ -93,7 +93,7 @@
     }
 
     /**
-     * @return string
+     * @return HTMLText|null
      **/
     public function CustomerNote()
     {

code/OrderConfirmationPage.php

Doc Comments +1 added lines, -2 removed lines

@@ -253,7 +253,6 @@ 
      * Return a link to view the order on this page.
      *
      * @param int|string $orderID                ID of the order
-     * @param string     $type                   - the type of email you want to send.
      * @param bool       $actuallySendEmail      - do we actually send the email
      * @param int        $alternativeOrderStepID - OrderStep to use
      *
@@ -290,7 +289,7 @@ 
      *
      * @param bool $isCurrentStep
      *
-     * @return Checkout_StepDescription
+     * @return CheckoutPage_StepDescription
      */
     public function CurrentCheckoutStep($isCurrentStep = false)
     {