From 2394ef23c7c03225812da7ceca089ec3617472f0 Mon Sep 17 00:00:00 2001
From: Thibault Dockx <thibault.dockx@fusiondirectory.org>
Date: Thu, 5 Sep 2024 20:48:30 +0100
Subject: [PATCH] :sparkles: (CORE) - add logging when user is locked
Fixing code style.
---
.../management/users/class_userManagement.inc | 41 ++++++++++---------
1 file changed, 21 insertions(+), 20 deletions(-)
diff --git a/plugins/management/users/class_userManagement.inc b/plugins/management/users/class_userManagement.inc
index 29d440afb..454e7a03b 100644
--- a/plugins/management/users/class_userManagement.inc
+++ b/plugins/management/users/class_userManagement.inc
@@ -25,14 +25,15 @@ class userManagement extends management
public static $skipTemplates = FALSE;
- public static $columns = [
- ['ObjectTypeColumn', []],
- ['LinkColumn', ['attributes' => 'sn', 'label' => 'Last name']],
- ['LinkColumn', ['attributes' => 'givenName', 'label' => 'First name']],
- ['Column', ['attributes' => 'uid', 'label' => 'Login']],
- ['PropertiesColumn', ['label' => 'Properties']],
- ['ActionsColumn', ['label' => 'Actions']],
- ];
+ public static $columns
+ = [
+ ['ObjectTypeColumn', []],
+ ['LinkColumn', ['attributes' => 'sn', 'label' => 'Last name']],
+ ['LinkColumn', ['attributes' => 'givenName', 'label' => 'First name']],
+ ['Column', ['attributes' => 'uid', 'label' => 'Login']],
+ ['PropertiesColumn', ['label' => 'Properties']],
+ ['ActionsColumn', ['label' => 'Actions']],
+ ];
static function plInfo (): array
{
@@ -117,10 +118,10 @@ class userManagement extends management
}
$error = new FusionDirectoryError(
htmlescape(sprintf(
- _('Password method "%s" does not support locking. Account "%s" has not been locked!'),
- $hn,
- $dn
- ))
+ _('Password method "%s" does not support locking. Account "%s" has not been locked!'),
+ $hn,
+ $dn
+ ))
);
$error->display();
return;
@@ -130,14 +131,14 @@ class userManagement extends management
if (($action == 'lock') && !$method->is_locked($dn)) {
$success = $method->lock_account($dn);
// Requiring logging mechanism for audit.
- if($success) {
- logging::log('security', 'account', $dn, [], 'DN : '.$dn.' is locked.' );
+ if ($success) {
+ logging::log('security', 'account', $dn, [], 'DN : ' . $dn . ' is locked.');
}
} elseif (($action == 'unlock') && $method->is_locked($dn)) {
$success = $method->unlock_account($dn);
// Requiring logging mechanism for audit.
- if($success) {
- logging::log('security', 'account', $dn, [], 'DN : '.$dn.' is unlocked.' );
+ if ($success) {
+ logging::log('security', 'account', $dn, [], 'DN : ' . $dn . ' is unlocked.');
}
}
@@ -149,10 +150,10 @@ class userManagement extends management
}
$error = new FusionDirectoryError(
htmlescape(sprintf(
- _('Locking failed using password method "%s". Account "%s" has not been locked!'),
- $hn,
- $dn
- ))
+ _('Locking failed using password method "%s". Account "%s" has not been locked!'),
+ $hn,
+ $dn
+ ))
);
$error->display();
}
--
GitLab