sunnysideup / silverstripe-ecommerce

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/search/filters/OrderFilters_AroundDateFilter.php

Doc Comments +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     private $divider = 3;
 
     /**
-     *@return SQLQuery
+     *@return DataQuery
      **/
     public function apply(DataQuery $query)
     {

code/search/filters/OrderFilters_HasBeenCancelled.php

Doc Comments +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 class OrderFilters_HasBeenCancelled extends ExactMatchFilter
 {
     /**
-     *@return SQLQuery
+     *@return DataQuery
      **/
     public function apply(DataQuery $query)
     {

code/search/filters/OrderFilters_MemberAndAddress.php

Doc Comments +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 class OrderFilters_MemberAndAddress extends ExactMatchFilter
 {
     /**
-     *@return SQLQuery
+     *@return DataQuery
      **/
     public function apply(DataQuery $query)
     {

code/search/filters/OrderFilters_MultiOptionsetStatusIDFilter.php

Doc Comments +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 class OrderFilters_MultiOptionsetStatusIDFilter extends ExactMatchFilter
 {
     /**
-     *@return SQLQuery
+     *@return DataQuery
      **/
     public function apply(DataQuery $query)
     {

code/search/filters/OrderFilters_MustHaveAtLeastOnePayment.php

Doc Comments +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 class OrderFilters_MustHaveAtLeastOnePayment extends ExactMatchFilter
 {
     /**
-     *@return SQLQuery
+     *@return DataQuery
      **/
     public function apply(DataQuery $query)
     {

code/model/Order.php

Doc Comments +16 added lines, -11 removed lines

@@ -192,7 +192,7 @@ 
      * @param bool (optional) $b
      * @param int (optional)  $orderID
      *
-     * @return bool
+     * @return boolean|null
      */
     public static function set_needs_recalculating($b = true, $orderID = 0)
     {
@@ -264,7 +264,7 @@ 
     /**
      * This function returns the OrderSteps.
      *
-     * @return ArrayList (OrderSteps)
+     * @return DataList (OrderSteps)
      **/
     public static function get_order_status_options()
     {
@@ -453,6 +453,7 @@ 
      * link to edit the record.
      *
      * @param string | Null $action - e.g. edit
+     * @param string $action
      *
      * @return string
      */
@@ -1159,7 +1160,7 @@ 
      * @param Member $member - the user cancelling the order
      * @param string $reason - the reason the order is cancelled
      *
-     * @return OrderStatusLog_Cancel
+     * @return integer
      */
     public function Cancel(Member $member, $reason = '')
     {
@@ -1594,7 +1595,7 @@ 
      * Stores the preferred currency of the order.
      * IMPORTANTLY we store the exchange rate for future reference...
      *
-     * @param EcommerceCurrency $currency
+     * @param EcommerceCurrency $newCurrency
      */
     public function UpdateCurrency($newCurrency)
     {
@@ -2055,7 +2056,7 @@ 
      * @param string|array $excluded               - Class(es) of modifier(s) to ignore in the calculation.
      * @param bool         $stopAtExcludedModifier - when this flag is TRUE, we stop adding the modifiers when we reach an excluded modifier.
      *
-     * @return float
+     * @return integer
      */
     public function ModifiersSubTotal($excluded = null, $stopAtExcludedModifier = false)
     {
@@ -2253,7 +2254,7 @@ 
      *
      * @param Member (optional) $member
      *
-     * @return bool
+     * @return boolean|null
      **/
     public function canViewAdminStuff($member = null)
     {
@@ -2306,7 +2307,7 @@ 
      * If any of them need immediate attention then this is done
      * first after which it will go through to the checkout page.
      *
-     * @param Member (optional) $member
+     * @param Member Member $member
      *
      * @return bool
      **/
@@ -2652,6 +2653,10 @@ 
     {
         return $this->getTitle($dateFormat, $includeName);
     }
+
+    /**
+     * @param string $dateFormat
+     */
     public function getTitle($dateFormat = null, $includeName = false)
     {
         if ($this->exists()) {
@@ -2715,7 +2720,7 @@ 
     /**
      * Returns the subtotal of the items for this order.
      *
-     * @return float
+     * @return integer
      */
     public function SubTotal()
     {
@@ -2781,7 +2786,7 @@ 
     /**
      * Returns the total cost of an order including the additional charges or deductions of its modifiers.
      *
-     * @return float
+     * @return integer
      */
     public function Total()
     {
@@ -3411,7 +3416,7 @@ 
     /**
      * Converts the Order into HTML, based on the Order Template.
      *
-     * @return HTML Object
+     * @return DBField Object
      **/
     public function ConvertToHTML()
     {
@@ -3475,7 +3480,7 @@ 
      * In templates, it is used like this:
      * $EcommerceConfigAjax.TableID.
      *
-     * @return EcommerceConfigAjax
+     * @return EcommerceConfigAjaxDefinitions
      **/
     public function AJAXDefinitions()
     {

code/model/process/OrderSteps/OrderStep_Paid.php

Doc Comments +2 added lines, -2 removed lines

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