Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

Update: Fix for Issue #912 ($position in repeat directive can't be both first and last at the same time) #964

Closed
wants to merge 4 commits into from

Conversation

maxmart
Copy link
Contributor

@maxmart maxmart commented May 17, 2012

Same pull request as #928 but it's from a separate branch, so future commits that I do to my master won't affect it. See #928 for more info.

Sorry for the mess.

Fix for issue #912 (#912)

Implementation of my suggested fix where three boolean values $first, $middle and $last replaces the $position variable. I also added a test case for single item lists.

@IgorMinar
Copy link
Contributor

landed as 1d38867

thanks! can we send you a t-shirt? http://goo.gl/D9uOx

@IgorMinar IgorMinar closed this May 22, 2012
@maxmart
Copy link
Contributor Author

maxmart commented May 23, 2012

Sweet! Thank you :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants