Page MenuHomePhabricator

Fix some of the most obvious bugs in fact generation from Maniphest tasks
ClosedPublic

Authored by epriestley on Feb 19 2018, 7:38 PM.
Tags
None
Referenced Files
F15408849: D19122.id45817.diff
Wed, Mar 19, 1:53 AM
F15406857: D19122.id45821.diff
Tue, Mar 18, 2:18 PM
F15336964: D19122.id45821.diff
Sun, Mar 9, 3:37 AM
F15336963: D19122.id45817.diff
Sun, Mar 9, 3:37 AM
F15336962: D19122.id.diff
Sun, Mar 9, 3:37 AM
F15336961: D19122.diff
Sun, Mar 9, 3:36 AM
Unknown Object (File)
Sat, Feb 22, 6:59 PM
Unknown Object (File)
Feb 9 2025, 5:30 AM
Subscribers

Details

Summary

Depends on D19121. Ref T13083. Group transactions and show groups in the debugging view.

Fix some of the most obvious issues with fact generation:

  • No more 0-point facts.
  • Engine can now generate at least one of every type of fact.
Test Plan

Generated facts, viewed them in the debugging view, fact generation largely appeared to align with reality. No more "no facts in storage" facts.

Diff Detail

Repository
rP Phabricator
Branch
facts4
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/fact/engine/PhabricatorManiphestTaskFactEngine.php:108XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 19625
Build 26570: Run Core Tests
Build 26569: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Feb 19 2018, 8:07 PM
This revision was automatically updated to reflect the committed changes.