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
F18791231: D7602.id17171.diff
Thu, Oct 16, 7:25 AM
F18742451: D7602.id.diff
Thu, Oct 2, 7:21 PM
F18736090: D7602.diff
Wed, Oct 1, 4:49 AM
F18623413: D7602.id17155.diff
Sep 15 2025, 3:40 PM
F18623412: D7602.id17155.diff
Sep 15 2025, 3:40 PM
F18617508: D7602.diff
Sep 14 2025, 9:21 PM
F18586684: D7602.diff
Sep 11 2025, 7:24 PM
F18460872: D7602.id.diff
Sep 1 2025, 7:35 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