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

Support topic "0" in psubscribe #106

Merged
merged 1 commit into from Jan 9, 2015

Conversation

Projects
None yet
2 participants
@kappa
Contributor

kappa commented Jan 9, 2015

There was no way of getting the original topic name in pubsub messages received via PSUBSCRIBE if the name of the topic was "falsy" in Perl sense.

A test and a basic '||' vs. 'defined' fix.

Support topic "0" in psubscribe
A test and a basic '||' vs. 'defined' fix.

dams added a commit that referenced this pull request Jan 9, 2015

Merge pull request #106 from kappa/zero-topic
Support topic "0" in psubscribe

@dams dams merged commit ef2e53f into PerlRedis:master Jan 9, 2015

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
@dams

This comment has been minimized.

Show comment
Hide comment
@dams

dams Jan 9, 2015

Member

thanks !

Member

dams commented Jan 9, 2015

thanks !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment