Filters/ExactMatch: deprecate getBlacklist() and getWhitelist() methods

[jrfnl]

• Deprecate two abstract methods which have needlessly cultural insensitive method names.
• Introduce - temporarily non-abstract - replacement methods for both: getBlockedFiles() ++getDisallowedFiles()++ and getAllowedFiles().

This is a soft deprecation (changelog only) as throwing a deprecation notice for these methods would be unnecessarily disruptive for end-users.

In version 4.0, the deprecated methods will be removed and the replacement methods will be made abstract and therefore required.

Introducing the replacement methods at the same time as the deprecation will allow external standards which contain custom filters and other packages offering custom filters, to make the filters cross-version compatible with PHPCS 3.9.0+ as well as 4.x.

When both methods are available, I propose for the replacement methods to take precedence over the deprecated methods.

The impact of this deprecation is expected to be low.

[stronk7]

+1 for this, just suggesting that (non-native English here!) I find allow/deny more balanced than allow/block.

[jrfnl]

+1 for this, just suggesting that (non-native English here!) I find allow/deny more balanced than allow/block.

@stronk7 Thanks for your input. I'm a non-native English speaker myself, so I'm not sure what would be best in this case. I lean towards "block" as I associate "deny" with lying/truth gathering, but that is probably just me.

I'd like to hear some more opinions. @GaryJones Care to pitch in ?

For context: this how the old names were being used:

• getWhitelist(): retrieve a list of files which should be accepted for the scan.
• getBlacklist(): retrieve a list of files which should not be accepted for the scan.

(with accept() being the name of the Iterator method these lists are fed into)

---

Note: depending on what is decided, PR #202 may need to be redone/may need a follow-up and open PR #203 may need to be updated.

[GaryJones]

There's not too much in it from an English point of view. Any attempt to discern a difference in definition would be sublte at best.

Looking at 202 quickly, it seems like the property and methods could be more descriptive along the lines of getIncludedFilesPaths() and getExcludedFilesPaths(). Bit more wordy but a bit more descriptive?

[jrfnl]

Looking at 202 quickly, it seems like the property and methods could be more descriptive along the lines of getIncludedFilesPaths() and getExcludedFilesPaths(). Bit more wordy but a bit more descriptive?

@GaryJones Thanks for your input. I'm not sure those names would be right though as the accept() still has additional logic which uses the input from these method to determine whether a file should be included or excluded.

Darn... naming is hard ;-)