Page MenuHomePhabricator

Make passing an object to newCurtain optional
ClosedPublic

Authored by chad on May 16 2017, 2:58 AM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 16 2026, 10:47 PM
Unknown Object (File)
Apr 15 2026, 11:56 PM
Unknown Object (File)
Feb 28 2026, 7:14 PM
Unknown Object (File)
Feb 28 2026, 3:16 PM
Unknown Object (File)
Feb 4 2026, 12:21 PM
Unknown Object (File)
Nov 23 2025, 6:43 AM
Unknown Object (File)
Nov 22 2025, 6:23 AM
Unknown Object (File)
Nov 1 2025, 10:18 AM
Subscribers

Details

Summary

We seem to already support this, just takes it fully there. We don't need to see things like "Flag", etc, on certain subpages of projects/people/etc.

Test Plan

Review Members, Subproject pages, no longer see "Flag for Later" which only is for the Project itself. Check manage, still there.

Diff Detail

Repository
rP Phabricator
Branch
curtain-object (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16968
Build 22684: Run Core Tests
Build 22683: arc lint + arc unit

Event Timeline

epriestley added inline comments.
src/applications/base/controller/PhabricatorController.php
494–497

Consider skipping this if $object is null -- some extension might run into trouble assuming that $object will always actually be an object.

This revision is now accepted and ready to land.May 16 2017, 9:52 AM
This revision was automatically updated to reflect the committed changes.