Page MenuHomePhabricator

Remove all readers/writers for task "subpriority"
ClosedPublic

Authored by epriestley on Mar 11 2019, 12:32 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 19, 8:01 AM
Unknown Object (File)
Fri, Apr 19, 7:11 AM
Unknown Object (File)
Fri, Apr 19, 6:00 AM
Unknown Object (File)
Fri, Apr 19, 3:03 AM
Unknown Object (File)
Thu, Apr 18, 12:28 AM
Unknown Object (File)
Wed, Apr 17, 10:31 PM
Unknown Object (File)
Sat, Apr 13, 9:43 AM
Unknown Object (File)
Wed, Apr 3, 6:35 PM
Subscribers
None

Details

Summary

Depends on D20265. Ref T10333. Now that neither task lists nor workboards use subpriority, we can remove all the readers and writers.

I'm not actually getting rid of the column data yet, but anticipate doing that in a future change.

Note that the subpriority algorithm (removed here) is possibly better than the "natural order" algorithm still in use. It's a bit more clever, and likely performs far fewer writes. I might make the "natural order" code use an algorithm more similar to the "subpriority" algorithm in the future.

Test Plan

Grepped for subpriority.

Diff Detail

Repository
rP Phabricator
Branch
grip3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22243
Build 30411: Run Core Tests
Build 30410: arc lint + arc unit

Event Timeline

amckinley added inline comments.
src/applications/maniphest/editor/ManiphestTransactionEditor.php
404–406

This is nuts; I had no idea this code was so complicated.

This revision is now accepted and ready to land.Mar 12 2019, 6:54 PM

Oh, I didn't submit this comment:

src/applications/maniphest/editor/ManiphestTransactionEditor.php
404–421

Particularly, this is the clever bit.

This revision was automatically updated to reflect the committed changes.