| @@ -29,7 +29,7 @@ | ||
| 29 | 29 | /** | 
| 30 | 30 | * Return a link to view the order on the account page. | 
| 31 | 31 | * | 
| 32 | - * @param int|string $orderID ID of the order | |
| 32 | + * @param integer $orderID ID of the order | |
| 33 | 33 | * @param boolean $urlSegment Return the URLSegment only | 
| 34 | 34 | */ | 
| 35 | 35 | public static function get_order_link($orderID, $urlSegment = false) | 
| @@ -51,7 +51,7 @@ discard block | ||
| 51 | 51 | * Get the order via url 'ID' or form submission 'OrderID'. | 
| 52 | 52 | * It will check for permission based on session stored ids or member id. | 
| 53 | 53 | * | 
| 54 | - * @return the order | |
| 54 | + * @return DataObject|null order | |
| 55 | 55 | */ | 
| 56 | 56 | public function orderfromid() | 
| 57 | 57 |      { | 
| @@ -123,7 +123,7 @@ discard block | ||
| 123 | 123 | /** | 
| 124 | 124 | * Build a form for cancelling, or retrying payment for a placed order. | 
| 125 | 125 | * | 
| 126 | - * @return Form | |
| 126 | + * @return null|OrderActionsForm | |
| 127 | 127 | */ | 
| 128 | 128 | public function ActionsForm() | 
| 129 | 129 |      { | 
| @@ -90,7 +90,6 @@ | ||
| 90 | 90 | * Add quantity, variation and remove fields to the | 
| 91 | 91 | * item set. | 
| 92 | 92 | * | 
| 93 | - * @param SS_List $items | |
| 94 | 93 | */ | 
| 95 | 94 | protected function editableItems() | 
| 96 | 95 |      { | 
| @@ -121,7 +121,7 @@ discard block | ||
| 121 | 121 | * Adds an item to the cart | 
| 122 | 122 | * | 
| 123 | 123 | * @param Buyable $buyable | 
| 124 | - * @param number $quantity | |
| 124 | + * @param integer $quantity | |
| 125 | 125 | * @param unknown $filter | 
| 126 | 126 | * | 
| 127 | 127 | * @return boolean|OrderItem false or the new/existing item | 
| @@ -270,7 +270,6 @@ discard block | ||
| 270 | 270 | * Finds an existing order item. | 
| 271 | 271 | * | 
| 272 | 272 | * @param Buyable $buyable | 
| 273 | - * @param string $filter | |
| 274 | 273 | * | 
| 275 | 274 | * @return the item requested, or false | 
| 276 | 275 | */ | 
| @@ -334,6 +333,7 @@ discard block | ||
| 334 | 333 | |
| 335 | 334 | /** | 
| 336 | 335 | * Store a new error. | 
| 336 | + * @param string $message | |
| 337 | 337 | */ | 
| 338 | 338 | protected function error($message) | 
| 339 | 339 |      { | 
| @@ -431,6 +431,8 @@ discard block | ||
| 431 | 431 | |
| 432 | 432 | /** | 
| 433 | 433 | * Helper for creating a url | 
| 434 | + * @param string $action | |
| 435 | + * @param Buyable $buyable | |
| 434 | 436 | */ | 
| 435 | 437 | protected static function build_url($action, $buyable, $params = array()) | 
| 436 | 438 |      { | 
| @@ -169,6 +169,7 @@ | ||
| 169 | 169 | |
| 170 | 170 | /** | 
| 171 | 171 | * Store a new error & return false; | 
| 172 | + * @param string $message | |
| 172 | 173 | */ | 
| 173 | 174 | protected function error($message) | 
| 174 | 175 |      { | 
| @@ -21,6 +21,8 @@ | ||
| 21 | 21 | * Returns the link to the checkout page on this site | 
| 22 | 22 | * | 
| 23 | 23 | * @param boolean $urlSegment If set to TRUE, only returns the URLSegment field | 
| 24 | + * @param string $action | |
| 25 | + * @param integer $id | |
| 24 | 26 | * | 
| 25 | 27 | * @return string Link to checkout page | 
| 26 | 28 | */ | 
| @@ -22,7 +22,6 @@ | ||
| 22 | 22 | * Get form fields for manipulating the current order, | 
| 23 | 23 | * according to the responsibilty of this component. | 
| 24 | 24 | * | 
| 25 | - * @param Form $form the form being updated | |
| 26 | 25 | * | 
| 27 | 26 | * @throws Exception | 
| 28 | 27 | * @return FieldList fields for manipulating order | 
| @@ -67,6 +67,7 @@ | ||
| 67 | 67 | * Returns the first available component with the given class or interface. | 
| 68 | 68 | * | 
| 69 | 69 | * @param String ClassName | 
| 70 | + * @param string $type | |
| 70 | 71 | * | 
| 71 | 72 | * @return GridFieldComponent | 
| 72 | 73 | */ | 
| @@ -194,7 +194,7 @@ discard block | ||
| 194 | 194 | /** | 
| 195 | 195 | * Determine if an order can be placed. | 
| 196 | 196 | * | 
| 197 | - * @param boolean $order | |
| 197 | + * @param Order $order | |
| 198 | 198 | */ | 
| 199 | 199 | public function canPlace(Order $order) | 
| 200 | 200 |      { | 
| @@ -219,7 +219,6 @@ discard block | ||
| 219 | 219 | /** | 
| 220 | 220 | * Takes an order from being a cart to awaiting payment. | 
| 221 | 221 | * | 
| 222 | - * @param Member $member - assign a member to the order | |
| 223 | 222 | * | 
| 224 | 223 | * @return boolean - success/failure | 
| 225 | 224 | */ | 
| @@ -314,6 +313,9 @@ discard block | ||
| 314 | 313 | return $this->error; | 
| 315 | 314 | } | 
| 316 | 315 | |
| 316 | + /** | |
| 317 | + * @param string $message | |
| 318 | + */ | |
| 317 | 319 | private function error($message) | 
| 318 | 320 |      { | 
| 319 | 321 | $this->error = $message; |