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

Problem iterating buffered ResultSet #6845

Closed
ChrisSchreiber opened this issue Nov 6, 2014 · 4 comments
Closed

Problem iterating buffered ResultSet #6845

ChrisSchreiber opened this issue Nov 6, 2014 · 4 comments
Assignees
@Ocramius
Labels
bug Db
Milestone
2.3.4

Comments

@ChrisSchreiber
Copy link
Contributor

If you iterate over a buffered result set but stop before reaching the end then try to iterate over it again the data past where the initial iteration stoped is not returned correctly.

The problem is Zend\Db\ResultSet\AbstractResultSet::rewind() does not reset the data source's position, but Zend\Db\ResultSet\AbstractResultSet::valid() and Zend\Db\ResultSet\AbstractResultSet::current() use it once you iterate past where the buffer has been filled.
@ChrisSchreiber ChrisSchreiber mentioned this issue Nov 6, 2014
Closed
@ChrisSchreiber
Copy link
Contributor Author

I just realized the fix in my pull request causes other problems. I'm working on a different fix.

@ChrisSchreiber ChrisSchreiber mentioned this issue Nov 6, 2014
Closed
@macnibblet
Copy link
Contributor

can you tag the test with a group annotation preferably @group issue-<issue id>

@ChrisSchreiber
Copy link
Contributor Author

I added the group annotation

@Ocramius Ocramius added this to the 2.3.4 milestone Jan 3, 2015
@Ocramius Ocramius self-assigned this Jan 3, 2015
Ocramius added a commit that referenced this issue Jan 3, 2015
@Ocramius
Merge branch 'hotfix/#6847-#6845-fix-multiple-iteration-over-buffered…
853f1c5 
…-resultset' into develop

Close #6847
Close #6845
Forward port #6847
Forward port #6845
@Ocramius
Copy link
Member

Ocramius commented Jan 3, 2015

Handled in #6847

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

@Ocramius Ocramius

Labels
bug Db
Projects
None yet
Milestone
2.3.4
Development

No branches or pull requests
3 participants
@Ocramius @macnibblet @ChrisSchreiber