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

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

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

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

	Unknown Object (File)
	Mon, Apr 8, 1:31 AM

	Unknown Object (File)
	Mon, Apr 1, 8:12 AM

	Unknown Object (File)
	Mar 28 2024, 6:14 AM

	Unknown Object (File)
	Mar 10 2024, 7:48 AM

	Unknown Object (File)
	Mar 4 2024, 4:52 PM

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 26249

src/applications/transactions/editor/PhabricatorApplicationTransactionCommentEditor.php

Loading...