Diffusion Phorge (master) History

phorge/src/applications/differential/editor/DifferentialTransactionEditor.php
master25aebab655fdb50118b8dcc…

Feb 5 2024

25aebab655fd Make misc get*ApplicationClass[Name]() calls return ::class constant instead of…
Author
Andre Klapper
Browse Repository
No Builds
No Audits

Jun 8 2023

44a8a1c408ed Fix PHP 8.1 "preg_match(null)" exception on "Create Revision" page in…
Author
Andre Klapper
Browse Repository
No Builds
No Audits

Mar 15 2021

26c68942bd00 Update "AffectedPath" table when a revision's repository changes
Author
epriestley
Browse Repository
No Builds
No Audits

38ef910da8ca Move "Affected Path" index updates to a separate class
Author
epriestley
Browse Repository
No Builds
No Audits

c317d16bdd56 Lift peculiar side effect of path indexing out of indexer
Author
epriestley
Browse Repository
No Builds
No Audits

Mar 3 2021

dec14136844d Provide a more general "Author" transaction for Differential
Author
epriestley
Browse Repository
No Builds
No Audits

May 4 2020

fe501bd7f736 Save drafts for inline comments currently being edited
Author
epriestley
Browse Repository
No Builds
No Audits

468aabd4ef30 When draft inline comments are submitted, disengage the editor
Author
epriestley
Browse Repository
No Builds
No Audits

Jun 18 2019

753828649956 Fix missing link targets for "View Object" header buttons in HTML email
Author
epriestley
Browse Repository
No Builds
No Audits

May 30 2019

9a32a563f047 Add a "View Task" button to HTML mail from Maniphest
Author
epriestley
Browse Repository
No Builds
No Audits

May 22 2019

1eff4fdca332 Prevent "Differential Revision: ..." from counting as a mention in commit…
Author
epriestley
Browse Repository
No Builds
No Audits

May 1 2019

a78f141b3fa8 Unify code for parsing "Reverts X" magic, and when something "reverts <hash>"…
Author
epriestley
Browse Repository
No Builds
No Audits

Jan 17 2019

c125ab7a42bf Remove "metamta.*.subject-prefix" options
Author
epriestley
Browse Repository
No Builds
No Audits

Jan 4 2019

73e3057c52f4 Rename "MetaMTA" mail attachments and add more mail message objects
Author
epriestley
Browse Repository
No Builds
No Audits

Dec 10 2018

46feccdfcf19 Share more inline "Done" code between Differential and Diffusion
Author
epriestley
Browse Repository
No Builds
No Audits

Dec 9 2018

1029081b2805 Correct two straggling "%Q" + "implode(...)" callsites in Revision updates
Author
epriestley
Browse Repository
No Builds
No Audits

Nov 26 2018

97e7ef0f015c When the last rejecting reviewer resigns from a revision, return it to "Needs…
Author
epriestley
Browse Repository
No Builds
No Audits

Oct 5 2018

99034efa8b8d Make Pholio mail render without a ton of over-escaped HTML
Author
epriestley
Browse Repository
No Builds
No Audits

Sep 7 2018

16a6fc8341ea Allow reviewers to mark their own inlines as "Done" before they submit them
Author
epriestley
Browse Repository
No Builds
No Audits

Sep 6 2018

e8e5dc0f5690 Make a language improvement ("inlines" -> "inline comments")
Author
epriestley
Browse Repository
No Builds
No Audits

041392988ece When a transaction adds more than 100 inline comments, include only the first…
Author
epriestley
Browse Repository
No Builds
No Audits

Aug 24 2018

5e4d9dfa9219 Condition "Changes Since Last Action" Differential link on "first broadcast"…
Author
epriestley
Browse Repository
No Builds
No Audits

Jul 27 2018

9e451879d998 Add a "Changes Since Last Action" view to Differential revisions
Author
epriestley
Browse Repository
No Builds
No Audits

Jun 12 2018

62a402491ae4 Allow encrypted mail to be more specific about which object is affected
Author
epriestley
Browse Repository
No Builds
No Audits

