| @@ -215,6 +215,7 @@ discard block | ||
| 215 | 215 | * Returns the "new order" link. | 
| 216 | 216 | * | 
| 217 | 217 | * @param int | String $orderID - not used in CartPage | 
| 218 | + * @param integer $orderID | |
| 218 | 219 | * | 
| 219 | 220 | * @return string (URLSegment) | 
| 220 | 221 | */ | 
| @@ -227,6 +228,7 @@ discard block | ||
| 227 | 228 | * Returns the "copy order" link. | 
| 228 | 229 | * | 
| 229 | 230 | * @param int | String $orderID - not used in CartPage | 
| 231 | + * @param integer $orderID | |
| 230 | 232 | * | 
| 231 | 233 | * @return string (URLSegment) | 
| 232 | 234 | */ | 
| @@ -240,7 +242,7 @@ discard block | ||
| 240 | 242 | * | 
| 241 | 243 | * @param int|string $orderID ID of the order | 
| 242 | 244 | * | 
| 243 | - * @return int | String (URLSegment) | |
| 245 | + * @return string | String (URLSegment) | |
| 244 | 246 | */ | 
| 245 | 247 | public static function get_order_link($orderID) | 
| 246 | 248 |      { | 
| @@ -382,6 +384,10 @@ discard block | ||
| 382 | 384 | * @todo: check if we need this....! | 
| 383 | 385 | **/ | 
| 384 | 386 | private $message = ''; | 
| 387 | + | |
| 388 | + /** | |
| 389 | + * @param string $s | |
| 390 | + */ | |
| 385 | 391 | public static function set_message($s) | 
| 386 | 392 |      { | 
| 387 | 393 |          $sessionCode = EcommerceConfig::get('CartPage_Controller', 'session_code'); | 
| @@ -648,7 +654,7 @@ discard block | ||
| 648 | 654 | } | 
| 649 | 655 | |
| 650 | 656 | /** | 
| 651 | - * @return string | |
| 657 | + * @return DBField | |
| 652 | 658 | **/ | 
| 653 | 659 | public function Message() | 
| 654 | 660 |      { | 
| @@ -162,6 +162,7 @@ discard block | ||
| 162 | 162 | * | 
| 163 | 163 | * @param string - $currentStep is the step that has just been actioned.... | 
| 164 | 164 | * @param bool - $doPreviousInstead - return previous rather than next step | 
| 165 | + * @param string $currentStep | |
| 165 | 166 | * | 
| 166 | 167 | * @return string (URLSegment) | 
| 167 | 168 | */ | 
| @@ -688,7 +689,7 @@ discard block | ||
| 688 | 689 | /** | 
| 689 | 690 | * returns the percentage of steps done (0 - 100). | 
| 690 | 691 | * | 
| 691 | - * @return int | |
| 692 | + * @return double | |
| 692 | 693 | */ | 
| 693 | 694 | public function PercentageDone() | 
| 694 | 695 |      { | 
| @@ -333,7 +333,7 @@ | ||
| 333 | 333 | } | 
| 334 | 334 | |
| 335 | 335 | /** | 
| 336 | - * @return array ???? | |
| 336 | + * @return string ???? | |
| 337 | 337 | */ | 
| 338 | 338 | public function AllTests() | 
| 339 | 339 |      { | 
| @@ -22,7 +22,7 @@ | ||
| 22 | 22 | * | 
| 23 | 23 | * @param SS_HTTPRequest | 
| 24 | 24 | * | 
| 25 | - * @return HTML | |
| 25 | + * @return EcommerceDatabaseAdmin | |
| 26 | 26 | **/ | 
| 27 | 27 | public function ecommerce(SS_HTTPRequest $request) | 
| 28 | 28 |      { | 
| @@ -22,7 +22,7 @@ | ||
| 22 | 22 | /** | 
| 23 | 23 | * Tells us the version of e-commerce in use. | 
| 24 | 24 | * | 
| 25 | - * @return int | |
| 25 | + * @return double | |
| 26 | 26 | */ | 
| 27 | 27 | public function Version() | 
| 28 | 28 |      { | 
| @@ -47,11 +47,11 @@ | ||
| 47 | 47 | /** | 
| 48 | 48 | * Builds json object to be returned via ajax. | 
| 49 | 49 | * | 
| 50 | - * @param array $message (Type, Message) | |
| 50 | + * @param array $messages (Type, Message) | |
| 51 | 51 | * @param array $additionalData | 
| 52 | 52 | * @param string $status | 
| 53 | 53 | * | 
| 54 | - * @return HEADER + JSON | |
| 54 | + * @return string + JSON | |
| 55 | 55 | **/ | 
| 56 | 56 | public function ReturnCartData(array $messages = array(), array $additionalData = null, $status = 'success') | 
| 57 | 57 |      { | 
| @@ -12,7 +12,7 @@ | ||
| 12 | 12 |  { | 
| 13 | 13 | /** | 
| 14 | 14 | * @param string $status the status to return | 
| 15 | - * @param string $message the message to return with the retur | |
| 15 | + * @param string $messages the message to return with the retur | |
| 16 | 16 | * @param null | Array $data, that should be included | 
| 17 | 17 | */ | 
| 18 | 18 | public function ReturnCartData(array $messages = array(), array $additionalData = null, $status = 'success') | 
| @@ -7,6 +7,9 @@ | ||
| 7 | 7 | 'read' => 'ShopAdmin', | 
| 8 | 8 | ); | 
| 9 | 9 | |
| 10 | + /** | |
| 11 | + * @param OrderEmailRecord $email | |
| 12 | + */ | |
| 10 | 13 | public static function review_link($email) | 
| 11 | 14 |      { | 
| 12 | 15 |          return Config::inst()->get('OrderEmailRecord_Review', 'url_segment').'/read/'.$email->ID; | 
| @@ -128,6 +128,7 @@ | ||
| 128 | 128 | /** | 
| 129 | 129 | * @oaram string $action | 
| 130 | 130 | * | 
| 131 | + * @param string $action | |
| 131 | 132 | * @return string | 
| 132 | 133 | */ | 
| 133 | 134 | public function Link($action = null) |