Values can be floats, like 1.5 days, etc. Cast instead to a float.
Bug: T175965
Change-Id: I14ba792f1cd435f89b2e09067b0a0e894a0a2557
abstract function getDefaultLimit();
+ /**
+ * Get the default value of the number of days to display when loading
+ * the result set.
+ * Supports fractional values, and should be cast to a float.
+ *
+ * @return float
+ */
abstract function getDefaultDays();
}
*/
public function getDefaultOptions() {
$opts = parent::getDefaultOptions();
- $user = $this->getUser();
- $opts->add( 'days', $user->getIntOption( 'rcdays' ), FormOptions::FLOAT );
- $opts->add( 'limit', $user->getIntOption( 'rclimit' ) );
+ $opts->add( 'days', $this->getDefaultDays(), FormOptions::FLOAT );
+ $opts->add( 'limit', $this->getDefaultLimit() );
$opts->add( 'from', '' );
$opts->add( 'categories', '' );
}
function getDefaultDays() {
- return $this->getUser()->getIntOption( 'rcdays' );
+ return floatval( $this->getUser()->getOption( 'rcdays' ) );
}
}
*/
public function getDefaultOptions() {
$opts = parent::getDefaultOptions();
- $user = $this->getUser();
- $opts->add( 'days', $user->getOption( 'watchlistdays' ), FormOptions::FLOAT );
- $opts->add( 'limit', $user->getIntOption( 'wllimit' ), FormOptions::INT );
+ $opts->add( 'days', $this->getDefaultDays(), FormOptions::FLOAT );
+ $opts->add( 'limit', $this->getDefaultLimit(), FormOptions::INT );
return $opts;
}
}
function getDefaultDays() {
- return $this->getUser()->getIntOption( 'watchlistdays' );
+ return floatval( $this->getUser()->getOption( 'watchlistdays' ) );
}
}