fix: set parent pointer on file to be the same shared pointer from mFileToArchive (Kenix3#452)

* fix: set parent pointer on file to be the same shared pointer from `mFileToArchive`

* make sure to set it in non-raw loadfile too

Author: briaguya-ai

src/resource/archive/ArchiveManager.cpp

@@ -45,12 +45,7 @@ std::shared_ptr<File> ArchiveManager::LoadFile(const std::string& filePath) {
         return nullptr;
     }
 
-    const auto archive = mFileToArchive[CRC64(filePath.c_str())];
-    if (archive == nullptr) {
-        return nullptr;
-    }
-
-    return archive->LoadFile(filePath);
+    return LoadFile(CRC64(filePath.c_str()));
 }
 
 std::shared_ptr<File> ArchiveManager::LoadFile(uint64_t hash) {
@@ -59,20 +54,17 @@ std::shared_ptr<File> ArchiveManager::LoadFile(uint64_t hash) {
         return nullptr;
     }
 
-    return archive->LoadFile(hash);
+    auto file = archive->LoadFile(hash);
+    file->Parent = archive;
+    return file;
 }
 
 std::shared_ptr<File> ArchiveManager::LoadFileRaw(const std::string& filePath) {
     if (filePath == "") {
         return nullptr;
     }
 
-    const auto archive = mFileToArchive[CRC64(filePath.c_str())];
-    if (archive == nullptr) {
-        return nullptr;
-    }
-
-    return archive->LoadFileRaw(filePath);
+    return LoadFileRaw(CRC64(filePath.c_str()));
 }
 
 std::shared_ptr<File> ArchiveManager::LoadFileRaw(uint64_t hash) {
@@ -81,7 +73,9 @@ std::shared_ptr<File> ArchiveManager::LoadFileRaw(uint64_t hash) {
         return nullptr;
     }
 
-    return archive->LoadFileRaw(hash);
+    auto file = archive->LoadFileRaw(hash);
+    file->Parent = archive;
+    return file;
 }
 
 bool ArchiveManager::HasFile(const std::string& filePath) {

src/resource/archive/O2rArchive.cpp

@@ -53,7 +53,6 @@ std::shared_ptr<File> O2rArchive::LoadFileRaw(const std::string& filePath) {
         SPDLOG_TRACE("Error closing file {} in zip archive  {}.", filePath, GetPath());
     }
 
-    fileToLoad->Parent = dynamic_pointer_cast<Archive>(std::make_shared<O2rArchive>(std::move(*this)));
     fileToLoad->IsLoaded = true;
 
     return fileToLoad;

src/resource/archive/OtrArchive.cpp

@@ -50,7 +50,6 @@ std::shared_ptr<File> OtrArchive::LoadFileRaw(const std::string& filePath) {
         SPDLOG_ERROR("({}) Failed to close file {} from mpq archive {}", GetLastError(), filePath, GetPath());
     }
 
-    fileToLoad->Parent = dynamic_pointer_cast<Archive>(std::make_shared<OtrArchive>(std::move(*this)));
     fileToLoad->IsLoaded = true;
 
     return fileToLoad;