Added support for cache chains with multiple cache providers #19

arjanvdbos · 2017-03-30T11:49:33Z

This change makes it possible to configure cache chains of existing cache providers.

…th PHP < 7.x

coveralls · 2017-03-30T11:57:25Z

Coverage increased (+6.4%) to 46.113% when pulling 98a0d6a on arjanvdbos:add-cache-chain-support into ab97e2e on DASPRiD:master.

coveralls · 2017-03-30T12:02:26Z

Coverage increased (+6.4%) to 46.113% when pulling 1a81043 on arjanvdbos:add-cache-chain-support into ab97e2e on DASPRiD:master.

asgrim

Looks clean enough, don't think there's any BC breaks here. Wonder if it might be useful to add an example usage to the example config?

coveralls · 2017-03-31T15:10:51Z

Coverage increased (+6.4%) to 46.113% when pulling 682bc9d on arjanvdbos:add-cache-chain-support into ab97e2e on DASPRiD:master.

arjanvdbos · 2017-03-31T15:11:07Z

Thanks for your feedback @asgrim, i've added an example to full-config.php. I'm looking forward to any other comments...

asgrim · 2017-04-03T14:35:30Z

Great - thanks @arjanvdbos!

arjanvdbos added 2 commits March 30, 2017 12:14

Added support for cache chains with multiple cache providers

104015a

Removed coalesce operator in order to make it backwards compatible wi…

98a0d6a

…th PHP < 7.x

Fixed code style issues

1a81043

asgrim reviewed Mar 30, 2017

View reviewed changes

asgrim added the enhancement label Mar 30, 2017

Example added for a chain configuration

682bc9d

asgrim approved these changes Apr 3, 2017

View reviewed changes

asgrim self-assigned this Apr 3, 2017

asgrim added this to the 1.0.0 milestone Apr 3, 2017

asgrim merged commit be47616 into DASPRiD:master Apr 3, 2017

Provide feedback