Changeset 474 for TFP-WebServer/WebServer/src/Permissions/AdminWebModules.cs

Timestamp:

Aug 24, 2023, 12:48:17 PM (15 months ago)

Author:

alloc

Message:

Added locks on AdminTools instead of local to the individual permission systems

File:

• TFP-WebServer/WebServer/src/Permissions/AdminWebModules.cs (modified) (6 diffs)

TFP-WebServer/WebServer/src/Permissions/AdminWebModules.cs

@@ -41 +41 @@
 
                 public void AddModule (WebModule _module) {
-                        lock (this) {
+                        lock (Parent) {
                                 allModulesList.Clear ();
                                 
@@ -50 +50 @@
 
                 public bool RemoveModule (string _module) {
-                        lock (this) {
+                        lock (Parent) {
                                 allModulesList.Clear ();
                                 
@@ -63 +63 @@
 
                 public List<WebModule> GetModules () {
-                        lock (this) {
+                        lock (Parent) {
                                 if (allModulesList.Count != 0) {
                                         return allModulesList;
@@ -259 +259 @@
                         _module.IsDefault = true;
                         
-                        lock (this) {
+                        lock (Parent) {
                                 allModulesList.Clear ();
                                 knownModules [_module.Name] = _module;
@@ -270 +270 @@
                         }
 
-                        lock (this) {
+                        lock (Parent) {
                                 return knownModules.ContainsKey (_module);
                         }
@@ -276 +276 @@
 
                 public bool ModuleAllowedWithLevel (string _module, int _level) {
-                        return GetModule (_module).LevelGlobal >= _level;
+                        lock (Parent) {
+                                return GetModule (_module).LevelGlobal >= _level;
+                        }
                 }
 
                 public WebModule GetModule (string _module) {
-                        if (modules.TryGetValue (_module, out WebModule result)) {
-                                return result;
-                        }
-
-                        return knownModules.TryGetValue (_module, out result) ? result : defaultModulePermission;
+                        lock (Parent) {
+                                if (modules.TryGetValue (_module, out WebModule result)) {
+                                        return result;
+                                }
+
+                                return knownModules.TryGetValue (_module, out result) ? result : defaultModulePermission;
+                        }
                 }