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

Use numeric input control for TOTP factor entry

Browse source 
Summary:
Finishes fixing T11365. rP28199bcb48 added the new numeric entry
control and used it for TOTP setup, but missed the case of entering
a factor when TOTP was already set up.

Test Plan:
Observe behaviour of TOTP setup and subsequent factor entry
in iOS browser, make sure they're consistent.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: epriestley

Maniphest Tasks: T11365

Differential Revision: https://secure.phabricator.com/D16325
This commit is contained in:
Daniel Stone 2016-07-26 04:47:02 -07:00 committed by epriestley
parent 020df6f5cb
commit 29d6e5fd4b
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/applications/auth/factor/PhabricatorTOTPAuthFactor.php
View file

@ -151,7 +151,7 @@ final class PhabricatorTOTPAuthFactor extends PhabricatorAuthFactor {
} }
$form->appendChild( $form->appendChild(
id(new AphrontFormTextControl()) id(new PHUIFormNumberControl())
->setName($this->getParameterName($config, 'totpcode')) ->setName($this->getParameterName($config, 'totpcode'))
->setLabel(pht('App Code')) ->setLabel(pht('App Code'))
->setCaption(pht('Factor Name: %s', $config->getFactorName())) ->setCaption(pht('Factor Name: %s', $config->getFactorName()))