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
F18586684: D7602.diff
Thu, Sep 11, 7:24 PM
F18460872: D7602.id.diff
Mon, Sep 1, 7:35 PM
F18431177: D7602.diff
Sun, Aug 31, 4:54 AM
F18106947: D7602.id.diff
Aug 10 2025, 11:33 PM
F18106737: D7602.diff
Aug 10 2025, 11:15 PM
F18103805: D7602.id17155.diff
Aug 10 2025, 7:55 AM
F18088841: D7602.diff
Aug 6 2025, 9:50 AM
F17933476: D7602.id.diff
Jul 30 2025, 9:20 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