Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Maniphest Tasks
- T6757: Removing a blocker while adding a blocker results in "edited tasks, added: 1; removed: 1"
- Commits
- Restricted Diffusion Commit
rP102e431feb01: Migrate Maniphest task blockers to modern EdgeType classes
Added and removed blockers to/from tasks, saw the expected history entries.
Diff Detail
- Repository
- rP Phabricator
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
src/infrastructure/edges/constants/PhabricatorEdgeConfig.php | ||
---|---|---|
199–200 | Oh, one other minor thing -- this should be a shouldPreventCycles() implementation in the new EdgeType class. Otherwise, users can create T1 -> T2 -> T1, etc., which messes a few things up a bit. I'll just fix that in the pull. |
src/applications/maniphest/edge/ManiphestTaskDependsOnTaskEdgeType.php | ||
---|---|---|
84 | Oh, and in doing "edited blocking task(s)", these feed strings are slightly off. The reason there are different "transaction" and "feed" strings is that the "transaction" strings are shown on the object, so the edited object is implicit (e.g., alincoln edited this object.). The "feed" strings are shown elsewhere (normally, in feed) so the edited object is not implicit and must be explicit (alincoln edited T123: foobar). If the object is excluded, the affected object isn't clear in the feed story: Specifically, the viewer can't tell which task relationships were edited for unless the object is included in the string. |