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

Implement iterator support at getLastKeyPair #132

Merged
merged 5 commits into from Feb 4, 2021

Conversation

cwreden
Copy link
Contributor

@cwreden cwreden commented Feb 1, 2021

Hello together,

we use this Guard with our Redis iterator and the Key() method from php doesn't work with iterators that have there data not as properties.
So I implement the iterator support like in enforceStorageLimit method.

I'm not sure how travis work, what are open tasks for me?

Copy link
Member

@akrabat akrabat left a comment

Choose a reason for hiding this comment

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

This looks like a useful addition. One thing to consider and a minor whitespace update to do.

src/Guard.php Show resolved Hide resolved
tests/GuardTest.php Outdated Show resolved Hide resolved
@akrabat akrabat added this to the 1.2.1 milestone Feb 4, 2021
@akrabat
Copy link
Member

akrabat commented Feb 4, 2021

Thanks!

@akrabat akrabat closed this in ee811a2 Feb 4, 2021
@akrabat akrabat merged commit ee811a2 into slimphp:master Feb 4, 2021
@akrabat
Copy link
Member

akrabat commented Feb 4, 2021

This has been released as version 1.2.1

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.

None yet

2 participants