Fix issue with ref/fixes in commit messages
ClosedPublic
Actions

Authored by epriestley on Mar 13 2014, 10:04 PM.

Tags

None

Referenced Files

	F19669481: D8519.id.diff
	Sat, Feb 7, 9:08 PM

	F19571846: D8519.id20223.diff
	Sun, Feb 1, 2:47 AM

	F19509963: D8519.diff
	Jan 11 2026, 3:30 AM

	F19094354: D8519.diff
	Dec 4 2025, 12:41 AM

	F19032224: D8519.diff
	Nov 25 2025, 5:10 AM

	F18997683: D8519.id.diff
	Nov 20 2025, 2:34 PM

	F18956953: D8519.id20215.diff
	Nov 12 2025, 11:09 PM

	F18944974: D8519.id20223.diff
	Nov 11 2025, 5:25 PM

Subscribers

Details

Reviewers

btrahan

Maniphest Tasks

T4600: "Fixes ___" in differentials and commits no longer closes tasks

Commits

Restricted Diffusion Commit
rP1d303542235d: Fix issue with ref/fixes in commit messages

Summary

Fixes T4600. If there's also a revision, the variable "$message" gets overwritten. groan~

Test Plan

Pushed a commit with "Fixes T123" and a revision, saw it parse on the first try.

Diff Detail

Repository

rP Phabricator

Branch

shadowvar

Lint

Lint Passed

Unit

Tests Passed

Event Timeline

epriestley updated this revision to Diff 20215.Mar 13 2014, 10:04 PM

epriestley retitled this revision from to Fix issue with ref/fixes in commit messages.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

epriestley added a task: T4600: "Fixes ___" in differentials and commits no longer closes tasks.

Herald added subscribers: epriestley, aran.Mar 13 2014, 10:04 PM

btrahan accepted this revision.Mar 14 2014, 12:40 AM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Mar 14 2014, 12:40 AM

Closed by commit rP1d303542235d (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

repository/

worker/

commitmessageparser/

PhabricatorRepositoryCommitMessageParserWorker.php

6 lines

Diff 20215

View Options

src/applications/repository/worker/commitmessageparser/PhabricatorRepositoryCommitMessageParserWorker.php

Fix issue with ref/fixes in commit messagesClosedPublicActions