Page MenuHomePhabricator
Differential D16797

Allow EditEngine to build NUX buttons that point at the right place
ClosedPublic
Actions

Authored by epriestley on Nov 3 2016, 6:24 PM.
Tags
None
Referenced Files
F19077780: D16797.diff
Mon, Dec 1, 4:12 PM
F19057289: D16797.diff
Fri, Nov 28, 9:53 PM
F18856510: D16797.diff
Nov 1 2025, 10:07 AM
F18815150: D16797.id.diff
Oct 21 2025, 1:04 AM
F18809050: D16797.diff
Oct 19 2025, 10:50 AM
F18765136: D16797.id40457.diff
Oct 7 2025, 9:44 AM
F18733431: D16797.id.diff
Sep 30 2025, 10:34 PM
F18684129: D16797.id.diff
Sep 26 2025, 8:43 AM
View All Files
Subscribers
avivey

Details

Reviewers
avivey
chad
Maniphest Tasks
T11812: "Create a Paste" button on welcome page doesn't work
Commits
rP0f1785c0aa8c: Allow EditEngine to build NUX buttons that point at the right place
Summary

Fixes T11812.

  • Pull the logic for building the "Create Whatever" dropdown out.
  • Use it to generate NUX buttons, too.
  • Use the new logic in Paste and Maniphest.
Test Plan
  • Viewed Paste NUX, button worked.
  • Viewed Maniphest NUX with multiple create forms, button worked.

Diff Detail

Repository
rP Phabricator
Branch
edit1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 14347
Build 18679: Run Core Tests
Build 18678: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 40454.Nov 3 2016, 6:24 PM
epriestley retitled this revision from to Allow EditEngine to build NUX buttons that point at the right place.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T11812: "Create a Paste" button on welcome page doesn't work.
avivey accepted this revision.Nov 3 2016, 7:57 PM
avivey added a reviewer: avivey.
avivey added a subscriber: avivey.
avivey added inline comments.
src/applications/transactions/editengine/PhabricatorEditEngine.php
1394

Maybe respect $spec['disabled'] and 'workflow' here too, just in case?

This revision is now accepted and ready to land.Nov 3 2016, 7:57 PM
epriestley updated this revision to Diff 40457.Nov 4 2016, 4:50 PM
epriestley edited edge metadata.
  • Respect disabled/workflow on items, just in case.
Closed by commit rP0f1785c0aa8c: Allow EditEngine to build NUX buttons that point at the right place (authored by epriestley, committed by epriestley). · Explain WhyNov 4 2016, 4:51 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
maniphest/
query/
10 lines
paste/
query/
10 lines
transactions/
editengine/
137 lines

Diff 40457

View Options

src/applications/maniphest/query/ManiphestTaskSearchEngine.php

Loading...
View Options

src/applications/paste/query/PhabricatorPasteSearchEngine.php

Loading...
View Options

src/applications/transactions/editengine/PhabricatorEditEngine.php

Loading...
Phabricator · Built by Phacility