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

Authored by epriestley on Mon, Feb 11, 2:57 PM.

Details

Reviewers

Commits

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.

Repository

rP Phabricator

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

epriestley requested review of this revision.Mon, Feb 11, 2:59 PM

This revision is now accepted and ready to land.Mon, Feb 11, 6:12 PM

This revision was automatically updated to reflect the committed changes.

			Path	Packages
M			src/applications/repository/worker/PhabricatorRepositoryCommitOwnersWorker.php (4 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	48076	a20f108		Mon, Feb 11, 2:57 PM	★	★
Diff 2	48089	711871f	rP77247084bd365e9020b942a6ac37b367dad1f4a7	Mon, Feb 11, 10:08 PM	★	★