Differential D10201

Use ManiphestTaskQuery instead of ad-hoc load in Maniphest reports
ClosedPublic
Actions

Authored by epriestley on Aug 8 2014, 11:06 PM.

Details

Reviewers

btrahan

Maniphest Tasks

T5829: PhabricatorDataNotAttachedException thrown at /maniphest/report/project/

Commits

Restricted Diffusion Commit
rP664507e45096: Use ManiphestTaskQuery instead of ad-hoc load in Maniphest reports

Summary

Fixes T5829. This stuff is old and busted, but keep it working for now.

Test Plan

No more fatal when there are recently closed tasks.

Diff Detail

Repository

rP Phabricator

Branch

boardreport

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 2122
Build 2126: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 24545.Aug 8 2014, 11:06 PM

epriestley retitled this revision from to Use ManiphestTaskQuery instead of ad-hoc load in Maniphest reports.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

epriestley added a task: T5829: PhabricatorDataNotAttachedException thrown at /maniphest/report/project/.

Herald added a subscriber: epriestley. · View Herald TranscriptAug 8 2014, 11:06 PM

Harbormaster completed remote builds in B2122: Diff 24545.Aug 8 2014, 11:07 PM

btrahan accepted this revision.Aug 8 2014, 11:18 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Aug 8 2014, 11:18 PM

Closed by commit rP664507e45096 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

maniphest/

controller/

ManiphestReportController.php

15 lines

Diff 24545

View Options

src/applications/maniphest/controller/ManiphestReportController.php

Use ManiphestTaskQuery instead of ad-hoc load in Maniphest reportsClosedPublicActions