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_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_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/ProductGroup.php

Doc Comments +6 added lines, -7 removed lines

@@ -372,7 +372,7 @@ 
      * by either type (e.g. FILER) or variable (e.g dbFieldName)
      * or both.
      *
-     * @param string $typeOfVariableName FILTER | SORT | DISPLAY or sessionName, getVariable, etc...
+     * @param string $typeOrVariable FILTER | SORT | DISPLAY or sessionName, getVariable, etc...
      * @param string $variable:          sessionName, getVariable, etc...
      *
      * @return array | String
@@ -1528,9 +1528,8 @@ 
 
     /**
      *
-     * @param string $name
-     * @param string $filterKey
      *
+     * @param string $cacheKey
      * @return string
      */
     public function cacheKey($cacheKey)
@@ -1827,7 +1826,7 @@ 
      * This is the call that is made from the template...
      * The actual final products being shown.
      *
-     * @return PaginatedList
+     * @return DataList
      **/
     public function Products()
     {
@@ -2151,7 +2150,7 @@ 
      *
      * This is different from IsSearchResults.
      *
-     * @return bool
+     * @return boolean|null
      */
     public function ActiveSearchTerm()
     {
@@ -2217,7 +2216,7 @@ 
      * Is there a special sort operating at the moment?
      * Is the current sort the default one (return inverse!)?
      *
-     * @return bool
+     * @return boolean|null
      */
     public function HasSort()
     {
@@ -2599,7 +2598,7 @@ 
      *
      * @param SS_List
      *
-     * @return PaginatedList
+     * @return ProductGroup_Controller|null
      */
     protected function paginateList(SS_List $list)
     {

code/search/filters/EcommercePaymentFilters.php

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
     private $divider = 90;
 
     /**
-     *@return SQLQuery
+     *@return DataQuery
      **/
     public function applyOne(DataQuery $query)
     {

code/search/filters/OrderEmailRecordFilters.php

Doc Comments +1 added lines, -1 removed lines

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

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;