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

	Unknown Object (File)
	Fri, Jan 24, 7:03 PM

	Unknown Object (File)
	Fri, Jan 24, 7:03 PM

	Unknown Object (File)
	Fri, Jan 24, 7:03 PM

	Unknown Object (File)
	Thu, Jan 23, 5:07 AM

	Unknown Object (File)
	Tue, Jan 21, 6:55 PM

	Unknown Object (File)
	Wed, Jan 15, 4:17 PM

	Unknown Object (File)
	Sun, Jan 5, 1:12 PM

	Unknown Object (File)
	Dec 20 2024, 8:22 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