@@ -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 |      { |