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

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

Tags

None

Referenced Files

	F13352786: D10201.diff
	Sun, Jun 23, 4:05 PM

	F13334623: D10201.diff
	Wed, Jun 19, 12:18 AM

	F13321343: D10201.diff
	Fri, Jun 14, 6:45 AM

	F13253241: D10201.diff
	May 25 2024, 2:31 AM

	F13240804: D10201.diff
	May 22 2024, 4:40 PM

	F13231675: D10201.diff
	May 21 2024, 12:41 AM

	F13217142: D10201.id24545.diff
	May 18 2024, 3:43 AM

	F13210694: D10201.diff
	May 17 2024, 5:06 AM

View All Files

Subscribers

epriestley

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

Lint

Lint Skipped

Unit

Tests Skipped

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 24547

View Options

src/applications/maniphest/controller/ManiphestReportController.php

Use ManiphestTaskQuery instead of ad-hoc load in Maniphest reportsClosedPublicActions