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
F15534844: D7602.id17171.diff
Thu, Apr 24, 2:26 AM
F15531149: D7602.id17171.diff
Wed, Apr 23, 9:57 AM
F15529388: D7602.id.diff
Tue, Apr 22, 7:21 PM
F15524001: D7602.diff
Mon, Apr 21, 5:19 AM
F15428761: D7602.id.diff
Mar 23 2025, 10:09 PM
F15422897: D7602.id17171.diff
Mar 22 2025, 11:09 AM
F15421118: D7602.id.diff
Mar 21 2025, 9:09 PM
F15415457: D7602.diff
Mar 20 2025, 6:06 AM
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