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

Keep floating time on Recur\EventIterator::getEventObject #188

Merged
merged 1 commit into from
Jan 21, 2015

Conversation

ahackschmitz
Copy link
Contributor

to correctly expand with reference timezone (if any).

This fixes a problem where recurrent events with floating times would get set to UTC by EventIterator::getEventObject() while it should be transferred to UTC in VCalendar::expand().

@Hywan Hywan added the bug label Jan 21, 2015
@Hywan
Copy link
Member

Hywan commented Jan 21, 2015

👍

@evert
Copy link
Member

evert commented Jan 21, 2015

Nice!

evert added a commit that referenced this pull request Jan 21, 2015
Keep floating time on Recur\EventIterator::getEventObject
@evert evert merged commit ef3d237 into sabre-io:3.3 Jan 21, 2015

/**
* This is a unittest for Issue #53.
*/
Copy link
Member

Choose a reason for hiding this comment

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

No :-) 53 is about something else, I think.

Copy link
Member

Choose a reason for hiding this comment

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

You're right

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Argh, my bad! Didn't recheck after duplicating the file from another test. :/

@Hywan
Copy link
Member

Hywan commented Jan 22, 2015

The Gang of Bad Reviewers™

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

Successfully merging this pull request may close these issues.

4 participants