AntonyThorpe / silvershop-core

code/account/AccountPage.php

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
     /**
      * Return a link to view the order on the account page.
      *
-     * @param int|string $orderID    ID of the order
+     * @param integer $orderID    ID of the order
      * @param boolean    $urlSegment Return the URLSegment only
      *
      * @return string

code/account/OrderActionsForm.php

Doc Comments +4 added lines

@@ -27,6 +27,10 @@
 
     protected      $order;
 
+    /**
+     * @param ContentController $controller
+     * @param string $name
+     */
     public function __construct($controller, $name, Order $order)
     {
         $this->order = $order;

code/account/OrderManipulation.php

Doc Comments +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
      * Get the order via url 'ID' or form submission 'OrderID'.
      * It will check for permission based on session stored ids or member id.
      *
-     * @return the order
+     * @return DataObject|null order
      */
     public function orderfromid()
     {
@@ -124,7 +124,7 @@ 
     /**
      * Build a form for cancelling, or retrying payment for a placed order.
      *
-     * @return Form
+     * @return null|OrderActionsForm
      */
     public function ActionsForm()
     {

code/cart/CartEditField.php

Doc Comments -1 removed lines

@@ -90,7 +90,6 @@
      * Add quantity, variation and remove fields to the
      * item set.
      *
-     * @param SS_List $items
      */
     protected function editableItems()
     {

code/cart/ShoppingCart.php

Doc Comments +4 added lines, -2 removed lines

@@ -108,7 +108,7 @@ 
      * Adds an item to the cart
      *
      * @param Buyable $buyable
-     * @param number $quantity
+     * @param integer $quantity
      * @param unknown $filter
      *
      * @return boolean|OrderItem false or the new/existing item
@@ -258,7 +258,6 @@ 
      * Finds an existing order item.
      *
      * @param Buyable $buyable
-     * @param string $filter
      *
      * @return the item requested, or false
      */
@@ -360,6 +359,7 @@ 
 
     /**
      * Store a new error.
+     * @param string $message
      */
     protected function error($message)
     {
@@ -457,6 +457,8 @@ 
 
     /**
      * Helper for creating a url
+     * @param string $action
+     * @param Buyable $buyable
      */
     protected static function build_url($action, $buyable, $params = array())
     {

code/checkout/CheckoutPage.php

Doc Comments +2 added lines

@@ -21,6 +21,8 @@
      * Returns the link to the checkout page on this site
      *
      * @param boolean $urlSegment If set to TRUE, only returns the URLSegment field
+     * @param string $action
+     * @param integer $id
      *
      * @return string Link to checkout page
      */

code/checkout/components/CheckoutComponent.php

Doc Comments -1 removed lines

@@ -22,7 +22,6 @@
      * Get form fields for manipulating the current order,
      * according to the responsibilty of this component.
      *
-     * @param  Form $form the form being updated
      *
      * @throws Exception
      * @return FieldList fields for manipulating order

code/checkout/components/CheckoutComponentConfig.php

Doc Comments +1 added lines

@@ -69,6 +69,7 @@
      * Returns the first available component with the given class or interface.
      *
      * @param String ClassName
+     * @param string $type
      *
      * @return GridFieldComponent
      */

code/checkout/OrderEmailNotifier.php

Doc Comments +3 added lines, -2 removed lines

@@ -65,8 +65,8 @@ 
     /**
      * Send a mail of the order to the client (and another to the admin).
      *
-     * @param $template    - the class name of the email you wish to send
-     * @param $subject     - subject of the email
+     * @param string $template    - the class name of the email you wish to send
+     * @param string $subject     - subject of the email
      * @param $copyToAdmin - true by default, whether it should send a copy to the admin
      *
      * @return bool
@@ -164,6 +164,7 @@ 
      * Used in {@link OrderReport}.
      *
      * @param string $note Optional note-content (instead of using the OrderStatusLog)
+     * @param string $title
      */
     public function sendStatusChange($title, $note = null)
     {