Review all instances of whitelist/blacklist

[joshmoore]

This does not try to maintain backwards compatibility
since the related methods and classes are internal. The
likeliest impact is that of the change in property name.
A redirecting property could be added here, but a nicer
choice is likely to automatically upgrade in omero-py.

see: ome/bioformats#3580

[pwalczysko]

failed travis

[joshmoore]

Now omero-test-infra is failing. This touches Ice and is going to be harder than expected:

(base) /opt/omero-build/omero-blitz $git grep -E "(black|white)list"
src/main/slice/omero/FS.ice:         * @param whitelist list of files or extensions of interest (Ice::StringSeq).
src/main/slice/omero/FS.ice:         * @param blacklist list of directories, files or extensions that are not of interest (Ice::StringSeq).
src/main/slice/omero/FS.ice:                                Ice::StringSeq whitelist,
src/main/slice/omero/FS.ice:                                Ice::StringSeq blacklist,

cc: @sbesson

[sbesson]

Very unfortunate. So even if the type/signature is identical, the name of the parameters in the .ice generating classes would need to be jointly renamed? If so, I wonder if this is something we want to test as we are about to cut release of the OMERO java components (and maybe deprecating the old method to prevent other breakages?)

[joshmoore]

Yes to all. We could also deprecate the fields in Ice. Not sure we want add the new fields at this point or not.