Page MenuHomePhabricator

Fix two Passphrase credential type issues
ClosedPublic

Authored by epriestley on May 26 2014, 12:35 PM.
Tags
None
Referenced Files
F14373642: D9299.diff
Fri, Dec 20, 8:51 PM
Unknown Object (File)
Thu, Dec 12, 10:55 PM
Unknown Object (File)
Sun, Dec 8, 3:05 PM
Unknown Object (File)
Oct 29 2024, 11:02 PM
Unknown Object (File)
Oct 24 2024, 9:43 AM
Unknown Object (File)
Oct 20 2024, 4:08 AM
Unknown Object (File)
Oct 15 2024, 8:01 PM
Unknown Object (File)
Oct 15 2024, 10:32 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).