May 14 2018

26d0862f4f6f Apply the new patch byte size limit to mail patch generation in Differential
Author
epriestley
Browse Repository
No Builds
No Audits

May 5 2018

5e2af4b9b5cd Prepare to support an "Ignore generated files" flag in Owners
Author
epriestley
Browse Repository
No Builds
No Audits

Apr 20 2018

9bf4df2c1ddf Allow demoted builds to automatically promote if builds pass after a restart
Author
epriestley
Browse Repository
No Builds
No Audits

Apr 12 2018

c52e10d1ecf0 Respect external unmentionable PHIDs in Differential revision editor
Author
epriestley
Browse Repository
No Builds
No Audits

Apr 3 2018

e70c9f72a48e Show revision sizes using a perplexing, inexplicable symbol code
Author
epriestley
Browse Repository
No Builds
No Audits

6f520e053462 Clean up an old transaction state flag
Author
epriestley
Browse Repository
No Builds
No Audits

804f9817c3e2 When a draft's builds fail and it demotes to "Changes Planned + Draft", notify…
Author
epriestley
Browse Repository
No Builds
No Audits

adf8fdef0e89 When remote builds fail, demote revisions to "Changes Planned + But, Still A…
Author
epriestley
Browse Repository
No Builds
No Audits

615d27c8e97f Show an additional "Draft" tag on non-broadcasting revisions in a non-draft…
Author
epriestley
Browse Repository
No Builds
No Audits

38e788c99a78 Partially decouple revision broadcasting from revision draft state
Author
epriestley
Browse Repository
No Builds
No Audits

3b5a7d1c88c4 Rename the Differential "hasBroadcast" flag to "shouldBroadcast"
Author
epriestley
Browse Repository
No Builds
No Audits

f350b9e4646b Explicitly condition Differential draft promotion on only "impactful" builds
Author
epriestley
Browse Repository
No Builds
No Audits

Mar 12 2018

1e93b49b1bf2 Allow custom actions in Differential to explicitly override "accept" stickiness
Author
epriestley
Browse Repository
No Builds
No Audits

Mar 6 2018

d14a0f4787b5 Add "All" and "With Non-Owner Author" options for all Owners Package autoreview…
Author
epriestley
Browse Repository
No Builds
No Audits

743d1ac426fb Mostly modularize the Differential "update" transaction
Author
epriestley
Browse Repository
No Builds
No Audits

Feb 12 2018

6f508a225869 Update buildable containerPHIDs in a proper way via BuildWorker rather than via…
Author
epriestley
Browse Repository
No Builds
No Audits

Feb 8 2018

d0a2e3c54f9a Fix an issue where some Differential edit pathways may not have reviewers…
Author
epriestley
Browse Repository
No Builds
No Audits

1cd3a593784a When users resign from revisions, stop expanding projects/packages to include…
Author
epriestley
Browse Repository
No Builds
No Audits

f090fa7426e4 Use object PHIDs for "Thread-Topic" headers in mail
Author
epriestley
Browse Repository
No Builds
No Audits

Feb 2 2018

032f5b22941f Allow revisions to revert commits and one another, and commits to revert…
Author
epriestley
Browse Repository
No Builds
No Audits

Jan 26 2018

fd66ab55791e Always use the same set of transactions to generate mail and mail tags
Author
epriestley
Browse Repository
No Builds
No Audits

Jan 24 2018

dabd3f0b417b Fix a race between Harbormaster and reviewers (often bots) to publish drafts…
Author
epriestley
Browse Repository
No Builds
No Audits

Dec 18 2017

a1ad184ddd8c Denormalize added and removed line counts for the current diff onto revisions
Author
epriestley
Browse Repository
No Builds
No Audits

Dec 5 2017

6d3baa92f908 Execute Herald again after promoting revisions out of the "Draft" state
Author
epriestley
Browse Repository
No Builds
No Audits

Nov 28 2017

54f131dc4baa Make "first broadcast" rules for Differential drafts more general
Author
epriestley
Browse Repository
No Builds
No Audits

Nov 14 2017

314e7266c3e5 Restore "Summary" and "Test Plan" to initial mail for non-draft configurations
Author
epriestley
Browse Repository
No Builds
No Audits

Nov 1 2017

03d059dd26e2 Don't include resigned reviewers in the Differential "To" list
Author
epriestley
Browse Repository
No Builds
No Audits

6d36eb911309 Denormalize Diff PHIDs onto Revisions
Author
epriestley
Browse Repository
No Builds
No Audits

Oct 31 2017

a4b934cad2b5 Clean up Differential draft mail behaviors
Author
epriestley
Browse Repository
No Builds
No Audits

7fa0d066bc45 Don't run Herald build rules when Differential revisions are updated…
Author
epriestley
Browse Repository
No Builds
No Audits

28cec2f8a2bf Allow revisions to be held as drafts, even after builds finish
Author
epriestley
Browse Repository
No Builds
No Audits

Oct 27 2017

f7f3dd5b2084 Don't run Herald build and mail rules when they don't make sense
Author
epriestley
Browse Repository
No Builds
No Audits

Oct 26 2017

beaf0ad9a636 Attribute revision promotion from "Draft" to "Needs Review" to the author
Author
epriestley
Browse Repository
No Builds
No Audits

Oct 20 2017

1755ec242989 Show more detailed hints about draft revisions in the UI
Author
epriestley
Browse Repository
No Builds
No Audits

Sep 21 2017

36df39761ea5 Create revisions into "Draft", publish them when builds finish
Author
epriestley
Browse Repository
No Builds
No Audits

fca553f142a2 Prepare revision mail for the "Draft" status
Author
epriestley
Browse Repository
No Builds
No Audits

Sep 13 2017

6fb3f857fb57 Stop the bleeding caused by attaching enormous patches to revision mail
Author
epriestley
Browse Repository
No Builds
No Audits

Aug 24 2017

9639ec0dfabd Slightly simplify logic for determining if an inline comment has an effect
Author
epriestley
Browse Repository
No Builds
No Audits

Aug 12 2017

48a74de0b649 Move all revision status transactions to modern values and mechanics
Author
epriestley
Browse Repository
No Builds
No Audits

Aug 11 2017

5348f34c9eec Make all revision status readers explicitly read modern or legacy status
Author
epriestley
Browse Repository
No Builds
No Audits

7f743c14d532 Remove remaining ArcanistDifferentialRevisionStatus references in revision…
Author
epriestley
Browse Repository
No Builds
No Audits

2b9838b482af Modularize remaining TYPE_ACTION transactions in Differential, reducing calls…
Author
epriestley
Browse Repository
No Builds
No Audits

19bc91fd208d Modularize the Differential "status" transaction and move away from…
Author
epriestley
Browse Repository
No Builds
No Audits

153e4d8a383c Remove old reviewer double writes to legacy edge table in Differential
Author
epriestley
Browse Repository
No Builds
No Audits

Apr 30 2017

b7c4f60e235d Only recognize "Fixes ..." in main revision content like the Summary or Test…
Author
epriestley
Browse Repository
No Builds
No Audits

Apr 11 2017

af1d494d66e9 Fix an issue where rejecting reviewers weren't powerful enough
Author
epriestley
Browse Repository
No Builds
No Audits

Apr 4 2017

f70ff34d97e8 Fix a copy/paste typo with sticky accept
Author
epriestley
Browse Repository
No Builds
No Audits

Apr 3 2017

9ebb5f8cda56 Don't downgrade accepts on update (fix "sticky accept")
Author
epriestley
Browse Repository
No Builds
No Audits

Mar 28 2017

2fbc9a52da9b Allow users to "Force accept" package reviews if they own a more general package
Author
epriestley
Browse Repository
No Builds
No Audits

ddc02ce420a5 When voiding "Accept" reviews, also void "Reject" reviews
Author
epriestley
Browse Repository
No Builds
No Audits

