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

We don't need call info method every time we want redis resource. +1 #119

Merged
merged 1 commit into from
Nov 4, 2016
Merged

We don't need call info method every time we want redis resource. +1 #119

merged 1 commit into from
Nov 4, 2016

Conversation

Moln
Copy link
Contributor

@Moln Moln commented Oct 27, 2016

Fix #100

The $resource['version'] can be set only once.

            if (!$resource['initialized']) {
                $this->connect($resource);
            }

            if (!$resource['version']) { //If version is not set .
                $info = $resource['resource']->info();
                $resource['version'] = $info['redis_version'];
            }

            return $resource['resource'];

@marc-mabe marc-mabe merged commit a7ca3ee into zendframework:master Nov 4, 2016
marc-mabe added a commit that referenced this pull request Nov 4, 2016
marc-mabe added a commit that referenced this pull request Nov 4, 2016
@marc-mabe marc-mabe added this to the 2.7.2 milestone Dec 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants