revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-30 01:10:58 +01:00
Code Issues Releases Wiki Activity

Don't disable the "Show Public Key" action for locked credentials

Browse source 
Summary: Fixes T5104. It's still OK to reveal public keys of locked credentials -- the controller has the right logic, this UI just isn't in sync.

Test Plan: Viewed passphrase; saw enabled menu item on locked credential.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T5104

Differential Revision: https://secure.phabricator.com/D9186
This commit is contained in:
epriestley 2014-05-19 12:40:42 -07:00
parent edd8242008
commit af0edf883d
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/applications/passphrase/controller/PassphraseCredentialViewController.php
View file

@ -134,8 +134,7 @@ final class PassphraseCredentialViewController extends PassphraseController {
->setName(pht('Show Public Key')) ->setName(pht('Show Public Key'))
->setIcon('fa-download') ->setIcon('fa-download')
->setHref($this->getApplicationURI("public/{$id}/")) ->setHref($this->getApplicationURI("public/{$id}/"))
->setWorkflow(true) ->setWorkflow(true));
->setDisabled($is_locked));
} }
$actions->addAction( $actions->addAction(