Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixed problem with init redis server #37

Closed
wants to merge 3 commits into from

Conversation

Mirocow
Copy link
Contributor

@Mirocow Mirocow commented Oct 12, 2015

fixed problem with init redis server if redis server don`t run
because @stream_socket_client return false (line https://github.com/yiisoft/yii2-redis/blob/master/Connection.php#L263)

fixed problem with init redis server if redis server don`t run
@samdark samdark added the type:bug Bug label Oct 12, 2015
@samdark samdark added this to the 2.0.5 milestone Oct 12, 2015
@Mirocow
Copy link
Contributor Author

Mirocow commented Oct 12, 2015

it`s fixed php 5.6
работу проверял на php 5.6

@Mirocow
Copy link
Contributor Author

Mirocow commented Oct 12, 2015

$ php -v
PHP 5.6.14 (cli) (built: Oct 3 2015 14:43:37)
Copyright (c) 1997-2015 The PHP Group
Zend Engine v2.6.0, Copyright (c) 1998-2015 Zend Technologies
with Xdebug v2.3.3, Copyright (c) 2002-2015, by Derick Rethans

@samdark samdark assigned samdark and unassigned cebe Oct 12, 2015
@samdark samdark closed this in 2ec0b75 Oct 12, 2015
@samdark
Copy link
Member

samdark commented Oct 12, 2015

Thanks!

@cebe cebe mentioned this pull request Mar 1, 2016
@cebe cebe mentioned this pull request Apr 27, 2016
cebe added a commit that referenced this pull request Apr 27, 2016
reconnecting a closed connection fails.

fixes #67
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants