Skip to content

Commit 9092a5d

Browse files
committed
codereview
1 parent f63faee commit 9092a5d

File tree

2 files changed

+8
-15
lines changed

2 files changed

+8
-15
lines changed

application/forms/Account/ChangePasswordForm.php

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -37,20 +37,16 @@ public function init()
3737
*/
3838
public function createElements(array $formData)
3939
{
40-
$passwordPolicyObject = null;
4140
$passwordPolicy = Config::app()->get(
4241
'global',
4342
'password_policy',
4443
DefaultPasswordPolicy::class
4544
);
45+
$passwordPolicyObject = new $passwordPolicy();
46+
$passwordPolicyDescription = $passwordPolicyObject->getDescription();
4647

47-
if (isset($passwordPolicy)) {
48-
$passwordPolicyObject = new $passwordPolicy();
49-
$passwordPolicyDescription = $passwordPolicyObject->getDescription();
50-
51-
if ($passwordPolicyDescription != '') {
52-
$this->addDescription($passwordPolicyDescription);
53-
}
48+
if ($passwordPolicyDescription != '') {
49+
$this->addDescription($passwordPolicyDescription);
5450
}
5551

5652
$this->addElement(

application/forms/Config/User/UserForm.php

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -20,19 +20,16 @@ class UserForm extends RepositoryForm
2020
*/
2121
protected function createInsertElements(array $formData)
2222
{
23-
$passwordPolicyObject = null;
2423
$passwordPolicy = Config::app()->get(
2524
'global',
2625
'password_policy',
2726
DefaultPasswordPolicy::class
2827
);
29-
if (isset($passwordPolicy)) {
30-
$passwordPolicyObject = new $passwordPolicy();
31-
$passwordPolicyDescription = $passwordPolicyObject->getDescription();
28+
$passwordPolicyObject = new $passwordPolicy();
29+
$passwordPolicyDescription = $passwordPolicyObject->getDescription();
3230

33-
if ($passwordPolicyDescription != '') {
34-
$this->addDescription($passwordPolicyDescription);
35-
}
31+
if ($passwordPolicyDescription != '') {
32+
$this->addDescription($passwordPolicyDescription);
3633
}
3734

3835
$this->addElement(

0 commit comments

Comments
 (0)