From 00136f05dd6d3e2bbc8772b12dacaa23ea805307 Mon Sep 17 00:00:00 2001 From: epriestley Date: Sun, 17 May 2015 07:18:44 -0700 Subject: [PATCH] Allow adding Auditors with the singular noun. Summary: Currently adding auditors in commit message only works with the plural. Make "Auditor: bla" work. Test Plan: Unit tests. I don't know if I am supposed to add a test. Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: joshuaspence, epriestley Differential Revision: https://secure.phabricator.com/D10636 --- src/applications/audit/editor/PhabricatorAuditEditor.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/applications/audit/editor/PhabricatorAuditEditor.php b/src/applications/audit/editor/PhabricatorAuditEditor.php index 587af71850..d30a4061b7 100644 --- a/src/applications/audit/editor/PhabricatorAuditEditor.php +++ b/src/applications/audit/editor/PhabricatorAuditEditor.php @@ -418,7 +418,7 @@ final class PhabricatorAuditEditor $message = $data->getCommitMessage(); $matches = null; - if (!preg_match('/^Auditors:\s*(.*)$/im', $message, $matches)) { + if (!preg_match('/^Auditors?:\s*(.*)$/im', $message, $matches)) { return array(); }