Homec4science

Integrate auxiliary fields into Maniphest transactions

Authored by epriestley <git@epriestley.com> on Dec 24 2011, 04:03.

Description

Integrate auxiliary fields into Maniphest transactions

Summary:

  • When changing auxiliary field values, use transactions.
  • Clean up some of the load/save logic for auxiliary fields so it's a little

more performant.

NOTE: The transaction display of auxiliary fields is incredibly hacky, I'll follow up with a more nuanced approach but wanted to limit scope here.

Test Plan: Created and edited tasks with custom fields configured; created and
edited tasks without custom fields configured.

Reviewers: btrahan, jungejason, zeeg

Reviewed By: jungejason

CC: aran, jungejason

Maniphest Tasks: T418

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

Details

Committed
epriestley <git@epriestley.com>Dec 28 2011, 01:01
Pushed
aubortJan 31 2017, 17:16
Parents
rPHabd8efc35855: Further relax same origin checks
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH1a1da7d6c276: Integrate auxiliary fields into Maniphest transactions (authored by epriestley <git@epriestley.com>).Dec 28 2011, 01:01