Homec4science

Account for raw limits properly in CalendarEventQuery

Authored by epriestley <git@epriestley.com> on Apr 7 2016, 14:10.

Description

Account for raw limits properly in CalendarEventQuery

Summary:
Fixes T8613. This was pretty straightforward, I just never dug into it originally.

rawResultLimit = 0 just means "no limit", so the fix is to only apply a limit if it is set to some nonzero value.

Also modernize a few pieces of code.

Test Plan: I'm actually not sure this can actually be hit normally? I faked setGenerateGhosts(true) into an unrelated query, hit the fatal, then fixed it.

Reviewers: lpriestley, chad

Reviewed By: chad

Maniphest Tasks: T8613

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

Details

Committed
epriestley <git@epriestley.com>Apr 7 2016, 17:39
Pushed
aubortJan 31 2017, 17:16
Parents
rPH8d6488f29071: Fix a typo in `bin/repository help update`
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH27104b57c827: Account for raw limits properly in CalendarEventQuery (authored by epriestley <git@epriestley.com>).Apr 7 2016, 17:39