Change Details

While reviewing the autofix I have seen a place where the parenthesis could be removed from the autofix. https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/CategoryTagSorter/+/564304/1..2/CategoryTagSorter_body.php If that makes it to complex to detect than I have no problem to make this invalid. Just want leave a note.

While reviewing the autofix I have seen a place where the parenthesis could be removed from the autofix. ```lang=php, name=Before if ( is_null( $parser->getUser()->getOption( self::PREF_NAME ) ) ) { ``` ```lang=php, counterexample, name=Current auto-fix if ( ( $parser->getUser()->getOption( self::PREF_NAME ) ) === null ) { ``` ```lang=php, name=Expected if ( $parser->getUser()->getOption( self::PREF_NAME ) === null ) { ``` https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/CategoryTagSorter/+/564304/1..2/CategoryTagSorter_body.php If that makes it to complex to detect than I have no problem to make this invalid. Just want leave a note.

While reviewing the autofix I have seen a place where the parenthesis could be removed from the autofix. ```lang=php, name=Before if ( is_null( $parser->getUser()->getOption( self::PREF_NAME ) ) ) { ``` ```lang=php, counterexample, name=Current auto-fix if ( ( $parser->getUser()->getOption( self::PREF_NAME ) ) === null ) { ``` ```lang=php, name=Expected if ( $parser->getUser()->getOption( self::PREF_NAME ) === null ) { ``` https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/CategoryTagSorter/+/564304/1..2/CategoryTagSorter_body.php If that makes it to complex to detect than I have no problem to make this invalid. Just want leave a note.