Thijs/wiki.techinc.nl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge "WLFilters: Respect default values"

Browse source
This commit is contained in:
jenkins-bot 2017-09-08 17:31:27 +00:00 committed by Gerrit Code Review
commit df0c0f52e3
1 changed files with 16 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
includes/changes/ChangesListBooleanFilter.php
View file

@ -235,10 +235,13 @@ class ChangesListBooleanFilter extends ChangesListFilter {
* @inheritDoc
*/
public function isSelected( FormOptions $opts ) {
return !$opts[ $this->getName() ] &&
array_filter( $this->getSiblings(), function ( $sibling ) use ( $opts ) {
return $opts[ $sibling->getName() ];
} );
return !$this->getValue( $opts ) &&
array_filter(
$this->getSiblings(),
function ( ChangesListBooleanFilter $sibling ) use ( $opts ) {
return $sibling->getValue( $opts );
}
);
}
/**
@ -251,6 +254,14 @@ class ChangesListBooleanFilter extends ChangesListFilter {
return false;
}
return $opts[ $this->getName() ] === $this->activeValue;
return $this->getValue( $opts ) === $this->activeValue;
}
/**
* @param FormOptions $opts
* @return bool The current value of this filter according to $opts but coerced to boolean
*/
public function getValue( FormOptions $opts ) {
return (bool)$opts[ $this->getName() ];
}
}