logging: Correct $logId default value in LogPager.php
authorAlangi Derick <alangiderick@gmail.com>
Mon, 3 Dec 2018 12:38:03 +0000 (13:38 +0100)
committerUmherirrender <umherirrender_de.wp@web.de>
Wed, 2 Jan 2019 11:58:32 +0000 (11:58 +0000)
$logId holds the log entry ID of some specific log and the datatype
is an integer (int) which is correct but looking at the constructor,
it does take a bool (false) which contradicts the docs above. So per
@Thiemo's advice after review of prev PS, it would be better to replace
false with 0 and leave the docs untouched.

Change-Id: Ieceb79a6cbaa248e1b7a7d3c1d51eb26bb3a2d84

includes/logging/LogPager.php

index b31f165..32afa37 100644 (file)
@@ -70,7 +70,7 @@ class LogPager extends ReverseChronologicalPager {
         */
        public function __construct( $list, $types = [], $performer = '', $title = '',
                $pattern = false, $conds = [], $year = false, $month = false, $day = false,
-               $tagFilter = '', $action = '', $logId = false
+               $tagFilter = '', $action = '', $logId = 0
        ) {
                parent::__construct( $list->getContext() );
                $this->mConds = $conds;