Differential D10928

Transactions - don't bother checking for fancy transactions on comment create
ClosedPublic
Actions

Authored by btrahan on Dec 4 2014, 7:58 PM.

Tags

None

Referenced Files

	F13206657: D10928.diff
	Wed, May 15, 2:39 PM

	F13204561: D10928.diff
	Wed, May 15, 12:42 AM

	F13187554: D10928.diff
	Sat, May 11, 4:36 AM

	Unknown Object (File)
	Tue, May 7, 7:46 AM

	Unknown Object (File)
	Fri, May 3, 7:23 AM

	Unknown Object (File)
	Sat, Apr 27, 9:46 PM

	Unknown Object (File)
	Thu, Apr 25, 1:31 AM

	Unknown Object (File)
	Apr 11 2024, 9:03 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T6690: Exception when creating new Conpherence threads

Commits

Restricted Diffusion Commit
rPa9f0bd9b8fbf: Transactions - don't bother checking for fancy transactions on comment create

Summary

Only necessary for edits, only bother if the comment version is greater than 1. Ref T6690. This is another way to fix T6690 -- this check will never run since you can't edit a conpherence comment -- but the fix already applied should happen too to future proof Conpherence.

Test Plan

made a comment on a diff - success. edited the comment and mentions were generated.

Diff Detail

Repository

Branch

morebugz

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 3176
Build 3182: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan updated this revision to Diff 26248.Dec 4 2014, 7:58 PM

btrahan retitled this revision from to Transactions - don't bother checking for fancy transactions on comment create.

btrahan updated this object.

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

btrahan added a reviewer: epriestley.

btrahan added a task: T6690: Exception when creating new Conpherence threads.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptDec 4 2014, 7:58 PM

epriestley accepted this revision.Dec 4 2014, 8:01 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Dec 4 2014, 8:01 PM

Closed by commit rPa9f0bd9b8fbf: Transactions - don't bother checking for fancy transactions on comment create (authored by btrahan, committed by btrahan). · Explain WhyDec 4 2014, 8:04 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

transactions/

editor/

PhabricatorApplicationTransactionCommentEditor.php

36 lines

Diff 26248

src/applications/transactions/editor/PhabricatorApplicationTransactionCommentEditor.php

Loading...