Page MenuHomePhabricator
Differential D20135

Fix inverted check in audit triggers for "uninvolved owner"
ClosedPublic
Actions

Authored by epriestley on Feb 11 2019, 2:57 PM.
Tags
None
Referenced Files
F13178973: D20135.diff
Wed, May 8, 8:50 PM
Unknown Object (File)
Sat, May 4, 6:40 PM
Unknown Object (File)
Thu, May 2, 7:30 AM
Unknown Object (File)
Tue, Apr 30, 11:23 PM
Unknown Object (File)
Sat, Apr 27, 4:58 PM
Unknown Object (File)
Sat, Apr 27, 10:21 AM
Unknown Object (File)
Mon, Apr 22, 10:40 PM
Unknown Object (File)
Fri, Apr 19, 3:34 PM
View All 94 Files
Subscribers
None

Details

Reviewers
amckinley
Commits
rP4f40cb0ff9c6: (stable) Fix inverted check in audit triggers for "uninvolved owner"
rP77247084bd36: Fix inverted check in audit triggers for "uninvolved owner"
Summary

See D20126. I was trying to be a little too cute here with the names and ended up confusing myself, then just tested the method behavior. :/

Test Plan

Persudaded by arguments in D20126.

Diff Detail

Repository
rP Phabricator
Branch
audit1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21915
Build 29919: Run Core Tests
Build 29918: arc lint + arc unit

Revision Contents
Changeset List

PathSize
src/
applications/
repository/
worker/
4 lines

Diff 48076

View Options

src/applications/repository/worker/PhabricatorRepositoryCommitOwnersWorker.php

Loading...
Phabricator · Built by Phacility