Page MenuHomePhabricator
Differential D11922

Fix ambiguous column names in Maniphest blocking queries
ClosedPublic
Actions

Authored by epriestley on Mar 2 2015, 2:24 PM.
Tags
None
Referenced Files
F14482767: D11922.diff
Mon, Dec 30, 11:27 AM
Unknown Object (File)
Mon, Dec 9, 4:30 AM
Unknown Object (File)
Nov 20 2024, 4:33 PM
Unknown Object (File)
Nov 20 2024, 10:31 AM
Unknown Object (File)
Nov 16 2024, 3:39 PM
Unknown Object (File)
Nov 14 2024, 4:36 AM
Unknown Object (File)
Nov 11 2024, 5:39 PM
Unknown Object (File)
Nov 8 2024, 2:05 PM
View All Files
Subscribers
epriestley

Details

Reviewers
joshuaspence
btrahan
chad
Maniphest Tasks
T7421: "Column 'status' in where clause is ambiguous" error when attempting to search for unblocked tasks
Commits
Restricted Diffusion Commit
rPbb6eb9bfcf52: Fix ambiguous column names in Maniphest blocking queries
Summary

Fixes T7421. Now that we join the task table again to ignore closed blockers, all the column names are ambiguous. Make them unambiguous.

Test Plan

Issued some searches with various different parameters.

Diff Detail

Repository
rP Phabricator
Branch
task1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4698
Build 4712: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 28724.Mar 2 2015, 2:24 PM
epriestley retitled this revision from to Fix ambiguous column names in Maniphest blocking queries.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: btrahan, joshuaspence.
epriestley added a task: T7421: "Column 'status' in where clause is ambiguous" error when attempting to search for unblocked tasks.
Herald added a subscriber: epriestley. · View Herald TranscriptMar 2 2015, 2:24 PM
chad accepted this revision.Mar 2 2015, 2:36 PM
chad added a reviewer: chad.
This revision is now accepted and ready to land.Mar 2 2015, 2:36 PM
Closed by commit rPbb6eb9bfcf52: Fix ambiguous column names in Maniphest blocking queries (authored by epriestley, committed by epriestley). · Explain WhyMar 2 2015, 2:43 PM
This revision was automatically updated to reflect the committed changes.
chad mentioned this in T7563: Dragging tasks on Maniphest appear in the wrong position (n+1).Mar 20 2015, 11:42 PM

Revision Contents
Changeset List

PathSize
src/
applications/
maniphest/
query/
52 lines

Diff 28724

View Options

src/applications/maniphest/query/ManiphestTaskQuery.php

Loading...
Phabricator · Built by Phacility