Skip to content

Commit f843f5c

Browse files
committed
fix: Use return value rather than pass-by-reference
1 parent 32ebc9e commit f843f5c

File tree

6 files changed

+6
-6
lines changed

6 files changed

+6
-6
lines changed

admin/alarms.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@
4343
}
4444

4545
if ($form->validate()) {
46-
$form->getInfo($vars, $info);
46+
$info = $form->getInfo($vars, $info);
4747
if (empty($info['alarm'])) {
4848
$info['alarm'] = strval(new Horde_Support_Uuid());
4949
}

admin/config/index.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -432,7 +432,7 @@ function _uploadFTP($params)
432432
$ftpform->addVariable(_("Password"), 'password', 'password', false);
433433

434434
if ($ftpform->validate($vars)) {
435-
$ftpform->getInfo($vars, $info);
435+
$info = $ftpform->getInfo($vars, $info);
436436
$upload = _uploadFTP($info);
437437
if ($upload) {
438438
$notification->push(_("Uploaded all application configuration files to the server."), 'horde.success');

admin/user.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@
5757
case 'add':
5858
$addForm->validate($vars);
5959
if ($addForm->isValid() && $vars->get('formname') == 'adduser') {
60-
$addForm->getInfo($vars, $info);
60+
$info = $addForm->getInfo($vars, $info);
6161

6262
if (empty($info['user_name']) && isset($info['extra']['user_name'])) {
6363
$info['user_name'] = $info['extra']['user_name'];

services/changepassword.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
$form->addVariable(_("Retype new password"), 'password_2', 'password', true);
3333

3434
if ($form->validate($vars)) {
35-
$form->getInfo($vars, $info);
35+
$info = $form->getInfo($vars, $info);
3636

3737
if ($registry->getAuthCredential('password') != $info['old_password']) {
3838
$notification->push(_("Old password is not correct."), 'horde.error');

services/resetpassword.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@
6161

6262
/* Validate the form. */
6363
if ($can_validate && $form->validate($vars)) {
64-
$form->getInfo($vars, $info);
64+
$info = $form->getInfo($vars, $info);
6565

6666
/* Fetch values from prefs for selected user. */
6767
$answer = $prefs->getValue('security_answer');

signup.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
$vars = $injector->getInstance('Horde_Variables');
3636
$formsignup = new Horde_Core_Auth_Signup_Form($vars);
3737
if ($formsignup->validate()) {
38-
$formsignup->getInfo($vars, $info);
38+
$info = $formsignup->getInfo($vars, $info);
3939
$error = $success_message = null;
4040

4141
if ($info instanceof PEAR_Error) {

0 commit comments

Comments
 (0)