Apply role based search filters in existing endpoints

[litvinovg]

What does this pull request do?

A follow-up PR for recently merged PR-433 and PR-434
to apply role based search filters in existing endpoints.

What's new?

Applied search filters in IndividualListController, AutocompleteController
Fixed unsafe DataAutoCompleteController, perform authorization checks for each returned result.
Refactored SearchFiltering methods to remove dependency on not needed VitroRequest parameter
Applied search filters to DefaultObjectPropertyFormGeneratory
Removed unused import in DefaultObjectPropertyFormGenerator

How should this be tested?

A description of what steps someone could take to:

• Apply test data provided in PR Role based search results filtering #434
• Try IndividualListController, Autocompletecontroller, DataAutoComplete controllers, DefaultObjectPropertyFormGenerator

Interested parties

@VIVO-project/vivo-committers

[chenejac]

@litvinovg thanks for this PR.

I have posted a couple of comments in the case you have any idea how to additionally improve efficiency of the implementation. Basically, it is about invoking current roles multiple time in some cases. It is the most important that permission set is loaded just one (and it is the case at the moment), therefore I am approving this PR.

[wwtamu]

Are there any other controllers that require updates?

https://github.com/search?q=repo%3Avivo-project%2FVitro+WebServlet&type=code&p=1

https://github.com/search?q=repo%3Avivo-project%2FVIVO%20WebServlet&type=code

[litvinovg]

Are there any other controllers that require updates?

https://github.com/search?q=repo%3Avivo-project%2FVitro+WebServlet&type=code&p=1

https://github.com/search?q=repo%3Avivo-project%2FVIVO%20WebServlet&type=code

I didn't find any.