Paths

Table of Contentst

Differential D17947

Migrate Project names to modular transactions
ClosedPublic
Actions

Authored by amckinley on May 18 2017, 12:01 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Feb 9, 4:08 AM

	Unknown Object (File)
	Sun, Feb 9, 4:08 AM

	Unknown Object (File)
	Sun, Feb 9, 4:07 AM

	Unknown Object (File)
	Sun, Feb 9, 4:07 AM

	Unknown Object (File)
	Mon, Feb 3, 4:24 AM

	Unknown Object (File)
	Tue, Jan 28, 2:36 AM

	Unknown Object (File)
	Sun, Jan 26, 7:25 AM

	Unknown Object (File)
	Fri, Jan 24, 9:07 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T12673: Update Projects for Modular Transactions

Commits

rP1e3c8df1c89d: Migrate Project names to modular transactions

Summary

Also changes access modifiers on PhabricatorProjectTransactionEditor and sets up storage for applyExternalEffects.

Test Plan

Created new projects, attempted to create without name, with too long of a name, and with a name that conflicts with other projects and observed expected errors.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

amckinley created this revision.May 18 2017, 12:01 AM

Herald added a subscriber: epriestley. · View Herald TranscriptMay 18 2017, 12:01 AM

Harbormaster completed remote builds in B17079: Diff 43163.May 18 2017, 12:02 AM

This all looks good to me.

src/applications/project/xaction/PhabricatorProjectNameTransaction.php
21	In theory, you could also `$this->generateOldValue($object)` to avoid needing to provide `$storage`, but I think your fix here is actually cleaner.

This revision is now accepted and ready to land.May 18 2017, 12:06 AM

Closed by commit rP1e3c8df1c89d: Migrate Project names to modular transactions (authored by amckinley, committed by amckinley). · Explain WhyMay 18 2017, 12:12 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

sql/

patches/

20131020.pxactionmig.php

2 lines

src/

__phutil_library_map__.php

6 lines

applications/

project/

__tests__/

PhabricatorProjectCoreTestCase.php

13 lines

conduit/

ProjectCreateConduitAPIMethod.php

2 lines

editor/

PhabricatorProjectTransactionEditor.php

80 lines

engine/

PhabricatorProjectEditEngine.php

2 lines

lipsum/

PhabricatorProjectTestDataGenerator.php

2 lines

storage/

PhabricatorProjectTransaction.php

35 lines

xaction/

PhabricatorProjectNameTransaction.php

112 lines

PhabricatorProjectTransactionType.php

4 lines

transactions/

editor/

PhabricatorApplicationTransactionEditor.php

2 lines

Diff 43164

resources/sql/patches/20131020.pxactionmig.php

Loading...

src/__phutil_library_map__.php

Loading...

src/applications/project/tests/PhabricatorProjectCoreTestCase.php

Loading...

src/applications/project/conduit/ProjectCreateConduitAPIMethod.php

Loading...

src/applications/project/editor/PhabricatorProjectTransactionEditor.php

Loading...

src/applications/project/engine/PhabricatorProjectEditEngine.php

Loading...

src/applications/project/lipsum/PhabricatorProjectTestDataGenerator.php

Loading...

src/applications/project/storage/PhabricatorProjectTransaction.php

Loading...

src/applications/project/xaction/PhabricatorProjectNameTransaction.php

Loading...

src/applications/project/xaction/PhabricatorProjectTransactionType.php

Loading...

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

Loading...