Fix missing setQuoteRef() on Commit detail pages in Diffusion
ClosedPublic
Actions

Authored by epriestley on Feb 13 2017, 3:42 PM.

Tags

None

Referenced Files

	F19034112: D17343.id.diff
	Tue, Nov 25, 1:09 PM

	F18831479: D17343.id.diff
	Oct 25 2025, 1:19 PM

	F18826873: D17343.diff
	Oct 24 2025, 6:09 AM

	F18807962: D17343.diff
	Oct 19 2025, 4:13 AM

	F18733466: D17343.id.diff
	Sep 30 2025, 10:35 PM

	F18680263: D17343.id41709.diff
	Sep 26 2025, 2:27 AM

	F18658399: D17343.diff
	Sep 23 2025, 6:14 AM

	F18267059: D17343.id.diff
	Aug 22 2025, 1:08 PM

View All Files

Subscribers

None

Details

Reviewers

avivey
chad

Maniphest Tasks

T12253: null in remarkup rather than comment reference when quoting a comment in audit

Commits

rP037c749ef332: Fix missing setQuoteRef() on Commit detail pages in Diffusion

Summary

Fixes T12253.

Test Plan

Before change: used "Quote Comment", saw "In null, alice wrote:" in quoted text.
After change: used "Quote Comment", saw proper reference to the commit/page. Clicked reference, was sent to the comment properly.

Screen Shot 2017-02-13 at 7.39.49 AM.png (523×715 px, 59 KB)

Diff Detail

Repository

rP Phabricator

Branch

quote1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 15608
Build 20597: Run Core Tests
Build 20596: arc lint + arc unit

Event Timeline

epriestley created this revision.Feb 13 2017, 3:42 PM

Harbormaster completed remote builds in B15608: Diff 41709.Feb 13 2017, 3:43 PM

¯\_(ツ)_/¯

This revision is now accepted and ready to land.Feb 13 2017, 3:43 PM

Closed by commit rP037c749ef332: Fix missing setQuoteRef() on Commit detail pages in Diffusion (authored by epriestley, committed by epriestley). · Explain WhyFeb 13 2017, 3:44 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

controller/

DiffusionCommitController.php

4 lines

Diff 41709

View Options

src/applications/diffusion/controller/DiffusionCommitController.php

Fix missing setQuoteRef() on Commit detail pages in DiffusionClosedPublicActions