@@ -556,7 +556,7 @@  | 
                                                    ||
| 556 | 556 | /**  | 
                                                        
| 557 | 557 | * Return the with the passed ID (via URL parameters) or the current user  | 
                                                        
| 558 | 558 | *  | 
                                                        
| 559 | - * @return null|Member Returns the member object or NULL if the member  | 
                                                        |
| 559 | + * @return DataObject|null Returns the member object or NULL if the member  | 
                                                        |
| 560 | 560 | * was not found  | 
                                                        
| 561 | 561 | */  | 
                                                        
| 562 | 562 |   	function Member() { | 
                                                        
@@ -3,6 +3,9 @@  | 
                                                    ||
| 3 | 3 |  class CheckableOption extends CompositeField { | 
                                                        
| 4 | 4 | protected $childField, $checkbox;  | 
                                                        
| 5 | 5 | |
| 6 | + /**  | 
                                                        |
| 7 | + * @param string $checkName  | 
                                                        |
| 8 | + */  | 
                                                        |
| 6 | 9 |  	function __construct($checkName, $childField, $value = "", $readonly = false) { | 
                                                        
| 7 | 10 | $this->name = $checkName;  | 
                                                        
| 8 | 11 | $this->checkbox = new CheckboxField($checkName, "", $value);  | 
                                                        
@@ -28,6 +28,10 @@  | 
                                                    ||
| 28 | 28 | return i18n::get_locale();  | 
                                                        
| 29 | 29 | }  | 
                                                        
| 30 | 30 | |
| 31 | + /**  | 
                                                        |
| 32 | + * @param string $name  | 
                                                        |
| 33 | + * @param string $title  | 
                                                        |
| 34 | + */  | 
                                                        |
| 31 | 35 |  	function __construct($name, $title = null, $source = null, $value = "", $form=null) { | 
                                                        
| 32 | 36 |  		if(!is_array($source)) { | 
                                                        
| 33 | 37 | // Get a list of countries from Zend  | 
                                                        
@@ -143,7 +143,7 @@  | 
                                                    ||
| 143 | 143 | * Check if they have visited this thread before. If they haven't increment  | 
                                                        
| 144 | 144 | * the NumViews value by 1 and set visited to true.  | 
                                                        
| 145 | 145 | *  | 
                                                        
| 146 | - * @return void  | 
                                                        |
| 146 | + * @return false|null  | 
                                                        |
| 147 | 147 | */  | 
                                                        
| 148 | 148 |  	public function incNumViews() { | 
                                                        
| 149 | 149 |  		if(Session::get('ForumViewed-' . $this->ID)) return false; | 
                                                        
@@ -383,7 +383,7 @@ discard block  | 
                                                    ||
| 383 | 383 | |
| 384 | 384 | /**  | 
                                                        
| 385 | 385 | * Returns the Topics (the first Post of each Thread) for this Forum  | 
                                                        
| 386 | - * @return DataList  | 
                                                        |
| 386 | + * @return PaginatedList  | 
                                                        |
| 387 | 387 | */  | 
                                                        
| 388 | 388 |  	public function getTopics() { | 
                                                        
| 389 | 389 | // Get a list of Posts  | 
                                                        
@@ -584,7 +584,7 @@ discard block  | 
                                                    ||
| 584 | 584 | *  | 
                                                        
| 585 | 585 | * Designed to be called via AJAX so return true / false  | 
                                                        
| 586 | 586 | *  | 
                                                        
| 587 | - * @return bool  | 
                                                        |
| 587 | + * @return null|false  | 
                                                        |
| 588 | 588 | */  | 
                                                        
| 589 | 589 |  	public function unsubscribe(SS_HTTPRequest $request) { | 
                                                        
| 590 | 590 | $member = Member::currentUser();  | 
                                                        
@@ -708,7 +708,7 @@ discard block  | 
                                                    ||
| 708 | 708 | /**  | 
                                                        
| 709 | 709 | * Get posts to display. This method assumes an URL parameter "ID" which contains the thread ID.  | 
                                                        
| 710 | 710 | * @param string sortDirection The sort order direction, either ASC for ascending (default) or DESC for descending  | 
                                                        
| 711 | - * @return DataObjectSet Posts  | 
                                                        |
| 711 | + * @return PaginatedList Posts  | 
                                                        |
| 712 | 712 | */  | 
                                                        
| 713 | 713 |  	function Posts($sortDirection = "ASC") { | 
                                                        
| 714 | 714 | $numPerPage = Forum::$posts_per_page;  | 
                                                        
@@ -737,7 +737,7 @@ discard block  | 
                                                    ||
| 737 | 737 | /**  | 
                                                        
| 738 | 738 | * Get the usable BB codes  | 
                                                        
| 739 | 739 | *  | 
                                                        
| 740 | - * @return DataObjectSet Returns the usable BB codes  | 
                                                        |
| 740 | + * @return ArrayList Returns the usable BB codes  | 
                                                        |
| 741 | 741 | * @see BBCodeParser::usable_tags()  | 
                                                        
| 742 | 742 | */  | 
                                                        
| 743 | 743 |  	function BBTags() { | 
                                                        
@@ -1018,6 +1018,8 @@ discard block  | 
                                                    ||
| 1018 | 1018 | |
| 1019 | 1019 | /**  | 
                                                        
| 1020 | 1020 | * Send email to moderators notifying them the thread has been created or post added/edited.  | 
                                                        
| 1021 | + * @param DataObject $post  | 
                                                        |
| 1022 | + * @param DataObject $thread  | 
                                                        |
| 1021 | 1023 | */  | 
                                                        
| 1022 | 1024 |  	function notifyModerators($post, $thread, $starting_thread = false) { | 
                                                        
| 1023 | 1025 | $moderators = $this->Moderators();  | 
                                                        
@@ -1340,6 +1342,10 @@ discard block  | 
                                                    ||
| 1340 | 1342 | * TODO: Remove once API in core  | 
                                                        
| 1341 | 1343 | */  | 
                                                        
| 1342 | 1344 |  class Forum_DataQuery extends DataQuery { | 
                                                        
| 1345 | +  | 
                                                        |
| 1346 | + /**  | 
                                                        |
| 1347 | + * @param string $dataClass  | 
                                                        |
| 1348 | + */  | 
                                                        |
| 1343 | 1349 |  	function __construct($dataClass, $query) { | 
                                                        
| 1344 | 1350 | parent::__construct($dataClass);  | 
                                                        
| 1345 | 1351 | $this->query = $query;  | 
                                                        
@@ -337,7 +337,7 @@ discard block  | 
                                                    ||
| 337 | 337 | |
| 338 | 338 | /**  | 
                                                        
| 339 | 339 | * Get a list of Forum Categories  | 
                                                        
| 340 | - * @return DataList  | 
                                                        |
| 340 | + * @return boolean  | 
                                                        |
| 341 | 341 | */  | 
                                                        
| 342 | 342 |  	function getShowInCategories() { | 
                                                        
| 343 | 343 |  	 	$forumCategories = ForumCategory::get()->filter('ForumHolderID', $this->ID); | 
                                                        
@@ -484,7 +484,7 @@ discard block  | 
                                                    ||
| 484 | 484 | * it (keys: 'last_id', 'last_created')  | 
                                                        
| 485 | 485 | * @param int $lastVisit Unix timestamp of the last visit (GMT)  | 
                                                        
| 486 | 486 | * @param int $lastPostID ID of the last read post  | 
                                                        
| 487 | - * @param int $thread ID of the relevant topic (set to NULL for all  | 
                                                        |
| 487 | + * @param int $threadID ID of the relevant topic (set to NULL for all  | 
                                                        |
| 488 | 488 | * topics)  | 
                                                        
| 489 | 489 | * @return bool Returns TRUE if there are new posts available, otherwise  | 
                                                        
| 490 | 490 | * FALSE.  | 
                                                        
@@ -857,7 +857,7 @@ discard block  | 
                                                    ||
| 857 | 857 | /**  | 
                                                        
| 858 | 858 | * Return the GlobalAnnouncements from the individual forums  | 
                                                        
| 859 | 859 | *  | 
                                                        
| 860 | - * @return DataObjectSet  | 
                                                        |
| 860 | + * @return ArrayList  | 
                                                        |
| 861 | 861 | */  | 
                                                        
| 862 | 862 |  	function GlobalAnnouncements() { | 
                                                        
| 863 | 863 | //dump(ForumHolder::baseForumTable());  |