Page MenuHomePhabricator
Differential D13679

Update Ponder for handleRequest
ClosedPublic
Actions

Authored by chad on Jul 22 2015, 6:35 PM.
Tags
None
Referenced Files
F15563623: D13679.diff
Wed, Apr 30, 9:48 AM
F15560761: D13679.id.diff
Tue, Apr 29, 4:28 PM
F15555873: D13679.diff
Mon, Apr 28, 1:30 PM
F15537962: D13679.id33063.diff
Thu, Apr 24, 4:58 PM
F15519485: D13679.id33063.diff
Sat, Apr 19, 11:34 PM
F15519484: D13679.diff
Sat, Apr 19, 11:34 PM
F15510606: D13679.id33069.diff
Wed, Apr 16, 8:27 PM
F15510605: D13679.id33063.diff
Wed, Apr 16, 8:27 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 33069

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