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.
Details
Details
- Reviewers
epriestley chad - Commits
- Restricted Diffusion Commit
rP9b0f906207eb: Fix calendar part 2
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)
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- fixcal2
- Lint
Lint Passed - Unit
No Test Coverage
Event Timeline
src/applications/people/controller/PhabricatorPeopleProfileController.php | ||
---|---|---|
185–186 | Could we remove this loop part instead? It looks like the logic is: foreach (day) { foreach (event) { foreach (future day) { put the event on all of the days it goes on } unset the event; } } Instead, maybe one of these would be simpler? foreach (day) { foreach (event) { maybe put this event on this day } } Or: foreach (event) { put it on all the days it goes on } |
Comment Actions
use foreach (epoch / day) { foreach (event) } algorithm.
(i don't think I can just do foreach (event) as it is about mapping these events to days.)
tested by adding a bunch more events to myself; it works