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

	F17665252: D13028.id.diff
	Sat, Jul 12, 8:03 PM

	F17657967: D13028.id31416.diff
	Sat, Jul 12, 12:44 PM

	F17651647: D13028.id31447.diff
	Sat, Jul 12, 5:55 AM

	F17639358: D13028.diff
	Fri, Jul 11, 5:33 PM

	Unknown Object (File)
	Sun, Jun 29, 11:05 AM

	Unknown Object (File)
	May 28 2025, 10:24 PM

	Unknown Object (File)
	May 26 2025, 12:24 AM

	Unknown Object (File)
	May 23 2025, 4:24 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...