sunnysideup / silverstripe-ecommerce

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/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.
      **/

code/model/process/OrderSteps/OrderStep_SentReceipt.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/tasks/EcommerceTaskTryToFinaliseOrders.php

Doc Comments +5 added lines

@@ -96,6 +96,11 @@
     }
 
 
+    /**
+     * @param DataList $orders
+     * @param integer|null $limit
+     * @param integer|null $startAt
+     */
     protected function tryToFinaliseOrders($orders, $limit, $startAt)
     {
         $orders = $orders->limit($limit, $startAt);

code/forms/OrderForm_Feedback.php

Doc Comments +6 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
     /**
      * @param array        $data The form request data submitted
      * @param Form         $form The {@link Form} this was submitted on
-     * @param HTTPRequest  $request The {@link Form} this was submitted on
+     * @param SS_HTTPRequest  $request The {@link Form} this was submitted on
      */
     public function dofeedback(array $data, Form $form, SS_HTTPRequest $request)
     {
@@ -77,7 +77,6 @@ 
     /**
      * saves the form into session.
      *
-     * @param array $data - data from form.
      */
     public function saveDataToSession()
     {
@@ -85,6 +84,11 @@ 
         Session::set("FormInfo.{$this->FormName()}.data", $data);
     }
 
+    /**
+     * @param string $value
+     *
+     * @return string
+     */
     protected function getValueFromOrderConfirmationPage($value)
     {
         if($page = $this->getOrderConfirmationPage()) {