HomePhorge

Update Diff: fix dead-end when not yours

Description

Update Diff: fix dead-end when not yours

Summary:
Fix "Update Patch" for generic Differential revisions.

You can now help a coworker to update their patch, from the web.

So, if you visit Diff 111 (not yours) and you click "Update Diff", and you paste a valid diff,
you arrive here:

https://we.phorge.it/differential/diff/222/?revisionID=111

That page as default was suggesting some Diffs that are yours. Plus, now it shows Diff 111.

Omitting the Diff from which the workflow was started made no sense and it's certainly an oversight.

Any follow-up change is welcome to modernize the selector using AphrontFormTokenizerControl.

Closes T15538

Test Plan:

  • create a Diff on Differential with user A
  • click on Update Diff from user B (pasting a valid diff) and Continue

Now that patch is selected (and selectable). No dead-end anymore.

Reviewers: O1 Blessed Committers, avivey

Reviewed By: O1 Blessed Committers, avivey

Subscribers: avivey, speck, tobiaswiese, Matthew, Cigaryno

Maniphest Tasks: T15538

Differential Revision: https://we.phorge.it/D25337

Details

Provenance
Valerio BozzolanAuthored on Jul 25 2023, 5:12 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rPbf025c5fb491: Aphlict - search for default config in the right place
Branches
Unknown
Tags
Unknown

Event Timeline