Page MenuHomePhabricator

Fix two Passphrase credential type issues
ClosedPublic

Authored by epriestley on May 26 2014, 12:35 PM.
Tags
None
Referenced Files
F19817995: D9299.id22071.diff
Thu, Mar 5, 4:54 PM
F19785111: D9299.id.diff
Tue, Feb 24, 9:27 PM
F19511839: D9299.id22070.diff
Jan 12 2026, 8:29 AM
F19464039: D9299.diff
Jan 3 2026, 12:17 AM
F19356861: D9299.id22070.diff
Dec 26 2025, 7:39 AM
F19304448: D9299.diff
Dec 24 2025, 11:15 AM
F19303862: D9299.id22071.diff
Dec 24 2025, 6:03 AM
F19303808: D9299.id22070.diff
Dec 24 2025, 5:43 AM
Subscribers

Details

Summary

Fixes T4991. Two issues:

  • These error messages pass an object to "%s", when they mean to pass a type constant.
  • The check for noncreatable credentials is incorrectly in the "edit" branch of the controller.
Test Plan
  • Edited a "SSH Key on disk" credential.
  • Tried to create a credential with a bogus type.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Fix two Passphrase credential type issues.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: joshuaspence, btrahan.
joshuaspence edited edge metadata.

This fixes the issue I was experiencing.

This revision is now accepted and ready to land.May 26 2014, 12:45 PM
epriestley updated this revision to Diff 22071.

Closed by commit rPb93c2f68521d (authored by @epriestley).