sunnysideup / silverstripe-ecommerce

code/model/process/OrderSteps/OrderStep_Confirmed.php

Doc Comments +3 added lines, -3 removed lines

@@ -30,7 +30,7 @@ 
      *
      * @see Order::doNextStatus
      *
-     * @param Order object
+     * @param Order Order
      *
      * @return bool - true if the current step is ready to be run...
      **/
@@ -47,7 +47,7 @@ 
      *
      * @see Order::doNextStatus
      *
-     * @param Order object
+     * @param Order Order
      *
      * @return bool - true if run correctly.
      **/
@@ -61,7 +61,7 @@ 
      *
      * @param DataObject $order Order
      *
-     * @return DataObject | Null - DataObject = OrderStep
+     * @return OrderStep|null | Null - DataObject = OrderStep
      **/
     public function nextStep(Order $order)
     {

code/model/process/OrderSteps/OrderStep_Created.php

Doc Comments +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      *
      * @see Order::doNextStatus
      *
-     * @param Order object
+     * @param Order Order
      *
      * @return bool - true if the current step is ready to be run...
      **/

code/model/process/OrderSteps/OrderStep_Paid.php

Doc Comments +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
      *
      * @see Order::doNextStatus
      *
-     * @param Order object
+     * @param Order Order
      *
      * @return bool - true if the current step is ready to be run...
      **/
@@ -43,7 +43,7 @@ 
      *
      * @see Order::doNextStatus
      *
-     * @param Order object
+     * @param Order Order
      *
      * @return bool - true if run correctly.
      **/

code/model/process/OrderSteps/OrderStep_Sent.php

Doc Comments +3 added lines, -3 removed lines

@@ -48,7 +48,7 @@ 
      *
      * @see Order::doNextStatus
      *
-     * @param Order object
+     * @param Order Order
      *
      * @return bool - true if the current step is ready to be run...
      **/
@@ -65,9 +65,9 @@ 
      *
      * @see Order::doNextStatus
      *
-     * @param Order object
+     * @param Order Order
      *
-     * @return bool - true if run correctly.
+     * @return boolean|null - true if run correctly.
      **/
     public function doStep(Order $order)
     {

code/model/process/OrderSteps/OrderStep_SentInvoice.php

Doc Comments +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
       * NOTE: must have a payment (even if it is a fake payment).
       * The reason for this is if people pay straight away then they want to see the payment shown on their invoice.
       *
-      * @param Order object
+      * @param Order Order
       *
       * @return bool - true if the current step is ready to be run...
       **/

code/model/process/OrderSteps/OrderStep_SentReceipt.php

Doc Comments +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
      *
      * @see Order::doNextStatus
      *
-     * @param Order object
+     * @param Order Order
      *
      * @return bool - true if the current step is ready to be run...
      **/

code/model/process/OrderSteps/OrderStep_Submitted.php

Doc Comments +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
      *
      * @see Order::doNextStatus
      *
-     * @param Order object
+     * @param Order Order
      *
      * @return bool - true if the current step is ready to be run...
      **/
@@ -67,7 +67,7 @@ 
      /**
       * Add a member to the order - in case he / she is not a shop admin.
       *
-      * @param Order object
+      * @param Order Order
       *
       * @return bool - true if run correctly.
       **/

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

code/Product.php

Doc Comments +10 added lines, -6 removed lines

@@ -322,7 +322,7 @@ 
     /**
      * Used in getCSMFields.
      *
-     * @return GridField
+     * @return UploadField
      **/
     protected function getAdditionalImagesField()
     {
@@ -337,7 +337,7 @@ 
     /**
      * Used in getCSMFields.
      *
-     * @return GridField
+     * @return UploadField
      **/
     protected function getAdditionalFilesField()
     {
@@ -626,7 +626,7 @@ 
     /**
      * returns the default image of the product.
      *
-     * @return Image | Null
+     * @return string | Null
      */
     public function DefaultImage()
     {
@@ -661,7 +661,7 @@ 
     /**
      * @TODO: complete
      *
-     * @param string $compontent - the has many relationship you are looking at, e.g. OrderAttribute
+     * @param string $component - the has many relationship you are looking at, e.g. OrderAttribute
      *
      * @return DataList (CHECK!)
      */
@@ -752,7 +752,7 @@ 
     /**
      * You can set an alternative class name for order item using this method.
      *
-     * @param string $ClassName
+     * @param string $className
      **/
     public function setAlternativeClassNameForOrderItem($className)
     {
@@ -879,6 +879,7 @@ 
      * set new specific new quantity for buyable's orderitem.
      *
      * @param float
+     * @param integer $quantity
      *
      * @return string (Link)
      */
@@ -970,7 +971,7 @@ 
      * In templates, it is used like this:
      * $EcommerceConfigAjax.TableID.
      *
-     * @return EcommerceConfigAjax
+     * @return EcommerceConfigAjaxDefinitions
      **/
     public function AJAXDefinitions()
     {
@@ -1023,6 +1024,9 @@ 
 
     private static $_calculated_price_cache = array();
 
+    /**
+     * @return double
+     */
     public function getCalculatedPrice()
     {
         if (! isset(self::$_calculated_price_cache[$this->ID])) {