Page MenuHomePhabricator

Fix a couple of calendar export daterange issues
ClosedPublic

Authored by epriestley on Nov 18 2016, 5:37 PM.
Tags
None
Referenced Files
F17900397: D16898.id40677.diff
Tue, Jul 29, 9:11 AM
F17819739: D16898.id.diff
Sat, Jul 26, 1:28 AM
F17815171: D16898.id.diff
Fri, Jul 25, 9:45 PM
F17809012: D16898.diff
Fri, Jul 25, 4:28 PM
F17784475: D16898.id40677.diff
Thu, Jul 24, 12:05 PM
F17760648: D16898.diff
Tue, Jul 22, 7:49 PM
F17760028: D16898.diff
Tue, Jul 22, 7:08 PM
Unknown Object (File)
Sat, Jul 5, 7:01 AM
Subscribers
None

Details

Summary

Ref T11816. In some cases, Calendar would only export a subset of events because the "export" flag was ignored or the "display" parameter applied an improper date range to the query.

  • Make sure the export flag gets processed, even though it isn't a "real" field on the search engine.
  • Clear the "display" parameter to avoid date range windowing coming from the day/month logic.
Test Plan

Exported a "display=month" view, verified future events came with it.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Fix a couple of calendar export daterange issues.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 18 2016, 6:01 PM
This revision was automatically updated to reflect the committed changes.