Set cache driver & path on the bound Storyblok\Client instance

[codyworthen]

There are multiple places in Storyblok\Client that call a private method called isCache(): bool and if the private property $cacheis not set via the publicsetCachemethod, then$client->cache` will be null.

storyblok/storyblok-php-client says that the cache needs to be set if caching is going to be used here

[codyworthen]

@RicLeP @roberto-butti I was wondering if I could get some feedback on this. Thanks

[roberto-butti]

From the Storyblok client perspective, the cache can be set via the setCache method.

$client->setCache('filesystem',
    [
        'path' => 'cache',
        'default_lifetime' => 3600
    ]);

You can use the file system or a database for example.
Because the Laravel packages uses under the hood the PHP client, the Laravel packages should set the cache (or via config or exposing a method).

I would like to listen to the thoughts of @RicLeP the maintainer of this package. Open to offer my support.

[RicLeP]

I like this, I think we should be making use of the cache driver.

Couple of thoughts

1. should we let the lifetime be configured too?
2. should be config keys be prefixed with sb_ or similar to differentiate them from the Laravel caching the package uses and help identify this is configuring the underlying Storyblok client?