Page MenuHomePhabricator
Differential D16572

Removed willProcessRequest from DifferentialRevisionLandController
ClosedPublic
Actions

Authored by jcox on Sep 19 2016, 1:19 PM.
Tags
None
Referenced Files
F18839311: D16572.id.diff
Mon, Oct 27, 3:32 PM
F18830244: D16572.id39889.diff
Sat, Oct 25, 5:21 AM
F18819111: D16572.id39889.diff
Wed, Oct 22, 4:25 AM
F18810872: D16572.id.diff
Sun, Oct 19, 11:49 PM
F18806121: D16572.diff
Sat, Oct 18, 5:52 PM
F18642336: D16572.id.diff
Sep 18 2025, 11:34 PM
F18618606: D16572.diff
Sep 15 2025, 12:33 AM
F18615080: D16572.diff
Sep 14 2025, 1:51 PM
View All Files
Subscribers
epriestley
Korvin
yelirekim

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T8628: Modernize AphrontControllers to implement handleRequest
Commits
rPadf9d5ffdd61: Removed willProcessRequest from DifferentialRevisionLandController
Summary

Ref T8628.

Test Plan

Landed a revision through the web UI

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jcox updated this revision to Diff 39884.Sep 19 2016, 1:19 PM
jcox retitled this revision from to Removed willProcessRequest from DifferentialRevisionLandController.
jcox updated this object.
jcox edited the test plan for this revision. (Show Details)
jcox added a task: T8628: Modernize AphrontControllers to implement handleRequest.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptSep 19 2016, 1:19 PM
Herald added subscribers: yelirekim, epriestley. · View Herald Transcript
jcox edited the test plan for this revision. (Show Details)Sep 19 2016, 1:50 PM
jcox edited edge metadata.
epriestley accepted this revision.Sep 19 2016, 5:08 PM
epriestley added a reviewer: epriestley.

This code isn't actually reachable ("Land Revision" in the web UI uses DifferentialRevisionOperationController, not this one) but it's being retained purely for backward compatibility until "Land Revision" has a better story on auth-to-GitHub stuff which this older thing technically sort-of supports.

This revision is now accepted and ready to land.Sep 19 2016, 5:08 PM
jcox added a comment.Sep 19 2016, 5:23 PM

ah gotcha. Should I land this anyway or just leave it be?

epriestley added a comment.Sep 19 2016, 5:45 PM

Just land it, if this somehow breaks things (unlikely) this code was slated for removal anyway.

jcox updated this revision to Diff 39889.Sep 19 2016, 5:47 PM
jcox edited edge metadata.

Rebase

Herald added a subscriber: Korvin. · View Herald TranscriptSep 19 2016, 5:47 PM
Closed by commit rPadf9d5ffdd61: Removed willProcessRequest from DifferentialRevisionLandController (authored by jcox, committed by jcox). · Explain WhySep 19 2016, 5:49 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
controller/
17 lines

Diff 39890

View Options

src/applications/differential/controller/DifferentialRevisionLandController.php

Loading...
Phabricator · Built by Phacility