@@ -80,6 +80,7 @@ |
||
80 | 80 | /** |
81 | 81 | * Return the link that we should redirect to. |
82 | 82 | * Only return a value if there is a legal redirection destination. |
83 | + * @return string |
|
83 | 84 | */ |
84 | 85 | public function redirectionLink() { |
85 | 86 | // Check external redirect |
@@ -316,6 +316,9 @@ discard block |
||
316 | 316 | } |
317 | 317 | } |
318 | 318 | |
319 | + /** |
|
320 | + * @param string $link |
|
321 | + */ |
|
319 | 322 | public function LinkWithSearch($link) { |
320 | 323 | // Whitelist to avoid side effects |
321 | 324 | $params = array( |
@@ -799,7 +802,7 @@ discard block |
||
799 | 802 | |
800 | 803 | /** |
801 | 804 | * @param HTTPRequest $request |
802 | - * @return string HTML |
|
805 | + * @return HTTPResponse HTML |
|
803 | 806 | */ |
804 | 807 | public function treeview($request) { |
805 | 808 | return $this->getResponseNegotiator()->respond($request); |
@@ -807,7 +810,7 @@ discard block |
||
807 | 810 | |
808 | 811 | /** |
809 | 812 | * @param HTTPRequest $request |
810 | - * @return string HTML |
|
813 | + * @return HTTPResponse HTML |
|
811 | 814 | */ |
812 | 815 | public function listview($request) { |
813 | 816 | return $this->getResponseNegotiator()->respond($request); |
@@ -1149,7 +1152,7 @@ discard block |
||
1149 | 1152 | * Actually perform the publication step |
1150 | 1153 | * |
1151 | 1154 | * @param Versioned|DataObject $record |
1152 | - * @return mixed |
|
1155 | + * @return HTTPResponse|null |
|
1153 | 1156 | */ |
1154 | 1157 | public function performPublish($record) { |
1155 | 1158 | if($record && !$record->canPublish()) { |