From 6c5fb2ca63b300b748f7b8e66efa532a450b0a58 Mon Sep 17 00:00:00 2001
From: Karol Kosek <krkk@krkk.ct8.pl>
Date: Thu, 12 Aug 2021 22:42:48 +0200
Subject: [PATCH] HackStudio: Remove an old file from the vectors in 'Save
 as...' action

If you saved a file under a different name and then went back to
the first file, then you had the same TextDocument buffer, and therefore
the same changes to the file as in the new one.
---
 Userland/DevTools/HackStudio/HackStudioWidget.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Userland/DevTools/HackStudio/HackStudioWidget.cpp b/Userland/DevTools/HackStudio/HackStudioWidget.cpp
index fb068f9c0f..4568188e63 100644
--- a/Userland/DevTools/HackStudio/HackStudioWidget.cpp
+++ b/Userland/DevTools/HackStudio/HackStudioWidget.cpp
@@ -701,7 +701,10 @@ NonnullRefPtr<GUI::Action> HackStudioWidget::create_save_as_action()
 
         auto new_project_file = m_project->create_file(relative_file_path);
         m_open_files.set(relative_file_path, *new_project_file);
+        m_open_files.remove(old_filename);
+
         m_open_files_vector.append(relative_file_path);
+        m_open_files_vector.remove_all_matching([&old_filename](auto const& element) { return element == old_filename; });
 
         update_window_title();