Homec4science

Fix maniphest "create" transactions

Authored by Bob Trahan <btrahan@phacility.com> on Apr 1 2014, 23:26.

Description

Fix maniphest "create" transactions

Summary: the quotes are 'cuz "create" is inferred. Previously, we inferred on "status", but since we set that on "initializeNewTask" instead infer off "title" (aka "name") like most other apps do. Only hairy tweak was to elevate TYPE_TITLE to the most important of all maniphest transactions, which doesn't actually seem too unreasonable if not correct even? Fixes T4686.

Test Plan: made a new task, used bin/mail, got the right headers (mail vary prefix == created)

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: epriestley, Korvin

Maniphest Tasks: T4686

Differential Revision: https://secure.phabricator.com/D8639

Details

Committed
Bob Trahan <btrahan@phacility.com>Apr 1 2014, 23:26
Pushed
aubortJan 31 2017, 17:16
Parents
rPHff98565db65e: Fix an issue where email for "accepted" revisions would fatal
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <btrahan@phacility.com> committed rPH69b8f56cda1c: Fix maniphest "create" transactions (authored by Bob Trahan <btrahan@phacility.com>).Apr 1 2014, 23:26