1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-24 06:20:56 +01:00
phorge-phorge/webroot/rsrc/js/application
epriestley 58f66fea80 Allow users to remove their own comments, and administrators to remove any comment
Summary:
Fixes T4909. Adds a "remove" link next to the edit link, which permanently hides a comment. Addresses two use cases:

  - Allowing administrators to clean up spam.
  - Allowing users to try to put the genie back in the bottle if they post passwords or sensitive links, etc.

The user who removed the comment is named in the removal text to enforce some level of administrative accountability.

No data is deleted, but there's currently no method to restore these comments. We'll see if we need one.

This is cheating a little bit by storing "removed" as "2" in the isDeleted field. This doesn't seem tooooo bad for now.

Test Plan:
  - Removed some of my comments.
  - As an administrator, removed other users' comments.
  - Failed to view history of a removed comment.
  - Failed to edit a removed comment.
  - Failed to remove a removed comment.
  - Verified feed doesn't show the old comment after comment removal.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: qgil, chad, epriestley

Maniphest Tasks: T4909

Differential Revision: https://secure.phabricator.com/D8945
2014-05-05 10:55:32 -07:00
..
aphlict Make the quick create menu more or less work correctly 2014-01-28 20:19:20 -08:00
auth Add Persona auth provider 2013-10-14 14:34:57 -07:00
config Various linter fixes. 2014-02-26 12:44:58 -08:00
conpherence Fix some issues where Conpherence would make to many draft requests 2014-03-01 11:23:08 -08:00
countdown Various linter fixes. 2014-02-26 12:44:58 -08:00
dashboard Give dashboards basic rendering and Ajax support 2014-04-30 14:29:14 -07:00
differential Move lint/unit test warning code forward to Transactions 2014-03-11 13:02:18 -07:00
diffusion Various linter fixes. 2014-02-26 12:44:58 -08:00
doorkeeper Various linter fixes. 2014-02-26 12:44:58 -08:00
files Various linter fixes. 2014-02-26 12:44:58 -08:00
harbormaster Various linter fixes. 2014-02-26 12:44:58 -08:00
herald Herald - make tokenizers have the purdy icons 2014-04-10 12:38:15 -07:00
maniphest Despecialize most task status handling 2014-03-25 13:47:42 -07:00
owners Allow excluding paths from package 2012-12-07 16:33:16 -08:00
passphrase Add "allow null" and username hinting to the Passphrase credential control 2013-11-22 14:35:35 -08:00
phame Use JsShrink if jsxmin is not available 2013-05-18 17:04:22 -07:00
pholio Pholio - back end for image re-ordering 2013-08-12 13:09:07 -07:00
phortune Allow PhortuneTestProvider to add payment methods 2013-04-25 09:50:08 -07:00
policy Allow "Custom" policies to be selected in the policy control 2013-10-14 16:59:16 -07:00
ponder Use PHUIRemarkupPreviewPanel in Ponder 2013-08-05 10:47:06 -07:00
projects Workboards - make priority changes less aggressive and generally better 2014-03-27 10:50:54 -07:00
releeph Use standard UI elements to render pull requests in Releeph 2014-04-18 06:44:45 -07:00
repository Removed getElementsByClassName in repositor-crossreference.js 2013-05-12 06:54:15 -07:00
search Various linter fixes. 2014-02-26 12:44:58 -08:00
slowvote Use JsShrink if jsxmin is not available 2013-05-18 17:04:22 -07:00
transactions Allow users to remove their own comments, and administrators to remove any comment 2014-05-05 10:55:32 -07:00
uiexample Fix more JavelinView example stuff 2013-07-16 13:46:49 -07:00