Completed
Push — develop ( a1a3a0 )
by
unknown
02:19
created
code/pagetypes/Forum.php 1 patch
Doc Comments   +13 added lines, -7 removed lines patch added patch discarded remove patch
@@ -354,7 +354,7 @@  discard block
 block discarded – undo
354 354
 	/**
355 355
 	 * Get the number of total topics (threads) in this Forum
356 356
 	 *
357
-	 * @return int Returns the number of topics (threads)
357
+	 * @return string Returns the number of topics (threads)
358 358
 	 */
359 359
 	function getNumTopics() {
360 360
 		$sqlQuery = new SQLQuery();
@@ -369,7 +369,7 @@  discard block
 block discarded – undo
369 369
 	/**
370 370
 	 * Get the number of total posts
371 371
 	 *
372
-	 * @return int Returns the number of posts
372
+	 * @return string Returns the number of posts
373 373
 	 */
374 374
 	function getNumPosts() {
375 375
 		$sqlQuery = new SQLQuery();
@@ -385,7 +385,7 @@  discard block
 block discarded – undo
385 385
 	/**
386 386
 	 * Get the number of distinct Authors
387 387
 	 *
388
-	 * @return int
388
+	 * @return string
389 389
 	 */
390 390
 	function getNumAuthors() {
391 391
 		$sqlQuery = new SQLQuery();
@@ -399,7 +399,7 @@  discard block
 block discarded – undo
399 399
 
400 400
 	/**
401 401
 	 * Returns the Topics (the first Post of each Thread) for this Forum
402
-	 * @return DataList
402
+	 * @return PaginatedList
403 403
 	 */
404 404
 	public function getTopics() {
405 405
 		// Get a list of Posts
@@ -598,7 +598,7 @@  discard block
 block discarded – undo
598 598
 	 *
599 599
 	 * Designed to be called via AJAX so return true / false
600 600
 	 *
601
-	 * @return bool
601
+	 * @return null|false
602 602
 	 */
603 603
 	public function unsubscribe(SS_HTTPRequest $request) {
604 604
 		$member = Member::currentUser();
@@ -720,7 +720,7 @@  discard block
 block discarded – undo
720 720
 	/**
721 721
 	 * Get posts to display. This method assumes an URL parameter "ID" which contains the thread ID.
722 722
 	 * @param string sortDirection The sort order direction, either ASC for ascending (default) or DESC for descending
723
-	 * @return DataObjectSet Posts
723
+	 * @return PaginatedList Posts
724 724
 	 */
725 725
 	function Posts($sortDirection = "ASC") {
726 726
 		$numPerPage = Forum::$posts_per_page;
@@ -771,7 +771,7 @@  discard block
 block discarded – undo
771 771
 	/**
772 772
 	 * Get the usable BB codes
773 773
 	 *
774
-	 * @return DataObjectSet Returns the usable BB codes
774
+	 * @return ArrayList Returns the usable BB codes
775 775
 	 * @see BBCodeParser::usable_tags()
776 776
 	 */
777 777
 	function BBTags() {
@@ -1053,6 +1053,8 @@  discard block
 block discarded – undo
1053 1053
 
1054 1054
 	/**
1055 1055
 	 * Send email to moderators notifying them the thread has been created or post added/edited.
1056
+	 * @param DataObject $post
1057
+	 * @param DataObject $thread
1056 1058
 	 */
1057 1059
 	function notifyModerators($post, $thread, $starting_thread = false) {
1058 1060
 		$moderators = $this->Moderators();
@@ -1377,6 +1379,10 @@  discard block
 block discarded – undo
1377 1379
  * TODO: Remove once API in core
1378 1380
  */
1379 1381
 class Forum_DataQuery extends DataQuery {
1382
+
1383
+	/**
1384
+	 * @param string $dataClass
1385
+	 */
1380 1386
 	function __construct($dataClass, $query) {
1381 1387
 		parent::__construct($dataClass);
1382 1388
 		$this->query = $query;
Please login to merge, or discard this patch.