Completed
Pull Request — master (#203)
by
unknown
40:36
created
src/Controllers/ForumMemberProfile.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,6 @@
 block discarded – undo
25 25
 use SilverStripe\Security\Member;
26 26
 use SilverStripe\Logging\Log;
27 27
 use SilverStripe\Security\Group;
28
-use SilverStripe\Core\Object;
29 28
 use SilverStripe\Forms\TextField;
30 29
 use SilverStripe\Forms\RequiredFields;
31 30
 use SilverStripe\Control\Director;
Please login to merge, or discard this patch.
src/Models/ForumThreadSubscription.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -7,7 +7,6 @@
 block discarded – undo
7 7
 use SilverStripe\Core\Config\Config;
8 8
 use SilverStripe\Core\Convert;
9 9
 use SilverStripe\ORM\DataObject;
10
-use SilverStripe\ORM\DB;
11 10
 use SilverStripe\ORM\FieldType\DBDatetime;
12 11
 use SilverStripe\Security\Member;
13 12
 
Please login to merge, or discard this patch.
src/Models/Post.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -355,7 +355,7 @@  discard block
 block discarded – undo
355 355
     /**
356 356
      * Returns a ban link
357 357
      *
358
-     * @return bool|string
358
+     * @return string|false
359 359
      */
360 360
     public function BanLink()
361 361
     {
@@ -372,7 +372,7 @@  discard block
 block discarded – undo
372 372
     /**
373 373
      * Returns a ghost link
374 374
      *
375
-     * @return bool|string
375
+     * @return string|false
376 376
      */
377 377
     public function GhostLink()
378 378
     {
Please login to merge, or discard this patch.
Unused Use Statements   -3 removed lines patch added patch discarded remove patch
@@ -15,9 +15,6 @@
 block discarded – undo
15 15
 use SilverStripe\Control\Controller;
16 16
 use SilverStripe\Security\SecurityToken;
17 17
 use SilverStripe\ORM\DataObject;
18
-use SilverStripe\Core\Convert;
19
-use SilverStripe\Control\HTTPRequest;
20
-use SilverStripe\Assets\File;
21 18
 
22 19
 /**
23 20
  * Forum Post Object. Contains a single post by the user. A thread is generated
Please login to merge, or discard this patch.
src/Pages/Forum.php 1 patch
Doc Comments   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -500,7 +500,7 @@  discard block
 block discarded – undo
500 500
 
501 501
     /**
502 502
      * Returns the Topics (the first Post of each Thread) for this Forum
503
-     * @return DataList
503
+     * @return PaginatedList|null
504 504
      */
505 505
     public function getTopics()
506 506
     {
@@ -707,7 +707,7 @@  discard block
 block discarded – undo
707 707
      *
708 708
      * Designed to be called via AJAX so return true / false
709 709
      *
710
-     * @return bool
710
+     * @return null|false
711 711
      */
712 712
     public function unsubscribe(HTTPRequest $request)
713 713
     {
@@ -1241,7 +1241,7 @@  discard block
 block discarded – undo
1241 1241
     /**
1242 1242
      * Return the Forbidden Words in this Forum
1243 1243
      *
1244
-     * @return Text
1244
+     * @return string
1245 1245
      */
1246 1246
     public function getForbiddenWords()
1247 1247
     {
@@ -1547,6 +1547,9 @@  discard block
 block discarded – undo
1547 1547
  */
1548 1548
 class Forum_DataQuery extends DataQuery
1549 1549
 {
1550
+    /**
1551
+     * @param string $dataClass
1552
+     */
1550 1553
     public function __construct($dataClass, $query)
1551 1554
     {
1552 1555
         parent::__construct($dataClass);
Please login to merge, or discard this patch.
src/Pages/ForumHolder.php 1 patch
Doc Comments   +6 added lines, -5 removed lines patch added patch discarded remove patch
@@ -241,7 +241,7 @@  discard block
 block discarded – undo
241 241
     /**
242 242
      * Get the number of total posts
243 243
      *
244
-     * @return int Returns the number of posts
244
+     * @return string Returns the number of posts
245 245
      */
246 246
     public function getNumPosts()
247 247
     {
@@ -259,7 +259,7 @@  discard block
 block discarded – undo
259 259
     /**
260 260
      * Get the number of total topics (threads)
261 261
      *
262
-     * @return int Returns the number of topics (threads)
262
+     * @return string Returns the number of topics (threads)
263 263
      */
264 264
     public function getNumTopics()
265 265
     {
@@ -277,7 +277,7 @@  discard block
 block discarded – undo
277 277
     /**
278 278
      * Get the number of distinct authors
279 279
      *
280
-     * @return int Returns the number of distinct authors
280
+     * @return string Returns the number of distinct authors
281 281
      */
282 282
     public function getNumAuthors()
283 283
     {
@@ -377,7 +377,7 @@  discard block
 block discarded – undo
377 377
 
378 378
     /**
379 379
      * Get a list of Forum Categories
380
-     * @return DataList
380
+     * @return boolean
381 381
      */
382 382
     public function getShowInCategories()
383 383
     {
@@ -542,8 +542,9 @@  discard block
 block discarded – undo
542 542
      *                    it (keys: 'last_id', 'last_created')
543 543
      * @param int $lastVisit Unix timestamp of the last visit (GMT)
544 544
      * @param int $lastPostID ID of the last read post
545
-     * @param int $thread ID of the relevant topic (set to NULL for all
545
+     * @param integer $threadID ID of the relevant topic (set to NULL for all
546 546
      *                     topics)
547
+     * @param integer $forumID
547 548
      * @return bool Returns TRUE if there are new posts available, otherwise
548 549
      *              FALSE.
549 550
      */
Please login to merge, or discard this patch.