Revert "Fix ACL request detection when using refresh (multitheftauto#1824) (multitheftauto#4409)"

This reverts commit d0e210a.

Author: PerikiyoXD

Server/mods/deathmatch/logic/CResource.AclRequest.cpp

@@ -123,6 +123,9 @@ bool CResource::HasAutoPermissions(CXMLNode* pNodeAclRequest)
 ///////////////////////////////////////////////////////////////
 void CResource::RefreshAutoPermissions(CXMLNode* pNodeAclRequest)
 {
+    // Check if permissions already active
+    if (HasAutoPermissions(pNodeAclRequest))
+        return;
 
     // Ensure group and acl exist
     CAccessControlListGroup* pAutoGroup = g_pGame->GetACLManager()->AddGroup(GetAutoGroupName());
@@ -369,51 +372,3 @@ bool CResource::FindAclRequest(SAclRequest& result)
 
     return pAclRight->GetAttributeValue("pending") != "";
 }
-
-std::string CResource::CalculateACLRequestFingerprint()
-{
-    std::string strPath;
-    if (!GetFilePath("meta.xml", strPath))
-        return {};
-
-    std::unique_ptr<CXMLFile> metaFile(g_pServerInterface->GetXML()->CreateXML(strPath.c_str()));
-    if (!metaFile || !metaFile->Parse())
-    {
-        return {};
-    }
-
-    CXMLNode* root = metaFile->GetRootNode();
-    if (!root)
-    {
-        return {};
-    }
-
-    std::ostringstream fingerprint;
-    CXMLNode* nodeAclRequest = root->FindSubNode("aclrequest", 0);
-    
-    if (nodeAclRequest)
-    {
-        for (std::uint8_t uiIndex = 0; true; uiIndex++)
-        {
-            CXMLNode* nodeRight = nodeAclRequest->FindSubNode("right", uiIndex);
-            if (!nodeRight)
-                break;
-
-            std::string strName = nodeRight->GetAttributeValue("name");
-            std::string strAccess = nodeRight->GetAttributeValue("access");
-
-            if (uiIndex > 0)
-                fingerprint << ";";
-                
-            fingerprint << strName << ":" << strAccess;
-        }
-    }
-    
-    return fingerprint.str();
-}
-
-bool CResource::HasACLRequestsChanged()
-{
-    std::string strCurrentFingerprint = CalculateACLRequestFingerprint();
-    return strCurrentFingerprint != m_strACLRequestFingerprint;
-}

Server/mods/deathmatch/logic/CResource.cpp

@@ -196,8 +196,6 @@ bool CResource::Load()
             else
                 RemoveAutoPermissions();
 
-            m_strACLRequestFingerprint = CalculateACLRequestFingerprint();
-
             // Find any map sync option
             m_bSyncMapElementData = true;
             m_bSyncMapElementDataDefined = false;
@@ -353,7 +351,6 @@ bool CResource::Unload()
     m_strResourceZip = "";
     m_strResourceCachePath = "";
     m_strResourceDirectoryPath = "";
-    m_strACLRequestFingerprint.clear();
     m_eState = EResourceState::None;
 
     return true;
@@ -405,8 +402,6 @@ CResource::~CResource()
 
 void CResource::TidyUp()
 {
-    RemoveAutoPermissions();
-
     // Close the zipfile stuff
     if (m_zipfile)
         unzClose(m_zipfile);
@@ -683,11 +678,6 @@ bool CResource::HasResourceChanged()
             return true;
     }
 
-    if (HasACLRequestsChanged())
-    {
-        return true;
-    }
-
     return false;
 }
 
@@ -1205,9 +1195,6 @@ bool CResource::Stop(bool bManualStop)
     // Clear the list of players where this resource is running
     std::exchange(m_isRunningForPlayer, {});
 
-    // Remove ACL permissions when stopping
-    RemoveAutoPermissions();
-
     OnResourceStateChange("loaded");
     m_eState = EResourceState::Loaded;
     return true;

Server/mods/deathmatch/logic/CResource.h

@@ -347,10 +347,7 @@ class CResource : public EHS
     void RefreshAutoPermissions(CXMLNode* pNodeAclRequest);
 
     void CommitAclRequest(const SAclRequest& request);
-    bool FindAclRequest(SAclRequest& result);
-
-    std::string CalculateACLRequestFingerprint();
-    bool        HasACLRequestsChanged();
+    bool FindAclRequest(SAclRequest& request);
 
 private:
     bool CheckState();            // if the resource has no Dependents, stop it, if it has, start it. returns true if the resource is started.
@@ -453,7 +450,6 @@ class CResource : public EHS
     SString     m_strMinServerReason;
 
     CChecksum m_metaChecksum;            // Checksum of meta.xml last time this was loaded, generated in GenerateChecksums()
-    std::string m_strACLRequestFingerprint;
 
     uint                              m_uiFunctionRightCacheRevision = 0;
     CFastHashMap<lua_CFunction, bool> m_FunctionRightCacheMap;

Server/mods/deathmatch/logic/CResourceManager.cpp

@@ -175,14 +175,6 @@ bool CResourceManager::Refresh(bool bRefreshAll, const SString strJustThisResour
                 // Add the resource
                 Load(!info.bIsDir, info.strAbsPath, info.strName);
             }
-            else if (bRefreshAll && pResource && pResource->HasResourceChanged())
-            {
-                if (g_pServerInterface->IsRequestingExit())
-                    return false;
-
-                // Resource exists but has changed, reload it
-                Load(!info.bIsDir, info.strAbsPath, info.strName);
-            }
         }
     }