sunnysideup / silverstripe-ecommerce

code/search/filters/OrderFilters.php

Doc Comments +5 added lines, -5 removed lines

@@ -28,7 +28,7 @@ 
     private $divider = 3;
 
     /**
-     *@return SQLQuery
+     *@return DataQuery
      **/
     public function apply(DataQuery $query)
     {
@@ -73,7 +73,7 @@ 
 class OrderFilters_MemberAndAddress extends ExactMatchFilter
 {
     /**
-     *@return SQLQuery
+     *@return DataQuery
      **/
     public function apply(DataQuery $query)
     {
@@ -146,7 +146,7 @@ 
 class OrderFilters_MultiOptionsetStatusIDFilter extends ExactMatchFilter
 {
     /**
-     *@return SQLQuery
+     *@return DataQuery
      **/
     public function apply(DataQuery $query)
     {
@@ -181,7 +181,7 @@ 
 class OrderFilters_HasBeenCancelled extends ExactMatchFilter
 {
     /**
-     *@return SQLQuery
+     *@return DataQuery
      **/
     public function apply(DataQuery $query)
     {
@@ -216,7 +216,7 @@ 
 class OrderFilters_MustHaveAtLeastOnePayment extends ExactMatchFilter
 {
     /**
-     *@return SQLQuery
+     *@return DataQuery
      **/
     public function apply(DataQuery $query)
     {

code/tasks/EcommerceTaskArchiveAllOrdersWithItems.php

Doc Comments +5 added lines

@@ -90,6 +90,11 @@
         }
     }
 
+    /**
+     * @param DataObject|null $lastOrderStep
+     * @param string $orderStatusLogClassName
+     * @param integer $offset
+     */
     public function getOrdersForCreateSubmissionLogForArchivedOrders($lastOrderStep, $orderStatusLogClassName, $offset)
     {
         return Order::get()

code/tasks/EcommerceTaskCleanupProducts.php

Doc Comments +3 added lines

@@ -19,6 +19,9 @@
 
     protected $deleteFirst = true;
 
+    /**
+     * @param boolean $b
+     */
     public function setDeleteFirst($b)
     {
         $this->deleteFirst = $b;

code/tasks/EcommerceTaskMigration.php

Doc Comments +3 added lines, -1 removed lines

@@ -196,8 +196,10 @@
      *
      * @param string - $field - name of the field to be tested
      * @param string - $table - name of the table to be tested
+     * @param string $table
+     * @param string $field
      *
-     * @return bool
+     * @return boolean|null
      */
     protected function makeFieldObsolete($table, $field, $format = '')
     {

code/tasks/EcommerceTaskOrderItemsPerCustomer.php

Doc Comments +3 added lines

@@ -69,6 +69,9 @@
         }
     }
 
+    /**
+     * @param DataList $orderItems
+     */
     public function generateExportFileData($email, $date, $orderItems)
     {
         $separator = ',';

code/tasks/EcommerceTaskTryToFinaliseOrders.php

Doc Comments +4 added lines

@@ -92,6 +92,10 @@
         $this->doNotSendEmails = false;
     }
 
+    /**
+     * @param integer|null $limit
+     * @param integer|null $startAt
+     */
     protected function tryToFinaliseOrders($orders, $limit, $startAt)
     {
         $orders = $orders->limit($limit, $startAt);

code/forms/gridfield/GridFieldPrintAllInvoicesButton.php

Doc Comments +1 added lines, -1 removed lines

@@ -18,7 +18,6 @@ 
 
     /**
      * @param string $targetFragment The HTML fragment to write the button into
-     * @param array $exportColumns The columns to include in the export
      */
     public function __construct($targetFragment = "after")
     {
@@ -72,6 +71,7 @@ 
 
     /**
      * Handle the print, for both the action button and the URL
+      * @param GridField $gridField
       */
     public function handlePrintAllPackingSlips($gridField, $request = null)
     {

code/forms/OrderForm_Cancel.php

Doc Comments -1 removed lines

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

code/forms/OrderForm_Payment.php

Doc Comments -1 removed lines

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