Improve ID modification logs (#12918)

* show modifications in access change logs

* skip logging and updates to name and job when no change was made

* add method to SharedAccessSystem.cs to get access tags

* add ID microwave logs

Content.Server/Access/Components/IdCardConsoleComponent.cs

@@ -95,12 +95,20 @@ namespace Content.Server.Access.Components
             }
 
             var accessSystem = _entities.EntitySysManager.GetEntitySystem<AccessSystem>();
+            var oldTags = accessSystem.TryGetTags(targetIdEntity) ?? new List<string>();
+            oldTags = oldTags.ToList();
+
+            if (oldTags.SequenceEqual(newAccessList))
+                return;
+
+            var addedTags = newAccessList.Except(oldTags).Select(tag => "+" + tag).ToList();
+            var removedTags = oldTags.Except(newAccessList).Select(tag => "-" + tag).ToList();
             accessSystem.TrySetTags(targetIdEntity, newAccessList);
 
             /*TODO: ECS IdCardConsoleComponent and then log on card ejection, together with the save.
             This current implementation is pretty shit as it logs 27 entries (27 lines) if someone decides to give themselves AA*/
             _adminLogger.Add(LogType.Action, LogImpact.Medium,
-                $"{_entities.ToPrettyString(player):player} has modified {_entities.ToPrettyString(targetIdEntity):entity} with the following accesses: [{string.Join(", ", newAccessList)}]");
+                $"{_entities.ToPrettyString(player):player} has modified {_entities.ToPrettyString(targetIdEntity):entity} with the following accesses: [{String.Join(", ", addedTags.Union(removedTags))}] [{string.Join(", ", newAccessList)}]");
 
             UpdateStationRecord(targetIdEntity, newFullName, newJobTitle, newJobProto);
         }

Content.Server/Access/Systems/IdCardSystem.cs

@@ -48,6 +48,8 @@ namespace Content.Server.Access.Systems
                         EntityManager.SpawnEntity("FoodBadRecipe",
                             transformComponent.Coordinates);
                     }
+                    _adminLogger.Add(LogType.Action, LogImpact.Medium,
+                        $"{ToPrettyString(args.Microwave)} burnt {ToPrettyString(uid):entity}");
                     EntityManager.QueueDeleteEntity(uid);
                     return;
                 }
@@ -56,6 +58,8 @@ namespace Content.Server.Access.Systems
                 {
                     _popupSystem.PopupEntity(Loc.GetString("id-card-component-microwave-bricked", ("id", uid)), uid);
                     access.Tags.Clear();
+                    _adminLogger.Add(LogType.Action, LogImpact.Medium,
+                        $"{ToPrettyString(args.Microwave)} cleared access on {ToPrettyString(uid):entity}");
                 }
                 else
                 {
@@ -65,6 +69,9 @@ namespace Content.Server.Access.Systems
                 // Give them a wonderful new access to compensate for everything
                 var random = _random.Pick(_prototypeManager.EnumeratePrototypes<AccessLevelPrototype>().ToArray());
                 access.Tags.Add(random.ID);
+
+                _adminLogger.Add(LogType.Action, LogImpact.Medium,
+                        $"{ToPrettyString(args.Microwave)} added {random.ID} access to {ToPrettyString(uid):entity}");
             }
         }
 
@@ -92,6 +99,8 @@ namespace Content.Server.Access.Systems
                 jobTitle = null;
             }
 
+            if (id.JobTitle == jobTitle)
+                return true;
             id.JobTitle = jobTitle;
             Dirty(id);
             UpdateEntityName(uid, id);
@@ -127,6 +136,8 @@ namespace Content.Server.Access.Systems
                 fullName = null;
             }
 
+            if (id.FullName == fullName)
+                return true;
             id.FullName = fullName;
             Dirty(id);
             UpdateEntityName(uid, id);

Content.Shared/Access/Systems/SharedAccessSystem.cs

@@ -67,6 +67,15 @@ namespace Content.Shared.Access.Systems
             return true;
         }
 
+        /// <summary>
+        ///     Gets the set of access tags.
+        /// </summary>
+        /// <param name="newTags">The new access tags</param>
+        public IEnumerable<string>? TryGetTags(EntityUid uid, AccessComponent? access = null)
+        {
+            return !Resolve(uid, ref access) ? null : access.Tags;
+        }
+
         public bool TryAddGroups(EntityUid uid, IEnumerable<string> newGroups, AccessComponent? access = null)
         {
             if (!Resolve(uid, ref access))