Page MenuHomePhabricator
Differential D12791

Deprecating BrowseController and getting rid of unneeded calls to `getTerseSummary()` and `getHumanStatus()`
ClosedPublic
Actions

Authored by lpriestley on May 10 2015, 5:48 PM.
Tags
None
Referenced Files
F14351967: D12791.diff
Thu, Dec 19, 12:00 PM
Unknown Object (File)
Sun, Dec 15, 7:21 PM
Unknown Object (File)
Sun, Dec 15, 5:10 PM
Unknown Object (File)
Thu, Nov 28, 12:10 AM
Unknown Object (File)
Sat, Nov 23, 4:37 PM
Unknown Object (File)
Nov 14 2024, 7:31 AM
Unknown Object (File)
Nov 9 2024, 12:15 AM
Unknown Object (File)
Nov 4 2024, 8:20 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T8050: Calendar List formatting
Commits
Restricted Diffusion Commit
rP51facbabba91: Deprecating BrowseController and getting rid of unneeded calls to…
Summary

Ref T8050, Deprecating BrowseController and getting rid of unneeded calls to getTerseSummary() and getHumanStatus()

Test Plan

Use calendar, make sure nothing explodes

Diff Detail

Repository
rP Phabricator
Branch
calendardeprecatebrowsecontroller
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 5890
Build 5910: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

lpriestley updated this revision to Diff 30745.May 10 2015, 5:48 PM
lpriestley retitled this revision from to Deprecating BrowseController and getting rid of unneeded calls to `getTerseSummary()` and `getHumanStatus()`.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
lpriestley added a task: T8050: Calendar List formatting.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 10 2015, 5:48 PM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
epriestley accepted this revision.May 10 2015, 5:52 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 10 2015, 5:52 PM
Closed by commit rP51facbabba91: Deprecating BrowseController and getting rid of unneeded calls to… (authored by lpriestley, committed by lpriestley). · Explain WhyMay 10 2015, 6:14 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
calendar/
controller/
query/
5 lines
storage/
5 lines

Diff 30745

View Options

src/applications/calendar/controller/PhabricatorCalendarBrowseController.php

Loading...
View Options

src/applications/calendar/query/PhabricatorCalendarEventSearchEngine.php

Loading...
View Options

src/applications/calendar/storage/PhabricatorCalendarEvent.php

Loading...
Phabricator · Built by Phacility