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

Already on GitHub? Sign in to your account

Test for the getArrayCopy method in AbstractRestultSet #5089

Merged
merged 1 commit into from Sep 9, 2013

Conversation

Projects
None yet
2 participants
Member

akrabat commented Sep 8, 2013

If you're using the ArraySerializable hydrator, then you will have a method called getArrayCopy in your row object, However, AbstractResultSet::toArray() currently checks that your row is an instance of ArrayObject and hence throws an exception.

This PR changes AbstractResultSet::toArray() to check for the method name instead, to match the way that ArraySerializable::extract() handles this.

Test for the getArrayCopy method in AbstractRestultSet
If you're using the `ArraySerializable` hydrator, then you will have a
method called `getArrayCopy` in your row object, so allow
`AbstractResultSet` to use it.

weierophinney added a commit that referenced this pull request Sep 9, 2013

Merge pull request #5089 from akrabat/getarraycopy-check
Test for the getArrayCopy method in AbstractRestultSet

weierophinney added a commit that referenced this pull request Sep 9, 2013

@ghost ghost assigned weierophinney Sep 9, 2013

@weierophinney weierophinney merged commit 360bbdc into zendframework:develop Sep 9, 2013

1 check passed

default The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment