1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-05 05:02:44 +01:00
phorge-phorge/src/applications/differential
epriestley 65634781b4 Don't re-mention users for comment edits
Summary:
Ref T11035. This only fixes half of the issue: comment editing has been fixed, but normal transactions which edit things like descriptions haven't yet.

The normal edits aren't fixed because the "oldValues" are populated too late. The code should start working once they get populated sooner, but I don't want to jump the gun on that since it'll probably have some spooky effects. I have some other transaction changes coming down the pipe which should provide a better context for testing "oldValue" population order.

Test Plan:
  - Mentioned `@dog` in a comment.
  - Removed `@dog` as a subscriber.
  - Edited the comment, adding some unrelated text at the end (e.g., fixing a typo).
    - Before change: `@dog` re-added as subscriber.
    - After change: no re-add.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T11035

Differential Revision: https://secure.phabricator.com/D16108
2016-06-13 13:57:59 -07:00
..
__tests__ phtize all the things 2015-05-22 21:16:39 +10:00
application Use new Differential bucketing logic on default (non-dashboard) homepage 2016-05-19 15:20:39 -07:00
capability Simplify the implementation of PhabricatorPolicyCapability subclasses 2014-07-25 08:25:42 +10:00
command Generate mail command documentation from active commands 2015-04-01 11:51:47 -07:00
conduit Implement basic differential.revision.search 2016-06-09 09:06:58 -07:00
config Modernize metamta.differential.patch-format 2016-06-04 12:17:14 +00:00
constants Update Diffusion UI 2016-03-17 12:05:14 -07:00
controller Modernize various menu collapse settings 2016-06-04 14:44:36 -07:00
customfield Fix "Reviewers" validation issue with empty reviewers 2016-05-23 17:16:21 -07:00
doorkeeper Remove email prefixes from doorkeeper titles 2016-01-28 18:48:28 +00:00
edge Don't write inverse edges for DifferentialRevisionHasReviewerEdgeType 2015-01-04 10:07:56 +11:00
editor Don't re-mention users for comment edits 2016-06-13 13:57:59 -07:00
engine Make diffusion.filecontentquery return file PHIDs instead of raw content 2016-01-08 09:29:16 -08:00
engineextension Convert Differential to new layout 2016-03-12 13:04:21 -08:00
event Modernize Hovercard implementation 2015-12-24 12:18:28 -08:00
exception Remove an unused class 2015-01-03 09:07:32 +11:00
garbagecollector Provide bin/garbage for interacting with garbage collection 2015-10-02 09:17:24 -07:00
herald Allow Herald to add package reviewers 2016-05-13 17:23:07 -07:00
landing Make "Land Revision" button state consistent, prevent non-accepted lands 2015-10-27 18:51:59 +00:00
lipsum Improve lipsum generation of projects 2015-12-24 09:06:56 -08:00
mail Improve HTML mail rendering of inline patches 2016-05-12 12:13:40 -07:00
management Modularize content sources 2016-03-26 11:59:45 -07:00
parser Convert some loadPreferences() to getUserSetting() 2016-06-02 06:29:20 -07:00
phid Mark PhabricatorPHIDType::getPHIDTypeApplicationClass() as abstract 2015-11-03 06:47:12 +11:00
query Add datasources to allow search revisions by project. 2016-06-13 18:08:44 +00:00
remarkup Rename PhutilRemarkupRule subclasses 2014-08-05 00:55:43 +10:00
render Render more info for binary files in Differential 2016-06-06 19:58:39 +00:00
search Convert all "DocumentIndexers" into "FulltextEngines" 2015-12-21 17:25:23 -08:00
storage Implement basic differential.revision.search 2016-06-09 09:06:58 -07:00
typeahead Add datasources to allow search revisions by project. 2016-06-13 18:08:44 +00:00
view Allow blocking reviewers to be added via the web UI 2016-05-17 10:56:12 -07:00
DifferentialGetWorkingCopy.php Extend from Phobject 2015-06-15 18:02:27 +10:00