Page MenuHomePhabricator
Differential D16439

Removed unused buildApplicationPage method from PhabricatorController
ClosedPublic
Actions

Authored by jcox on Aug 24 2016, 3:01 PM.
Tags
None
Referenced Files
F15554149: D16439.id39536.diff
Mon, Apr 28, 5:43 AM
F15548949: D16439.id.diff
Sun, Apr 27, 3:00 AM
F15544480: D16439.diff
Sat, Apr 26, 5:19 AM
F15518170: D16439.id39534.diff
Sat, Apr 19, 11:34 AM
F15485964: D16439.diff
Thu, Apr 10, 3:59 AM
F15479145: D16439.diff
Tue, Apr 8, 5:18 AM
F15458230: D16439.id.diff
Mar 30 2025, 9:59 PM
F15448816: D16439.id39536.diff
Mar 28 2025, 7:03 AM
View All Files
Subscribers
epriestley
yelirekim

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T9690: Remove calls to methods deprecated by newPage()
Commits
rP2201c65eb73f: Removed unused buildApplicationPage method from PhabricatorController
Summary

Getting rid of some code! This method has no callsites so it should be safe to remove completely. Ref T9690

Test Plan

Removed method and clicked around to make sure nothing broke.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jcox updated this revision to Diff 39534.Aug 24 2016, 3:01 PM
jcox retitled this revision from to Removed unused buildApplicationPage method from PhabricatorController.
jcox updated this object.
jcox edited the test plan for this revision. (Show Details)
jcox added a task: T9690: Remove calls to methods deprecated by newPage().
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptAug 24 2016, 3:01 PM
Herald added a subscriber: epriestley. · View Herald Transcript
epriestley accepted this revision.Aug 24 2016, 3:22 PM
epriestley added a reviewer: epriestley.

Nice.

This revision is now accepted and ready to land.Aug 24 2016, 3:22 PM
yelirekim added a subscriber: yelirekim.Aug 24 2016, 3:24 PM
yelirekim added inline comments.
src/applications/base/controller/PhabricatorController.php
587

RIP

yelirekim added a comment.Aug 24 2016, 3:25 PM

This is an ATC meme-ish at this point because I wrote so many controllers without page titles when I first starting extending Phab.

Closed by commit rP2201c65eb73f: Removed unused buildApplicationPage method from PhabricatorController (authored by jcox, committed by jcox). · Explain WhyAug 24 2016, 3:25 PM
This revision was automatically updated to reflect the committed changes.
jcox added inline comments.Aug 24 2016, 3:27 PM
src/applications/base/controller/PhabricatorController.php
585

I'm glad this lets me know we're dealing with serious-business here.

Revision Contents
Changeset List

PathSize
src/
applications/
base/
controller/
40 lines

Diff 39536

View Options

src/applications/base/controller/PhabricatorController.php

Loading...
Phabricator · Built by Phacility