pagers: Fix and add missing @param documentation
authorThiemo Kreuz <thiemo.kreuz@wikimedia.de>
Tue, 19 Feb 2019 14:27:25 +0000 (15:27 +0100)
committerUmherirrender <umherirrender_de.wp@web.de>
Tue, 19 Feb 2019 18:50:07 +0000 (18:50 +0000)
Sometimes the code expects a specific subclass as it calls a specific
method that only exists in this subclass. If this is not the case, all
the code needs to know is that it got some special page.

Some of these constructors should be changed to expect a IContextSource
instead of a special page. But this is for another patch. Here in this
patch I intentionally touch documentation only.

Change-Id: I33bae9641a8871f359edccacf88f09cfe8e83139

includes/specials/pagers/NewPagesPager.php
includes/specials/pagers/ProtectedPagesPager.php

index 2b094b1..d03401d 100644 (file)
  */
 class NewPagesPager extends ReverseChronologicalPager {
 
-       // Stored opts
+       /**
+        * @var FormOptions
+        */
        protected $opts;
 
        /**
-        * @var HTMLForm
+        * @var SpecialNewpages
         */
        protected $mForm;
 
+       /**
+        * @param SpecialNewpages $form
+        * @param FormOptions $opts
+        */
        public function __construct( $form, FormOptions $opts ) {
                parent::__construct( $form->getContext() );
                $this->mForm = $form;
index f457d2f..fbe69c4 100644 (file)
@@ -32,7 +32,7 @@ class ProtectedPagesPager extends TablePager {
        private $linkRenderer;
 
        /**
-        * @param SpecialProtectedpages $form
+        * @param SpecialPage $form
         * @param array $conds
         * @param string $type
         * @param string $level