Completed
Push — 0.8 ( 37e1d3...8a42a0 )
by
unknown
30s queued 27s
created
code/controllers/ForumMemberProfile.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -556,7 +556,7 @@
 block discarded – undo
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() {
Please login to merge, or discard this patch.
code/formfields/CheckableOption.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -3,6 +3,9 @@
 block discarded – undo
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);
Please login to merge, or discard this patch.
code/formfields/ForumCountryDropdownField.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -28,6 +28,10 @@
 block discarded – undo
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
Please login to merge, or discard this patch.
code/model/ForumThread.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -143,7 +143,7 @@
 block discarded – undo
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;
Please login to merge, or discard this patch.
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|null
403 403
 	 */
404 404
 	function getTopics() {
405 405
 		// Get a list of Posts
@@ -591,7 +591,7 @@  discard block
 block discarded – undo
591 591
 	 *
592 592
 	 * Designed to be called via AJAX so return true / false
593 593
 	 *
594
-	 * @return bool
594
+	 * @return null|false
595 595
 	 */
596 596
 	function unsubscribe(SS_HTTPRequest $request) {
597 597
 		$member = Member::currentUser();
@@ -713,7 +713,7 @@  discard block
 block discarded – undo
713 713
 	/**
714 714
 	 * Get posts to display. This method assumes an URL parameter "ID" which contains the thread ID.
715 715
 	 * @param string sortDirection The sort order direction, either ASC for ascending (default) or DESC for descending
716
-	 * @return DataObjectSet Posts
716
+	 * @return PaginatedList Posts
717 717
 	 */
718 718
 	function Posts($sortDirection = "ASC") {
719 719
 		$numPerPage = Forum::$posts_per_page;
@@ -764,7 +764,7 @@  discard block
 block discarded – undo
764 764
 	/**
765 765
 	 * Get the usable BB codes
766 766
 	 *
767
-	 * @return DataObjectSet Returns the usable BB codes
767
+	 * @return ArrayList Returns the usable BB codes
768 768
 	 * @see BBCodeParser::usable_tags()
769 769
 	 */
770 770
 	function BBTags() {
@@ -1046,6 +1046,8 @@  discard block
 block discarded – undo
1046 1046
 
1047 1047
 	/**
1048 1048
 	 * Send email to moderators notifying them the thread has been created or post added/edited.
1049
+	 * @param DataObject $post
1050
+	 * @param DataObject $thread
1049 1051
 	 */
1050 1052
 	function notifyModerators($post, $thread, $starting_thread = false) {
1051 1053
 		$moderators = $this->Moderators();
@@ -1370,6 +1372,10 @@  discard block
 block discarded – undo
1370 1372
  * TODO: Remove once API in core
1371 1373
  */
1372 1374
 class Forum_DataQuery extends DataQuery {
1375
+
1376
+	/**
1377
+	 * @param string $dataClass
1378
+	 */
1373 1379
 	function __construct($dataClass, $query) {
1374 1380
 		parent::__construct($dataClass);
1375 1381
 		$this->query = $query;
Please login to merge, or discard this patch.
code/pagetypes/ForumHolder.php 1 patch
Doc Comments   +6 added lines, -5 removed lines patch added patch discarded remove patch
@@ -219,7 +219,7 @@  discard block
 block discarded – undo
219 219
 	/**
220 220
 	 * Get the number of total posts
221 221
 	 *
222
-	 * @return int Returns the number of posts
222
+	 * @return string Returns the number of posts
223 223
 	 */
224 224
 	public function getNumPosts() {
225 225
 			$sqlQuery = new SQLQuery();
@@ -236,7 +236,7 @@  discard block
 block discarded – undo
236 236
 	/**
237 237
 	 * Get the number of total topics (threads)
238 238
 	 *
239
-	 * @return int Returns the number of topics (threads)
239
+	 * @return string Returns the number of topics (threads)
240 240
 	 */
241 241
 	function getNumTopics() {
242 242
 		$sqlQuery = new SQLQuery();
@@ -253,7 +253,7 @@  discard block
 block discarded – undo
253 253
 	/**
254 254
 	 * Get the number of distinct authors
255 255
 	 *
256
-	 * @return int Returns the number of distinct authors
256
+	 * @return string Returns the number of distinct authors
257 257
 	 */
258 258
 	public function getNumAuthors() {
259 259
 		$sqlQuery = new SQLQuery();
@@ -348,7 +348,7 @@  discard block
 block discarded – undo
348 348
 
349 349
 	/**
350 350
 	 * Get a list of Forum Categories
351
-	 * @return DataList
351
+	 * @return boolean
352 352
 	 */
353 353
 	function getShowInCategories() {
354 354
 	 	$forumCategories = ForumCategory::get()->filter('ForumHolderID', $this->ID);
@@ -495,8 +495,9 @@  discard block
 block discarded – undo
495 495
 	 *                    it (keys: 'last_id', 'last_created')
496 496
 	 * @param int $lastVisit Unix timestamp of the last visit (GMT)
497 497
 	 * @param int $lastPostID ID of the last read post
498
-	 * @param int $thread ID of the relevant topic (set to NULL for all
498
+	 * @param integer $threadID ID of the relevant topic (set to NULL for all
499 499
 	 *                     topics)
500
+	 * @param integer $forumID
500 501
 	 * @return bool Returns TRUE if there are new posts available, otherwise
501 502
 	 *              FALSE.
502 503
 	 */
Please login to merge, or discard this patch.