Page MenuHomePhabricator
Differential D13679

Update Ponder for handleRequest
ClosedPublic
Actions

Authored by chad on Jul 22 2015, 6:35 PM.
Tags
None
Referenced Files
F14069053: D13679.diff
Wed, Nov 20, 3:23 AM
F14053942: D13679.diff
Fri, Nov 15, 8:48 PM
F14041573: D13679.diff
Mon, Nov 11, 7:43 PM
F14028095: D13679.diff
Fri, Nov 8, 11:48 AM
F13997559: D13679.id.diff
Thu, Oct 24, 5:07 AM
F13994717: D13679.id33069.diff
Wed, Oct 23, 8:00 AM
F13964164: D13679.id33069.diff
Oct 15 2024, 7:53 PM
F13964163: D13679.id33063.diff
Oct 15 2024, 7:53 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
btrahan
Maniphest Tasks
T8628: Modernize AphrontControllers to implement handleRequest
Commits
Restricted Diffusion Commit
rP1aed0e67a9a9: Update Ponder for handleRequest
Summary

Update all Ponder callsites.

Test Plan

New Question, New Comment, New Answer, Vote Up and Down

Diff Detail

Repository
rP Phabricator
Branch
ponder1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7349
Build 7738: [Placeholder Plan] Wait for 30 Seconds
Build 7737: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 33063.Jul 22 2015, 6:35 PM
chad retitled this revision from to Update Ponder for handleRequest.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
Herald added subscribers: Korvin, epriestley. · View Herald TranscriptJul 22 2015, 6:35 PM
chad added a task: T8628: Modernize AphrontControllers to implement handleRequest.Jul 22 2015, 7:48 PM
epriestley accepted this revision.Jul 22 2015, 8:21 PM
epriestley edited edge metadata.

One catch, see inline.

src/applications/ponder/controller/PonderQuestionListController.php
10

should likely remain getURIData('queryKey').

This revision is now accepted and ready to land.Jul 22 2015, 8:21 PM
Closed by commit rP1aed0e67a9a9: Update Ponder for handleRequest (authored by chad, committed by chad). · Explain WhyJul 22 2015, 8:23 PM
This revision was automatically updated to reflect the committed changes.
chad marked an inline comment as done.

Revision Contents
Changeset List

Diff 33063

View Options

src/applications/ponder/controller/PonderAnswerCommentController.php

Loading...
View Options

src/applications/ponder/controller/PonderAnswerEditController.php

Loading...
View Options

src/applications/ponder/controller/PonderAnswerHistoryController.php

Loading...
View Options

src/applications/ponder/controller/PonderAnswerSaveController.php

Loading...
View Options

src/applications/ponder/controller/PonderQuestionCommentController.php

Loading...
View Options

src/applications/ponder/controller/PonderQuestionEditController.php

Loading...
View Options

src/applications/ponder/controller/PonderQuestionHistoryController.php

Loading...
View Options

src/applications/ponder/controller/PonderQuestionListController.php

Loading...
View Options

src/applications/ponder/controller/PonderQuestionStatusController.php

Loading...
View Options

src/applications/ponder/controller/PonderQuestionViewController.php

Loading...
View Options

src/applications/ponder/controller/PonderVoteSaveController.php

Loading...
Phabricator · Built by Phacility