| @@ -37,7 +37,7 @@ | ||
| 37 | 37 | * Set the next step | 
| 38 | 38 | * should this be configurable .. ? | 
| 39 | 39 | * | 
| 40 | - * @param $step | |
| 40 | + * @param null|string $step | |
| 41 | 41 | */ | 
| 42 | 42 | public function setNextStep($step) | 
| 43 | 43 |      { | 
| @@ -34,6 +34,10 @@ | ||
| 34 | 34 | */ | 
| 35 | 35 | protected $reservation; | 
| 36 | 36 | |
| 37 | + /** | |
| 38 | + * @param ReservationController $controller | |
| 39 | + * @param string $name | |
| 40 | + */ | |
| 37 | 41 | public function __construct($controller, $name, Reservation $reservation = null) | 
| 38 | 42 |      { | 
| 39 | 43 | $requiredFields = array(); | 
| @@ -27,6 +27,10 @@ discard block | ||
| 27 | 27 | */ | 
| 28 | 28 | protected $reservation; | 
| 29 | 29 | |
| 30 | + /** | |
| 31 | + * @param SummaryController $controller | |
| 32 | + * @param string $name | |
| 33 | + */ | |
| 30 | 34 | public function __construct($controller, $name, Reservation $reservation) | 
| 31 | 35 |      { | 
| 32 | 36 | $fields = FieldList::create( | 
| @@ -108,7 +112,7 @@ discard block | ||
| 108 | 112 | /** | 
| 109 | 113 | * Get the last error message from the payment attempts | 
| 110 | 114 | * | 
| 111 | - * @return bool|string | |
| 115 | + * @return false|string | |
| 112 | 116 | */ | 
| 113 | 117 | public function getPaymentErrorMessage() | 
| 114 | 118 |      { | 
| @@ -32,6 +32,9 @@ discard block | ||
| 32 | 32 | */ | 
| 33 | 33 | protected $event; | 
| 34 | 34 | |
| 35 | + /** | |
| 36 | + * @param string $name | |
| 37 | + */ | |
| 35 | 38 | public function __construct($controller, $name, DataList $tickets = null, CalendarEvent $event = null) | 
| 36 | 39 |      { | 
| 37 | 40 | $this->event = $event; | 
| @@ -65,7 +68,7 @@ discard block | ||
| 65 | 68 | * @param array $data | 
| 66 | 69 | * @param TicketForm $form | 
| 67 | 70 | * | 
| 68 | - * @return string | |
| 71 | + * @return \SS_HTTPResponse | |
| 69 | 72 | */ | 
| 70 | 73 | public function handleTicketForm(array $data, TicketForm $form) | 
| 71 | 74 |      { | 
| @@ -14,6 +14,7 @@ | ||
| 14 | 14 | * Modify the given total | 
| 15 | 15 | * | 
| 16 | 16 | * @param float $total | 
| 17 | + * @return void | |
| 17 | 18 | */ | 
| 18 | 19 | public function updateTotal(&$total); | 
| 19 | 20 | |
| @@ -98,7 +98,7 @@ | ||
| 98 | 98 | /** | 
| 99 | 99 | * Set the price modification on the join | 
| 100 | 100 | * | 
| 101 | - * @param $value | |
| 101 | + * @param double $value | |
| 102 | 102 | */ | 
| 103 | 103 | public function setPriceModification($value) | 
| 104 | 104 |      { | 
| @@ -288,7 +288,7 @@ | ||
| 288 | 288 | * Safely change to a state | 
| 289 | 289 | * todo check if state direction matches | 
| 290 | 290 | * | 
| 291 | - * @param $state | |
| 291 | + * @param string $state | |
| 292 | 292 | * | 
| 293 | 293 | * @return boolean | 
| 294 | 294 | */ | 
| @@ -50,7 +50,7 @@ discard block | ||
| 50 | 50 | * @param $gridField | 
| 51 | 51 | * @param null $request | 
| 52 | 52 | * | 
| 53 | - * @return mixed | |
| 53 | + * @return \SS_HTTPResponse|null | |
| 54 | 54 | */ | 
| 55 | 55 |      public function handleExport($gridField, $request = null) { | 
| 56 | 56 |          $now = Date("d-m-Y-H-i"); | 
| @@ -65,7 +65,7 @@ discard block | ||
| 65 | 65 | * Generate export fields for CSV. | 
| 66 | 66 | * | 
| 67 | 67 | * @param GridField $gridField | 
| 68 | - * @return array | |
| 68 | + * @return string | |
| 69 | 69 | */ | 
| 70 | 70 |      public function generateExportFileData($gridField) { | 
| 71 | 71 | $separator = $this->csvSeparator; | 
| @@ -111,7 +111,7 @@ | ||
| 111 | 111 | /** | 
| 112 | 112 | * Translate the given type to a readable message | 
| 113 | 113 | * | 
| 114 | - * @param $message string | |
| 114 | + * @param string $message string | |
| 115 | 115 | * | 
| 116 | 116 | * @return string | 
| 117 | 117 | */ |