Differential D17851

Clean up some Passphrase transaction bugs
ClosedPublic
Actions

Authored by chad on May 8 2017, 10:06 PM.

Details

Reviewers

epriestley

Maniphest Tasks

T12685: Modular Transactions Migrations Errata

Commits

rPc505876d6162: Clean up some Passphrase transaction bugs

Summary

Ref T12685. Makes the description field full remarkup and fixes setting a credential secret after destruction.

Test Plan

Change description a lot, set and destroy credentials.

Diff Detail

Repository

rP Phabricator

Branch

passphrase-xaction-3 (branched from master)

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 16858
Build 22510: Run Core Tests
Build 22509: arc lint + arc unit

Event Timeline

chad created this revision.May 8 2017, 10:06 PM

Herald added a subscriber: Korvin. · View Herald TranscriptMay 8 2017, 10:06 PM

Harbormaster completed remote builds in B16858: Diff 42932.May 8 2017, 10:07 PM

epriestley accepted this revision.May 8 2017, 10:50 PM

This revision is now accepted and ready to land.May 8 2017, 10:50 PM

Closed by commit rPc505876d6162: Clean up some Passphrase transaction bugs (authored by chad, committed by chad). · Explain WhyMay 8 2017, 10:55 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

passphrase/

controller/

PassphraseCredentialEditController.php

4 lines

xaction/

PassphraseCredentialSecretIDTransaction.php

14 lines

Diff 42932

View Options

src/applications/passphrase/controller/PassphraseCredentialEditController.php