Skip to content
This repository has been archived by the owner on Jan 2, 2019. It is now read-only.

Added Redis (PHP Extension) support for CachedObjectStore in PHPExcel #772

Closed
wants to merge 1 commit into from

Conversation

stefandoorn
Copy link

Adds support for cell caching in Redis. This extension (https://github.com/phpredis/phpredis) is required. Predis is not supported right now.

@MarkBaker
Copy link
Member

Currently testing, thanks for the PR.... it's been on my to do list for a while now

@stefandoorn
Copy link
Author

I would be happy to start using this feature. Does it need polishing or any additional changes?

@PowerKiKi
Copy link
Member

@stefandoorn it should be re-submitted against https://github.com/PHPOffice/phpspreadsheet develop branch (implying rebase-ing and codestyle and namespace adaptations). It should also be tested with unit tests.

@stefandoorn
Copy link
Author

@PowerKiKi Is that the successor of this library? Or is this one based on that one?

@PowerKiKi
Copy link
Member

@stefandoorn it is the next version indeed. It's not yet released, but all development happens over there now.

@PowerKiKi
Copy link
Member

Before rebasing we should wait to see what happens with cache refactoring as planned in PHPOffice/PhpSpreadsheet#3

@PowerKiKi
Copy link
Member

This is now obsolete.

@PowerKiKi PowerKiKi closed this Jul 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants