IONOS(search): Fix SearchComposer.php filtering logic

[printminion-co]

keep the $this->providers types

TODO

Test via

./occ config:app:set --value '["files","settings"]' --type array core unified_search.providers_allowed

Tests can be executed via

 composer run test -- tests/lib/Search/SearchComposerTest.php

• review
• e.g wait for fix(search): Fix SearchComposer.php filtering logic nextcloud/server#54120 to be accepted (cherry pick or merge)

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)

[Copilot]

Pull Request Overview

This PR fixes the filtering logic in SearchComposer.php to properly maintain the provider types and handler associations when filtering providers based on the 'unified_search.providers_allowed' configuration.

• Refactors the filterProviders method to modify provider arrays in place instead of returning a filtered array
• Updates method signature and documentation to reflect the change from returning filtered data to modifying instance variables
• Adds comprehensive test coverage for the filtering functionality

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
lib/private/Search/SearchComposer.php	Refactors filterProviders method to maintain provider types by filtering in place and also removing corresponding handlers
tests/lib/Search/SearchComposerTest.php	Adds comprehensive test suite covering provider filtering, sorting, and various edge cases

[printminion-co]

Cherry-picked commits from approved PR nextcloud#54120