Page MenuHomePhabricator
Differential D21598

Remove very old Audit status constants and AuditRequest data
ClosedPublic
Actions

Authored by epriestley on Mar 10 2021, 12:01 AM.
Tags
None
Referenced Files
F18817021: D21598.id.diff
Tue, Oct 21, 11:30 AM
F18813286: D21598.id51422.diff
Mon, Oct 20, 3:07 PM
F18602335: D21598.id.diff
Sep 13 2025, 2:49 PM
F18591055: D21598.diff
Sep 12 2025, 8:06 AM
F18571723: D21598.id51422.diff
Sep 10 2025, 5:19 AM
F18571722: D21598.id51418.diff
Sep 10 2025, 5:19 AM
F18506383: D21598.diff
Sep 5 2025, 1:25 AM
F18126128: D21598.id.diff
Aug 13 2025, 2:17 PM
View All Files
Subscribers
Restricted Owners Package

Details

Reviewers
None
Maniphest Tasks
T13631: Support an "auditors" attachment for "diffusion.commit.search"
Commits
rP2636d84d0c24: Remove very old Audit status constants and AuditRequest data
Summary

Ref T13631. See that task for discussion.

  • "NONE": Probably never used?
  • "CC": Obsoleted by subscribers.
  • "AUDIT_NOT_REQUIRED": For Owners packages, obsoleted by edges.
  • "CLOSED": For "Close Audit", obsoleted by "Request Verification".
Test Plan
  • Grepped for constants, browsed Diffusion.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 10 2021, 12:01 AM
Owners added a subscriber: Restricted Owners Package.Mar 10 2021, 12:01 AM
Harbormaster completed remote builds in B25225: Diff 51418.Mar 10 2021, 12:02 AM
epriestley requested review of this revision.Mar 10 2021, 12:02 AM
epriestley added inline comments.Mar 10 2021, 12:08 AM
src/applications/repository/storage/PhabricatorRepositoryAuditRequest.php
52

After these changes, this just means "is not-resigned", and we already have "isResigned()" to test for that.

65–73

After these changes, all audit requests are "interesting".

This revision was not accepted when it landed; it landed in state Needs Review.Mar 10 2021, 5:22 PM
Closed by commit rP2636d84d0c24: Remove very old Audit status constants and AuditRequest data (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP2636d84d0c24: Remove very old Audit status constants and AuditRequest data.

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
5 lines
src/
applications/
audit/
constants/
18 lines
editor/
7 lines
diffusion/
controller/
4 lines
herald/
4 lines
6 lines
repository/
storage/
23 lines

Diff 51422

View Options

resources/sql/autopatches/20210309.auditors.01.status.sql

Loading...
View Options

src/applications/audit/constants/PhabricatorAuditStatusConstants.php

Loading...
View Options

src/applications/audit/editor/PhabricatorAuditEditor.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionCommitController.php

Loading...
View Options

src/applications/diffusion/herald/DiffusionAuditorsHeraldAction.php

Loading...
View Options

src/applications/diffusion/herald/DiffusionCommitAuditorsHeraldField.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepositoryAuditRequest.php

Loading...
Phabricator · Built by Phacility