- changed log messages for edit user record

html/admin/users/editauth.php

@@ -82,7 +82,7 @@ if (isset($_POST["editauth"]) and !$old_auth_info['deleted']) {
             }
         if ($new['nagios'] ==0) { $new['nagios_status']='UP'; }
         $changes = get_diff_rec($db_link,"User_auth","id='$id'", $new, 0);
-        if (!empty($changes)) { LOG_WARNING($db_link,"Изменен адрес доступа! Список изменений: $changes",$id); }
+        if (!empty($changes)) { LOG_WARNING($db_link,"Изменена запись для адреса $ip! Список изменений: $changes",$id); }
         if (is_auth_bind_changed($db_link,$id,$ip,$mac)) {
             $new_id = copy_auth($db_link,$id,$new);
             header("Location: /admin/users/editauth.php?id=".$new_id,TRUE, 302);

html/admin/users/edituser.php

@@ -11,6 +11,9 @@ require_once ($_SERVER['DOCUMENT_ROOT']."/inc/sortfilter.php");
 
 $msg_error = "";
 
+$sSQL = "SELECT * FROM User_list WHERE id=$id";
+$user_info = get_record_sql($db_link, $sSQL);
+
 if (isset($_POST["edituser"])) {
     unset($new);
     $new["ou_id"] = $_POST["f_ou"] * 1;
@@ -32,7 +35,7 @@ if (isset($_POST["edituser"])) {
         $new["month_quota"] = trim($_POST["f_permonth"]) * 1;
     }
     $changes = get_diff_rec($db_link,"User_list","id='$id'", $new, 0);
-    if (!empty($changes)) { LOG_WARNING($db_link,"Изменён пользователь id: $id. \r\nПрименено: $changes"); }
+    if (!empty($changes)) { LOG_WARNING($db_link,"Изменён пользователь id: $id login: ".$new["login"].". \r\nПрименено: $changes"); }
     update_record($db_link, "User_list", "id='$id'", $new);
     run_sql($db_link, "UPDATE User_auth SET ou_id=".$new["ou_id"]." WHERE user_id=".$id);
     run_sql($db_link, "UPDATE devices SET device_name='".$new["login"]."' WHERE user_id=".$id);
@@ -48,7 +51,7 @@ if (isset($_POST["addMacRule"])) {
         $new['type']=2;
         $new['rule']=$first_auth['mac'];
 	insert_record($db_link,"auth_rules",$new);
-	LOG_INFO($db_link,"Создано правило атоназначения юзеру id: ".$id." для мака ".$first_auth['mac']);
+	LOG_INFO($db_link,"Создано правило атоназначения юзеру id: ".$id." login: ".$user_info["login"]." для мака ".$first_auth['mac']);
 	}
     header("Location: " . $_SERVER["REQUEST_URI"]);
     exit;
@@ -56,7 +59,7 @@ if (isset($_POST["addMacRule"])) {
 
 if (isset($_POST["delMacRule"])) {
     run_sql($db_link,"DELETE FROM auth_rules WHERE user_id=".$id." AND type=2");
-    LOG_INFO($db_link,"Удалены все правила атоназначения юзеру id: $id по маку");
+    LOG_INFO($db_link,"Удалены все правила атоназначения юзеру id: $id login: ".$user_info["login"]." по маку");
     header("Location: " . $_SERVER["REQUEST_URI"]);
     exit;
 }
@@ -69,7 +72,7 @@ if (isset($_POST["addIPRule"])) {
         $new['type']=1;
         $new['rule']=$first_auth['ip'];
 	insert_record($db_link,"auth_rules",$new);
-	LOG_INFO($db_link,"Создано правило атоназначения юзеру id: ".$id." для IP ".$first_auth['IP']);
+	LOG_INFO($db_link,"Создано правило атоназначения юзеру id: ".$id." login: ".$user_info["login"]." для IP ".$first_auth['IP']);
 	}
     header("Location: " . $_SERVER["REQUEST_URI"]);
     exit;
@@ -77,7 +80,7 @@ if (isset($_POST["addIPRule"])) {
 
 if (isset($_POST["delIPRule"])) {
     run_sql($db_link,"DELETE FROM auth_rules WHERE user_id=".$id." AND type=1");
-    LOG_INFO($db_link,"Удалены все правила атоназначения юзеру id: $id по ip");
+    LOG_INFO($db_link,"Удалены все правила атоназначения юзеру id: $id login: ".$user_info["login"]." по ip");
     header("Location: " . $_SERVER["REQUEST_URI"]);
     exit;
 }
@@ -86,8 +89,6 @@ if (isset($_POST["showDevice"])) {
     $device = get_record_sql($db_link,"SELECT * FROM devices WHERE user_id=".$id);
     $auth = get_record_sql($db_link,"SELECT * FROM User_auth WHERE user_id=".$id);
     if (empty($device) and !empty($auth)) {
-	$sSQL = "SELECT * FROM User_list WHERE id=$id";
-	$user_info = get_record_sql($db_link, $sSQL);
         global $snmp_default_version;
         global $snmp_default_community;
 	$new['user_id']=$id;
@@ -146,7 +147,7 @@ if (isset($_POST["addauth"])) {
                 $new['dhcp']=$f_dhcp;
                 update_record($db_link,"User_auth","id=".$fid,$new);
                 apply_auth_rule($db_link,$fid,$id);
-                LOG_WARNING($db_link,"Создан новый адрес доступа: ip => $fip, mac => $fmac",$fid);
+                LOG_WARNING($db_link,"Создан новый адрес доступа для login: ".$user_info["login"].": ip => $fip, mac => $fmac",$fid);
                 header("Location: /admin/users/editauth.php?id=".$fid);
                 exit;
                 }
@@ -169,7 +170,7 @@ if (isset($_POST["removeauth"])) {
             run_sql($db_link, 'DELETE FROM User_auth_alias WHERE auth_id='.$val);
             $auth["deleted"] = 1;
             $changes = get_diff_rec($db_link,"User_auth","id='$val'", '', 0);
-            if (!empty($changes)) { LOG_WARNING($db_link,"Удалён адрес доступа: \r\n $changes",$val); }
+            if (!empty($changes)) { LOG_WARNING($db_link,"Удалён адрес доступа для login: ".$user_info["login"].": \r\n $changes",$val); }
             update_record($db_link, "User_auth", "id=" . $val, $auth);
         }
     }
@@ -179,9 +180,6 @@ if (isset($_POST["removeauth"])) {
 
 unset($_POST);
 
-$sSQL = "SELECT * FROM User_list WHERE id=$id";
-$user_info = get_record_sql($db_link, $sSQL);
-
 require_once ($_SERVER["DOCUMENT_ROOT"]."/inc/header.php");
 ?>
 <div id="cont">

html/inc/common.php

@@ -1629,9 +1629,9 @@ if (isset($debug) and $debug) { write_log($db,$msg,$L_DEBUG,$auth_id); }
 }
 
 function get_first_line($msg) {
-if (empty $msg) { return; }
+if (empty($msg)) { return; }
 preg_match('/(.*)(\n|\<br\\>)/', $msg, $matches);
-if (!empty $matches[1]) { return $matches[1]; }
+if (!empty($matches[1])) { return $matches[1]; }
 return;
 }