Web-648: OcTableFiles enable multiple highlighting using selection

[lookacat]

Description

See owncloud/web#5164

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Technical debt
• Tests

Checklist:

• Code changes
• Unit tests added
• Acceptance tests added
• Documentation added/updated

[update-docs]

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

[kulmann]

Cool, didn't expect the changes to be so minimal. Awesome! Added some ideas for improvement.

[kulmann]

As discussed in a call:

• the highlight event from OcTable needs to be used to update the selection model. don't emit showDetails anymore.
• in oC Web we'll watch the selection model to decide whether or not the right sidebar needs to be shown
• the show details button needs a new implementation: stop the propagation of the click event and add a new method for it, which resets the selection model to the resource id of the clicked row/button (in oC Web this results in a right sidebar for the clicked row)

[lookacat]

I would only change the unittests so they work for now since the highlighting logic will changed again soon.

[kulmann]

LGTM 👍

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

16.7% Coverage
0.0% Duplication