fix(skymp5-server): fix form list reversed order (#2158)

skyrim-multiplayer · Sep 12, 2024 · 291ea9f · 291ea9f
1 parent f53833f
commit 291ea9f
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/libespm/src/FLST.cpp b/libespm/src/FLST.cpp
@@ -17,7 +17,6 @@ FLST::Data FLST::GetData(
       }
     },
     compressedFieldsCache);
-  std::reverse(result.formIds.begin(), result.formIds.end());
   return result;
 }
 

diff --git a/unit/EspmTest.cpp b/unit/EspmTest.cpp
@@ -246,7 +246,7 @@ TEST_CASE("Loads FormList", "[espm]")
   REQUIRE(form.rec->GetType() == "FLST");
 
   auto data = reinterpret_cast<const espm::FLST*>(form.rec)->GetData(cache);
-  REQUIRE(data.formIds == std::vector<uint32_t>({ 0x3eab9, 0x4e4bb }));
+  REQUIRE(data.formIds == std::vector<uint32_t>({ 0x4e4bb, 0x3eab9 }));
 }
 
 TEST_CASE("Loads refr with primitive", "[espm]")

diff --git a/unit/PapyrusFormListTest.cpp b/unit/PapyrusFormListTest.cpp
@@ -17,10 +17,10 @@ TEST_CASE("GetSize/GetAt", "[Papyrus][FormList][espm]")
   REQUIRE(PapyrusFormList().GetSize(formlist, {}) == VarValue(2));
 
   VarValue element0 = PapyrusFormList().GetAt(formlist, { VarValue(0) });
-  REQUIRE(GetRecordPtr(element0).rec->GetId() == 0x3eab9);
+  REQUIRE(GetRecordPtr(element0).rec->GetId() == 0x4e4bb);
 
   VarValue element1 = PapyrusFormList().GetAt(formlist, { VarValue(1) });
-  REQUIRE(GetRecordPtr(element1).rec->GetId() == 0x4e4bb);
+  REQUIRE(GetRecordPtr(element1).rec->GetId() == 0x3eab9);
 
   REQUIRE(PapyrusFormList().GetAt(formlist, { VarValue(-1) }) ==
           VarValue::None());
-Original file line number
+Diff line change
@@ Expand Up / @@ -17,7 +17,6 @@ FLST::Data FLST::GetData( @@
           }
         },
         compressedFieldsCache);
-      std::reverse(result.formIds.begin(), result.formIds.end());
       return result;
     }
@@ Expand Down @@