Implement caching strategy for the API calls.

add time dependency injection as well.

Thanks for the issue. This makes sense to add! The current implementation will probably break existing plugins though. Not sure yet how to get this done in a backwards compatible way. One thing we could do is add a service to wrap the current plugin calls:

$provider->search($keyword, $this->pagerManager->findPage());

$photo = $provider->load($external_id);

This new service could then implement the caching so the plugins themselves don't need to worry about that. What do you think about that?

Well we could make a Major release?

Nonetheless this way a Plugin could still choose to not have caching. For example, for specific external media libraries you maybe don't want caching?

If a major release isn't possible we could look into it further.

another solution would be to have 2 different interfaces.

Any other change won't have any effect from our point of view.
since extending classes still call parent::load for example, it doesn't change

I've already worked a intermediary service for caching.

Not yet tested though. I suppose this is the way you want it?

tested and fixed some issues.

Caching is working properly.

Todo check diff of main branch

All seems good. Putting to review.

Nice work! Added some changes before merging to use a single cache backend and renamed the cache wrapper services, but overall it worked perfectly. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.