| @@ -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) | 
| @@ -136,6 +136,7 @@ discard block | ||
| 136 | 136 | /** | 
| 137 | 137 | * returns ABSOLUTE link to the shopping cart controller. | 
| 138 | 138 | * @param null | array | string $actionAndOtherLinkVariables | 
| 139 | + * @param string $actionAndOtherLinkVariables | |
| 139 | 140 | * @return string | 
| 140 | 141 | */ | 
| 141 | 142 | protected static function create_link($actionAndOtherLinkVariables = null) | 
| @@ -283,7 +284,7 @@ discard block | ||
| 283 | 284 | |
| 284 | 285 | /** | 
| 285 | 286 | * | 
| 286 | - * @return null | string | |
| 287 | + * @return string|null | string | |
| 287 | 288 | */ | 
| 288 | 289 | public static function copy_order_link($orderID, $parameters = array()) | 
| 289 | 290 |      { | 
| @@ -598,7 +599,7 @@ discard block | ||
| 598 | 599 | * | 
| 599 | 600 | * @param SS_HTTPRequest | 
| 600 | 601 | * | 
| 601 | - * @return HTML | |
| 602 | + * @return HTMLText | |
| 602 | 603 | */ | 
| 603 | 604 | public function showcart(SS_HTTPRequest $request) | 
| 604 | 605 |      { | 
| @@ -610,7 +611,7 @@ discard block | ||
| 610 | 611 | * | 
| 611 | 612 | * @param SS_HTTPRequest | 
| 612 | 613 | * | 
| 613 | - * @return REDIRECT | |
| 614 | + * @return SS_HTTPResponse|null | |
| 614 | 615 | */ | 
| 615 | 616 | public function loadorder(SS_HTTPRequest $request) | 
| 616 | 617 |      { | 
| @@ -706,7 +707,7 @@ discard block | ||
| 706 | 707 | * | 
| 707 | 708 | * @param SS_HTTPRequest | 
| 708 | 709 | * | 
| 709 | - * @return REDIRECT | |
| 710 | + * @return SS_HTTPResponse|null | |
| 710 | 711 | */ | 
| 711 | 712 | public function placeorderformember(SS_HTTPRequest $request) | 
| 712 | 713 |      { | 
| @@ -737,7 +738,7 @@ discard block | ||
| 737 | 738 | * | 
| 738 | 739 | * @param SS_HTTPRequest | 
| 739 | 740 | * | 
| 740 | - * @return REDIRECT | |
| 741 | + * @return SS_HTTPResponse|null | |
| 741 | 742 | */ | 
| 742 | 743 | public function loginas(SS_HTTPRequest $request) | 
| 743 | 744 |      { |