415ad784844e Remove old code for "Request Review" action from Differential
Author
epriestley
Browse Repository
No Builds
No Audits

aea46e55dac0 Fix an issue where "Request Review" of a fully-accepted revision would…
Author
epriestley
Browse Repository
No Builds
No Audits

Mar 20 2017

0ceab7d36f91 Rename "getReviewerStatus()" to "getReviewers()"
Author
epriestley
Browse Repository
No Builds
No Audits

a15df4f8d580 Rename "needReviewerStatus()" into "needReviewers()"
Author
epriestley
Browse Repository
No Builds
No Audits

794b456530bc Store "last comment" and "last action" diffs on reviewers
Author
epriestley
Browse Repository
No Builds
No Audits

77b3efafbd4e Use ModularTransactions for accept/reject/resign in "differential.createcomment"
Author
epriestley
Browse Repository
No Builds
No Audits

a9cbbf3e5e15 Apply Owners reviewers using ModularTransactions
Author
epriestley
Browse Repository
No Builds
No Audits

Jan 10 2017

00e2755eab62 Provide tailored strings for revision creation
Author
epriestley
Browse Repository
No Builds
No Audits

Jan 5 2017

1f2306999bb5 Fix a case where "Accept + Comment" would ignore the "Accept"
Author
epriestley
Browse Repository
No Builds
No Audits

Dec 31 2016

69194fdaf510 Make marking comments as "Done" work cleanly on EditEngine
Author
epriestley
Browse Repository
No Builds
No Audits

b373dcef7448 Restore some minor state behaviors to Differential on EditEngine
Author
epriestley
Browse Repository
No Builds
No Audits

Dec 13 2016

77fa1ea73835 Rename "DifferentialReviewer" to "DifferentialReviewerProxy"
Author
epriestley
Browse Repository
No Builds
No Audits

Oct 17 2016

dad17fb98a6b Make "metamta.differential.inline-patches" imply a reasonable byte limit, not…
Author
epriestley
Browse Repository
No Builds
No Audits

Sep 16 2016

7f6fa283639f When loading packages affected by a change to a particular path, ignore…
Author
epriestley
Browse Repository
No Builds
No Audits

Aug 14 2016

15021a0bcc91 Fix bad array index test in Differential package code
Author
epriestley
Browse Repository
No Builds
No Audits

Jun 27 2016

de6349dd67af Revision substate CLOSED_FROM_ACCEPTED
Author
Aviv Eyal
Browse Repository
No Builds
No Audits

Jun 13 2016

65634781b44f Don't re-mention users for comment edits
Author
epriestley
Browse Repository
No Builds
No Audits

May 18 2016

5bb3cbe23922 Add a "View Revision" button to HTML email
Author
Chad Little
Browse Repository
No Builds
No Audits

May 17 2016

809c7bf996af Allow users to manage package dominion rules
Author
epriestley
Browse Repository
No Builds
No Audits

May 13 2016

c9365e48d8e4 Don't trigger "Auto Review" if the author is already an owner; document "Auto…
Author
epriestley
Browse Repository
No Builds
No Audits

332d787dc8f4 Support "Review Changes" and "Block Changes" settings for Owners package "Auto…
Author
epriestley
Browse Repository
No Builds
No Audits

52ac242eb3b2 Implement "Auto Review" in packages with a "Subscribe" option
Author
epriestley
Browse Repository
No Builds
No Audits

May 12 2016

bd9bcaa8ffea Improve HTML mail rendering of inline patches
Author
epriestley
Browse Repository
No Builds
No Audits

May 6 2016

371051ff3779 Minor tweaks to pre/inline style for inline comments in HTML mail
Author
epriestley
Browse Repository
No Builds
No Audits

May 5 2016

94c7bb605c35 Highlight inline diff context in HTML mail
Author
epriestley
Browse Repository
No Builds
No Audits

2025ecd3d826 Rough cut of inline comment context
Author
epriestley
Browse Repository
No Builds
No Audits