Expand permissions system (#1267)

* Add call to ChangePermission

* Move permissions system to another assembly

* Add bool to toggle set owner permission on server host

* Fix issues with merge

* Fixed merge issues and added documentation to permission system

* Fix missing import

* Fix build error

Author: joaoburatto

Assets/Scripts/SS3D/Systems/Permissions.meta renamed to Assets/Scripts/SS3D/Permissions.meta

@@ -1,16 +1,14 @@
-using System.Collections.Generic;
-using System.Linq;
 using Coimbra;
 using Coimbra.Services.Events;
-using Cysharp.Threading.Tasks;
 using SS3D.Core;
 using SS3D.Core.Behaviours;
 using SS3D.Core.Settings;
-using SS3D.Core.Utils;
-using SS3D.Systems.Permissions.Events;
+using System.Collections.Generic;
+using System.Linq;
 using UnityEngine;
+using UserPermissionsChangedEvent = SS3D.Permissions.Events.UserPermissionsChangedEvent;
 
-namespace SS3D.Systems.Permissions
+namespace SS3D.Permissions
 {
     public class DisableIfNotAdmin : NetworkActor
     {

Assets/Scripts/SS3D/Systems/Permissions/DisableIfNotAdmin.cs renamed to Assets/Scripts/SS3D/Permissions/DisableIfNotAdmin.cs

@@ -1,7 +1,7 @@
-using System.Collections.Generic;
-using Coimbra.Services.Events;
+using Coimbra.Services.Events;
+using System.Collections.Generic;
 
-namespace SS3D.Systems.Permissions.Events
+namespace SS3D.Permissions.Events
 {
     public partial struct UserPermissionsChangedEvent : IEvent
     {

Assets/Scripts/SS3D/Systems/Permissions/DisableIfNotAdmin.cs.meta renamed to Assets/Scripts/SS3D/Permissions/DisableIfNotAdmin.cs.meta

@@ -0,0 +1,20 @@
+using Coimbra;
+using UnityEngine;
+
+namespace SS3D.Permissions
+{
+    /// <summary>
+    /// This settings has general options for the permission system, so if we add a database/SQL/API support we can set stuff here.
+    /// </summary>
+    [ProjectSettings("SS3D/Server")]
+    public class PermissionSettings : ScriptableSettings
+    {
+        [SerializeField]
+        private bool _addServerOwnerPermissionToServerHost;
+
+        /// <summary>
+        /// We can define if the host will get the owner permission when he joins the game.
+        /// </summary>
+        public static bool AddServerOwnerPermissionToServerHost => GetOrFind<PermissionSettings>()._addServerOwnerPermissionToServerHost;
+    }
+}

Assets/Scripts/SS3D/Systems/Permissions/Events.meta renamed to Assets/Scripts/SS3D/Permissions/Events.meta

@@ -1,15 +1,16 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using FishNet.Object;
+using FishNet.Object;
 using FishNet.Object.Synchronizing;
 using SS3D.Core.Behaviours;
 using SS3D.Data;
 using SS3D.Logging;
-using SS3D.Systems.Permissions.Events;
+using SS3D.Permissions.Events;
+using System;
+using System.Collections.Generic;
+using System.Linq;
 using File = System.IO.File;
+using UserPermissionsChangedEvent = SS3D.Permissions.Events.UserPermissionsChangedEvent;
 
-namespace SS3D.Systems.Permissions
+namespace SS3D.Permissions
 {
     /// <summary>
     /// Handles user permission on what he can do and can't.
@@ -87,15 +88,33 @@ public bool TryGetUserRole(string ckey, out ServerRoleTypes userPermission)
         /// <summary>
         /// Updates a user permission.
         /// </summary>
-        /// <param name="ckey">The desired user to update the permission.</param>
+        /// <param name="ckey">The desired user to update the permission.</param>|
         /// <param name="role">The new user role.</param>
         [Server]
         public void ChangeUserPermission(string ckey, ServerRoleTypes role)
         {
-            throw new NotImplementedException();
-            // TODO: This
-            // Add new user permission to list
-            // Add new user permission to text file
+            ServerRoleTypes previousRole = _userPermissions.TryGetValue(ckey, out ServerRoleTypes permission) ? permission : ServerRoleTypes.None;
+
+            Log.Information(this, $"Updating user {ckey} role from {previousRole} to {role}");
+
+            _userPermissions[ckey] = role;
+
+            SaveUserPermissions();
+        }
+
+        /// <summary>
+        /// Saves the permissions text file with the updated permissions list.
+        /// </summary>
+        public void SaveUserPermissions()
+        {
+            string fileContent = string.Empty;
+
+            foreach (KeyValuePair<string,ServerRoleTypes> userPermission in _userPermissions)
+            {
+                fileContent += $"{userPermission.Key} {userPermission.Value.ToString()}\n";
+            }
+
+            File.WriteAllText(PermissionsPath, fileContent);
         }
 
         /// <summary>
@@ -164,6 +183,12 @@ private void SyncHandleUserPermissionsChanged(SyncDictionaryOperation op, string
             SyncUserPermissions();
         }
 
+        /// <summary>
+        /// Returns if the user is at least at a level of access.
+        /// </summary>
+        /// <param name="ckey">The user to check permission for</param>
+        /// <param name="permissionLevelCheck">The lowest required permission to perform the action.</param>
+        /// <returns></returns>
         public bool IsAtLeast(string ckey, ServerRoleTypes permissionLevelCheck)
         {
             TryGetUserRole(ckey, out ServerRoleTypes userPermission);