[interfaces/json-rpc] Allow VideoLibrary.Clean by directory

[CastagnaIT]

Description

Add directory parameter to VideoLibrary.Clean, as done on VideoLibrary.Scan

Motivation and Context

there are two ways to remove items that no longer exist from the Kodi library
call single jsonRPC's (e.g. VideoLibrary.RemoveEpisode) for each file or clean the library

The first method (each call per file) is intensive when you have to delete hundreds of files, and have side effect to show loading screen (e.g. if you are doing the operation while watching a video)

The second method you do only one call, at the end of your addon operations, is very fast and easy to do because you do not need a separate job to collect the list of paths to remove.

Why add directory parameter:

• Very fast library cleaning on big video databases
• Avoid removing items from other addons
• Avoid side effects with "non-standard" addons (like Jellyfin causes the removal of the entire library, issue: Netflix addon deletes all my Kodi library, includind libraries from another addon CastagnaIT/plugin.video.netflix#888)

How Has This Been Tested?

Tested on windows 10 with Jellyfin library items and netflix library items at same time
When i set VideoLibrary.Clean to netflix folder check only the netflix kodi library path

Another test is done on the issue referenced before, with nf+samba library source

Screenshots (if appropriate):

Types of change

• Bug fix (non-breaking change which fixes an issue)
• Clean up (non-breaking change which removes non-working, unmaintained functionality)
• Improvement (non-breaking change which improves existing functionality)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that will cause existing functionality to change)
• Cosmetic change (non-breaking change that doesn't touch code)
• None of the above (please explain below)

Checklist:

• My code follows the Code Guidelines of this project
• My change requires a change to the documentation, either Doxygen or wiki
• I have updated the documentation accordingly
• I have read the Contributing document
• I have added tests to cover my change
• All new and existing tests passed

[Montellese]

Looks good apart from the minors.

[Montellese]

Thanks for the changes. jenkins build this please

[dagwieers]

Will this work also for plugin://-type locations?
That would be a requirement for Kodi Library integrations from add-ons.

[CastagnaIT]

I have not tested i have no addon ready to this,
but theoretically it should work because the directory sent in the param is searched directly in the kodi database
if someone has the opportunity to confirm