Paths

Table of Contentst

Differential D16690

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

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

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