From b1b06cc9f2dc97f92b78d993854722dfadeb8231 Mon Sep 17 00:00:00 2001
From: in1tiate <32779090+in1tiate@users.noreply.github.com>
Date: Tue, 4 Feb 2025 10:29:32 -0600
Subject: [PATCH] fix erroneous position update and remove restriction (#396)

- Don't set the client's position for no reason
- Don't stop positions other than wit from recording testimony
---
 src/packet/packet_ms.cpp | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/src/packet/packet_ms.cpp b/src/packet/packet_ms.cpp
index 4586e7d..8701e33 100644
--- a/src/packet/packet_ms.cpp
+++ b/src/packet/packet_ms.cpp
@@ -394,9 +394,7 @@ AOPacket *PacketMS::validateIcPacket(AOClient &client) const
         client_name = client.character(); // fallback in case of empty ooc name
     }
     if (area->testimonyRecording() == AreaData::TestimonyRecording::RECORDING || area->testimonyRecording() == AreaData::TestimonyRecording::ADD) {
-        if (!l_args[5].startsWith("wit"))
-            return PacketFactory::createPacket("MS", l_args);
-
+        // -1 indicates title
         if (area->statement() == -1) {
             l_args[4] = "~~-- " + l_args[4] + " --";
             l_args[14] = "3";
@@ -445,7 +443,6 @@ AOPacket *PacketMS::validateIcPacket(AOClient &client) const
 
         QRegularExpressionMatch match = isTestimonyJumpCommand(client.decodeMessage(l_args[4])); // Get rid of that pesky encoding, then do the fun part
         if (match.hasMatch()) {
-            client.m_pos = "wit";
             int jump_idx = match.captured("int").toInt();
             auto l_statement = area->jumpToStatement(jump_idx);
             l_args = l_statement.first;