Paths

Table of Contentst

Force a couple of Conduit results to the proper types in Calendar
ClosedPublic
Actions

Authored by epriestley on Oct 11 2016, 7:34 PM.

Tags

None

Referenced Files

	F19508933: D16690.id40180.diff
	Jan 10 2026, 12:25 PM

	F19131419: D16690.diff
	Dec 10 2025, 5:57 AM

	F18835568: D16690.id40182.diff
	Oct 26 2025, 4:55 PM

	F18787370: D16690.id40180.diff
	Oct 14 2025, 11:24 PM

	F18784482: D16690.id.diff
	Oct 13 2025, 12:39 PM

	F18783105: D16690.id40182.diff
	Oct 13 2025, 3:45 AM

	F18763233: D16690.diff
	Oct 6 2025, 11:21 PM

	F18751572: D16690.id.diff
	Oct 4 2025, 11:43 AM

View All Files

Subscribers

None

Tokens

"Love" token, awarded by 20after4.

Details

Reviewers

chad

Maniphest Tasks

T11706: Retrieve event date from Calendar "calendar.event.search" method API

Commits

rP13b4b37d30d2: Force a couple of Conduit results to the proper types in Calendar

Summary

Ref T11706. Add some casts so we don't return "0" for false.

Also I forgot to document one of the things.

Test Plan

Called calendar.event.search.

Diff Detail

Repository

rP Phabricator

Branch

cal2

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 14073
Build 18258: Run Core Tests
Build 18257: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 40180.Oct 11 2016, 7:34 PM

epriestley retitled this revision from to Force a couple of Conduit results to the proper types in Calendar.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: chad.

epriestley added a task: T11706: Retrieve event date from Calendar "calendar.event.search" method API.

chad accepted this revision.Oct 11 2016, 7:35 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Oct 11 2016, 7:35 PM

Closed by commit rP13b4b37d30d2: Force a couple of Conduit results to the proper types in Calendar (authored by epriestley, committed by epriestley). · Explain WhyOct 11 2016, 8:00 PM

This revision was automatically updated to reflect the committed changes.

20after4 awarded a token.Oct 12 2016, 8:10 AM

Revision Contents
Changeset List

Path

Size

src/

applications/

calendar/

storage/

PhabricatorCalendarEvent.php

8 lines

Diff 40180

View Options

src/applications/calendar/storage/PhabricatorCalendarEvent.php

Force a couple of Conduit results to the proper types in CalendarClosedPublicActions