Homec4science

Fix calendar part 2

Authored by Bob Trahan <bob.trahan@gmail.com> on Feb 25 2014, 23:20.

Description

Fix calendar part 2

Summary: D8341 was a good start. However, I was looping through all the statuses each time, when I should only deal with a given status once. Instead, unset() a status from the list of statuses once we handled it. Also, delete the last old $key thing, which interfered with my chosen strategy.

Test Plan: made a two day event and verified it showed up in just those two days. (will push and test again just in case but this should be it)

Reviewers: epriestley, chad

Reviewed By: epriestley

CC: Korvin, epriestley, aran

Differential Revision: https://secure.phabricator.com/D8342

Details

Committed
Bob Trahan <bob.trahan@gmail.com>Feb 25 2014, 23:20
Pushed
aubortJan 31 2017, 17:16
Parents
rPHb1e44239ba11: Fix calendar display on profile.
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <bob.trahan@gmail.com> committed rPH9b0f906207eb: Fix calendar part 2 (authored by Bob Trahan <bob.trahan@gmail.com>).Feb 25 2014, 23:20