Page MenuHomePhabricator

Correct an issue where Maniphest's awful legacy "reports" UI was extra broken on merges
ClosedPublic

Authored by epriestley on Sep 14 2017, 11:29 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 17, 10:06 PM
Unknown Object (File)
Sat, Dec 14, 1:39 AM
Unknown Object (File)
Mon, Dec 9, 4:22 PM
Unknown Object (File)
Mon, Dec 9, 4:22 PM
Unknown Object (File)
Mon, Dec 9, 4:21 PM
Unknown Object (File)
Mon, Dec 9, 4:19 PM
Unknown Object (File)
Mon, Nov 25, 3:13 AM
Unknown Object (File)
Fri, Nov 22, 6:58 AM
Subscribers

Details

Summary

See PHI66. See that issue for context. This UI is bad broken legacy junk, but was especially broken when reporting merges.

These do not currently generate a "status" transaction, so they were never counted as task closures. Pretend they're normal closures.

This is still wrong, but should be much closer to the real numbers. Specifically, if you merge a closed task into another task, it will incorrectly be counted as an extra close. This could result in negative tasks, but the numbers should be much closer to reality than they are today even so.

The "Facts" application (T1562) is the real pathway forward here in the longer term.

Test Plan
  • Moved my maniphest_transactions table aside with RENAME TABLE ....
  • Created a new empty table with CREATE TABLE ... LIKE ....
  • Reloaded reports UI, saw empty chart.
  • Created, closed, and reopened tasks while reloading the chart, saw accurate reporting.
  • Merged an open task into another task, saw bad reporting.
  • Applied patch, saw the right chart again.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Sep 14 2017, 4:39 PM
This revision was automatically updated to reflect the committed changes.

For completeness's sake, this sounds like the issue that hit me in T9025 where the charts accumulated task counts greater than the number of open tasks (and indeed, the counts in a maniphest search by project now seem to pretty closely match the end state of the corresponding chart).