Fix issue with ref/fixes in commit messages
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F15514271: D8519.id20215.diff
	Fri, Apr 18, 1:19 AM

	F15512922: D8519.id.diff
	Thu, Apr 17, 2:56 PM

	F15508822: D8519.diff
	Wed, Apr 16, 6:44 AM

	F15476932: D8519.id20223.diff
	Mon, Apr 7, 10:12 AM

	F15458603: D8519.id20215.diff
	Mon, Mar 31, 5:22 AM

	F15444101: D8519.id.diff
	Thu, Mar 27, 7:52 AM

	F15382173: D8519.id20223.diff
	Mar 14 2025, 11:05 AM

	F15381844: D8519.id.diff
	Mar 14 2025, 9:36 AM

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

Lint

Lint Skipped

Unit

Tests Skipped

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 20223

View Options

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

Fix issue with ref/fixes in commit messagesClosedPublicActions