Page MenuHomePhabricator
Differential D7602

Restore population of `ownerOrdering` to ManiphestTasks
ClosedPublic
Actions

Authored by epriestley on Nov 18 2013, 10:15 PM.
Tags
None
Referenced Files
F14420357: D7602.diff
Wed, Dec 25, 5:29 AM
Unknown Object (File)
Sat, Dec 14, 12:52 AM
Unknown Object (File)
Mon, Dec 9, 4:27 PM
Unknown Object (File)
Thu, Dec 5, 4:55 PM
Unknown Object (File)
Tue, Dec 3, 5:49 PM
Unknown Object (File)
Tue, Dec 3, 2:37 PM
Unknown Object (File)
Wed, Nov 27, 8:53 AM
Unknown Object (File)
Tue, Nov 26, 1:55 PM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Maniphest Tasks
T4110: Maniphest `ownerOrdering` column is no longer populating, disrupting "Group By: Assigned"
Commits
Restricted Diffusion Commit
rPd9db1d61e0a6: Restore population of `ownerOrdering` to ManiphestTasks
Summary

Ref T4110. This denormalized field used to power "Group By: Assigned" got dropped in the T2217 migration at some point.

Restore its population, and fix all the data in the database.

Test Plan

Ran migration, verified database came out reasonable-looking. Reassigned a task, verified database. Ran a "Group By: assigned" query.

Diff Detail

Branch
ownerordering
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

btrahan accepted this revision.Nov 19 2013, 5:51 PM
btrahan added inline comments.
resources/sql/patches/20131118.ownerorder.php
31

I dig this %ns stuff

epriestley closed this revision.Nov 19 2013, 10:11 PM

Closed by commit rPd9db1d61e0a6 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
resources/
sql/
patches/
38 lines
src/
applications/
maniphest/
editor/
21 lines
infrastructure/
storage/
patch/
4 lines

Diff 17155

View Options

resources/sql/patches/20131118.ownerorder.php

Loading...
View Options

src/applications/maniphest/editor/ManiphestTransactionEditor.php

Loading...
View Options

src/infrastructure/storage/patch/PhabricatorBuiltinPatchList.php

Loading...
Phabricator · Built by Phacility