I believe this is uncontroversial. The constructor or the top-level
base class is public. Subclasses are not even allowed to change that.
So what this patch effectively does is basically documentation, but it
does not change anything.
Change-Id: Icd5da81de284bc4bf382bfdfd33dab428f1fab09
13 files changed:
* @param string $tagFilter
* @param array $conds
*/
* @param string $tagFilter
* @param array $conds
*/
- function __construct( $historyPage, $year = '', $month = '', $tagFilter = '', $conds = [] ) {
+ public function __construct(
+ HistoryAction $historyPage,
+ $year = '',
+ $month = '',
+ $tagFilter = '',
+ array $conds = []
+ ) {
parent::__construct( $historyPage->getContext() );
$this->historyPage = $historyPage;
$this->tagFilter = $tagFilter;
parent::__construct( $historyPage->getContext() );
$this->historyPage = $historyPage;
$this->tagFilter = $tagFilter;
* @param IContextSource|null $context
* @param FormOptions $opts
*/
* @param IContextSource|null $context
* @param FormOptions $opts
*/
- function __construct( IContextSource $context = null, FormOptions $opts ) {
+ public function __construct( IContextSource $context = null, FormOptions $opts ) {
parent::__construct( $context );
$this->RCMaxAge = $this->getConfig()->get( 'ActiveUserDays' );
parent::__construct( $context );
$this->RCMaxAge = $this->getConfig()->get( 'ActiveUserDays' );
- function __construct( $page, $conds, Language $langObj = null ) {
+ public function __construct( $page, $conds, Language $langObj = null ) {
parent::__construct( $page->getContext() );
$this->mIndexField = 'am_title';
$this->mPage = $page;
parent::__construct( $page->getContext() );
$this->mIndexField = 'am_title';
$this->mPage = $page;
* @param SpecialPage $page
* @param array $conds
*/
* @param SpecialPage $page
* @param array $conds
*/
- function __construct( $page, $conds ) {
+ public function __construct( $page, $conds ) {
$this->page = $page;
$this->conds = $conds;
$this->mDefaultDirection = IndexPager::DIR_DESCENDING;
$this->page = $page;
$this->conds = $conds;
$this->mDefaultDirection = IndexPager::DIR_DESCENDING;
*/
protected $templateParser;
*/
protected $templateParser;
- function __construct( IContextSource $context, array $options ) {
+ public function __construct( IContextSource $context, array $options ) {
parent::__construct( $context );
$msgs = [
parent::__construct( $context );
$msgs = [
*/
protected $mNavigationBar;
*/
protected $mNavigationBar;
- function __construct( IContextSource $context, $target, $namespace = false ) {
+ public function __construct( IContextSource $context, $target, $namespace = false ) {
parent::__construct( $context );
$msgs = [ 'deletionlog', 'undeleteviewlink', 'diff' ];
foreach ( $msgs as $msg ) {
parent::__construct( $context );
$msgs = [ 'deletionlog', 'undeleteviewlink', 'diff' ];
foreach ( $msgs as $msg ) {
protected $mTableName = 'image';
protected $mTableName = 'image';
- function __construct( IContextSource $context, $userName = null, $search = '',
+ public function __construct( IContextSource $context, $userName = null, $search = '',
$including = false, $showAll = false
) {
$this->setContext( $context );
$including = false, $showAll = false
) {
$this->setContext( $context );
/** @var array */
public $mConds;
/** @var array */
public $mConds;
- function __construct( SpecialMergeHistory $form, $conds, Title $source, Title $dest ) {
+ public function __construct( SpecialMergeHistory $form, $conds, Title $source, Title $dest ) {
$this->mForm = $form;
$this->mConds = $conds;
$this->title = $source;
$this->mForm = $form;
$this->mConds = $conds;
$this->title = $source;
* @param IContextSource $context
* @param FormOptions $opts
*/
* @param IContextSource $context
* @param FormOptions $opts
*/
- function __construct( IContextSource $context, FormOptions $opts ) {
+ public function __construct( IContextSource $context, FormOptions $opts ) {
parent::__construct( $context );
$this->opts = $opts;
parent::__construct( $context );
$this->opts = $opts;
- function __construct( $form, FormOptions $opts ) {
+ public function __construct( $form, FormOptions $opts ) {
parent::__construct( $form->getContext() );
$this->mForm = $form;
$this->opts = $opts;
parent::__construct( $form->getContext() );
$this->mForm = $form;
$this->opts = $opts;
* @param bool $noredirect
* @param LinkRenderer $linkRenderer
*/
* @param bool $noredirect
* @param LinkRenderer $linkRenderer
*/
- function __construct( $form, $conds, $type, $level, $namespace,
+ public function __construct( $form, $conds, $type, $level, $namespace,
$sizetype, $size, $indefonly, $cascadeonly, $noredirect,
LinkRenderer $linkRenderer
) {
$sizetype, $size, $indefonly, $cascadeonly, $noredirect,
LinkRenderer $linkRenderer
) {
- function __construct( $form, $conds, $type, $level, $namespace,
+ public function __construct( $form, $conds, $type, $level, $namespace,
$sizetype = '', $size = 0
) {
$this->mForm = $form;
$sizetype = '', $size = 0
) {
$this->mForm = $form;
* @param bool|null $including Whether this page is being transcluded in
* another page
*/
* @param bool|null $including Whether this page is being transcluded in
* another page
*/
- function __construct( IContextSource $context = null, $par = null, $including = null ) {
+ public function __construct( IContextSource $context = null, $par = null, $including = null ) {
if ( $context ) {
$this->setContext( $context );
}
if ( $context ) {
$this->setContext( $context );
}