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 | ||
|---|---|---|
| 186–195 | 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