Skip to content

Conversation

@cergfix
Copy link

@cergfix cergfix commented Jul 29, 2015

No description provided.

@Nyholm
Copy link
Member

Nyholm commented Jul 29, 2015

Thank you for this. Would you mind adding a small test for the new read function?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Debug code?

@cergfix
Copy link
Author

cergfix commented Sep 28, 2015

Can be tested using a simple PHP test described here #6 (comment).

@Nyholm
Copy link
Member

Nyholm commented Sep 29, 2015

I ment an unit test. I know that this library currently have none but if you add a small test for just this bugfix it will help future changes not to reintroduce this issue.

@Nyholm
Copy link
Member

Nyholm commented Oct 21, 2015

Ive tried to reproduce this with the test code you provided. Can you one of the following:

  1. provide a unit test to prove the error
  2. Write detailed steps to reproduce

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants