Page MenuHomePhabricator

Skeleton of calendar icon infrastructure
ClosedPublic

Authored by lpriestley on May 19 2015, 5:58 PM.
Tags
None
Referenced Files
F18876124: D12928.diff
Nov 5 2025, 9:47 PM
F18764134: D12928.id31119.diff
Oct 7 2025, 4:28 AM
F18743018: D12928.diff
Oct 2 2025, 10:03 PM
F18663794: D12928.diff
Sep 23 2025, 10:52 PM
F18607188: D12928.id31116.diff
Sep 13 2025, 11:27 PM
F18498002: D12928.diff
Sep 4 2025, 6:23 PM
F18438894: D12928.id.diff
Aug 31 2025, 2:05 PM
F18425373: D12928.diff
Aug 30 2025, 9:47 PM
Subscribers

Details

Summary

Ref T7936, Skeleton of calendar icon infrastructure

Test Plan

Navigate to install/calendar/icon/, observe temporary calendar icons

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

lpriestley retitled this revision from to Skeleton of calendar icon infrastructure.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/calendar/controller/PhabricatorCalendarEventEditIconController.php
8–13

Prefer the slightly more modern/simpler: handleRequest(AphrontRequest $request) method (instead of processRequest()) then $id = $request->getURIData('id');, and pull $viewer with $this->getViewer().

30

Trailing slash probably incorrect?

This revision is now accepted and ready to land.May 19 2015, 6:10 PM
lpriestley marked an inline comment as done.
lpriestley edited edge metadata.

retro-garbage

This revision was automatically updated to reflect the committed changes.