Page MenuHomePhabricator
Differential D16572

Removed willProcessRequest from DifferentialRevisionLandController
ClosedPublic
Actions

Authored by jcox on Sep 19 2016, 1:19 PM.
Tags
None
Referenced Files
F13086932: D16572.diff
Thu, Apr 25, 12:47 AM
F13086732: D16572.diff
Thu, Apr 25, 12:41 AM
Unknown Object (File)
Fri, Apr 5, 12:12 PM
Unknown Object (File)
Fri, Apr 5, 8:04 AM
Unknown Object (File)
Thu, Apr 4, 8:08 AM
Unknown Object (File)
Tue, Apr 2, 10:19 PM
Unknown Object (File)
Sat, Mar 30, 4:31 AM
Unknown Object (File)
Jan 28 2024, 6:16 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