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

	F13091022: D17343.diff
	Thu, Apr 25, 2:42 AM

	Unknown Object (File)
	Tue, Apr 23, 9:08 PM

	Unknown Object (File)
	Tue, Apr 23, 9:08 PM

	Unknown Object (File)
	Tue, Apr 23, 9:08 PM

	Unknown Object (File)
	Tue, Apr 23, 9:07 PM

	Unknown Object (File)
	Sat, Apr 20, 11:51 AM

	Unknown Object (File)
	Thu, Apr 11, 9:56 AM

	Unknown Object (File)
	Tue, Apr 9, 4:45 AM

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 41710

View Options

src/applications/diffusion/controller/DiffusionCommitController.php

Fix missing setQuoteRef() on Commit detail pages in DiffusionClosedPublicActions