Differential D13028

Use ApplicationTransactions for all non-path edits to Owners packages
ClosedPublic
Actions

Authored by epriestley on May 26 2015, 11:22 PM.

Tags

None

Referenced Files

	F15476818: D13028.id31447.diff
	Mon, Apr 7, 9:33 AM

	F15468024: D13028.id.diff
	Thu, Apr 3, 7:26 PM

	F15465182: D13028.diff
	Wed, Apr 2, 3:44 PM

	F15442455: D13028.id31447.diff
	Wed, Mar 26, 11:31 PM

	F15311867: D13028.id31416.diff
	Mar 6 2025, 11:50 AM

	F15311865: D13028.id31447.diff
	Mar 6 2025, 11:50 AM

	F15308923: D13028.diff
	Mar 6 2025, 6:46 AM

	F15308376: D13028.diff
	Mar 6 2025, 6:12 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T8320: Bring Owners out of the dark ages

Commits

Restricted Diffusion Commit
rPda9a61fb70db: Use ApplicationTransactions for all non-path edits to Owners packages

Summary

Ref T8320.

Test Plan

Screen Shot 2015-05-26 at 4.21.27 PM.png (327×643 px, 54 KB)

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 31416.May 26 2015, 11:22 PM

epriestley retitled this revision from to Use ApplicationTransactions for all non-path edits to Owners packages.

epriestley updated this object.

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

epriestley added a reviewer: btrahan.

epriestley added a task: T8320: Bring Owners out of the dark ages.

Herald added a subscriber: epriestley. · View Herald TranscriptMay 26 2015, 11:22 PM

btrahan accepted this revision.May 27 2015, 5:14 PM

btrahan edited edge metadata.

btrahan added inline comments.

resources/sql/autopatches/20150526.owners.mailkey.2.php
2–19	maybs we should support this in automagically in bin/storage ? I guess we're probably too low on legacy code to make that worthwhile now.

This revision is now accepted and ready to land.May 27 2015, 5:14 PM

epriestley added inline comments.May 27 2015, 5:22 PM

resources/sql/autopatches/20150526.owners.mailkey.2.php
2–19	Yeah, it does feel a bit like a ragged edge but if there are only like 4 more of them it doesn't feel worthwhile to generalize. I'm not sure if the real number is ~4 or ~20. Seems like it should probably be closer to 4 by now, though. Hopefully.

Closed by commit rPda9a61fb70db: Use ApplicationTransactions for all non-path edits to Owners packages (authored by epriestley, committed by epriestley). · Explain WhyMay 27 2015, 5:30 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

sql/

autopatches/

20150526.owners.mailkey.1.sql

2 lines

20150526.owners.mailkey.2.php

18 lines

20150526.owners.xaction.sql

19 lines

src/

__phutil_library_map__.php

7 lines

applications/

owners/

controller/

PhabricatorOwnersDetailController.php

6 lines

PhabricatorOwnersEditController.php

154 lines

editor/

PhabricatorOwnersPackageTransactionEditor.php

240 lines

query/

PhabricatorOwnersPackageTransactionQuery.php

10 lines

storage/

PhabricatorOwnersPackage.php

46 lines

PhabricatorOwnersPackageTransaction.php

152 lines

infrastructure/

internationalization/

translation/

PhabricatorUSEnglishTranslation.php

18 lines

Diff 31447

resources/sql/autopatches/20150526.owners.mailkey.1.sql

Loading...

resources/sql/autopatches/20150526.owners.mailkey.2.php

Loading...

resources/sql/autopatches/20150526.owners.xaction.sql

Loading...

src/__phutil_library_map__.php

Loading...

src/applications/owners/controller/PhabricatorOwnersDetailController.php

Loading...

src/applications/owners/controller/PhabricatorOwnersEditController.php

Loading...

src/applications/owners/editor/PhabricatorOwnersPackageTransactionEditor.php

Loading...

src/applications/owners/query/PhabricatorOwnersPackageTransactionQuery.php

Loading...

src/applications/owners/storage/PhabricatorOwnersPackage.php

Loading...

src/applications/owners/storage/PhabricatorOwnersPackageTransaction.php

Loading...

src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php

Loading...