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
Unknown Object (File)
Tue, Apr 30, 7:21 PM
Unknown Object (File)
Thu, Apr 25, 1:00 AM
Unknown Object (File)
Fri, Apr 19, 11:00 AM
Unknown Object (File)
Thu, Apr 11, 10:30 AM
Unknown Object (File)
Fri, Apr 5, 9:04 PM
Unknown Object (File)
Fri, Apr 5, 9:04 PM
Unknown Object (File)
Fri, Apr 5, 8:40 PM
Unknown Object (File)
Fri, Apr 5, 8:14 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

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

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

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 17171

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