Chess: Port to `Core::Stream`

3 files changed, 44 insertions, 35 deletions

diff --git a/Userland/Games/Chess/ChessWidget.cpp b/Userland/Games/Chess/ChessWidget.cpp
index 48cc9a3ba9..5f6fa5fb33 100644
--- a/Userland/Games/Chess/ChessWidget.cpp
+++ b/Userland/Games/Chess/ChessWidget.cpp
@@ -8,8 +8,9 @@
 #include "PromotionDialog.h"
 #include <AK/DeprecatedString.h>
 #include <AK/Random.h>
+#include <AK/String.h>
 #include <LibCore/DateTime.h>
-#include <LibCore/File.h>
+#include <LibCore/Stream.h>
 #include <LibGUI/MessageBox.h>
 #include <LibGUI/Painter.h>
 #include <LibGfx/AntiAliasingPainter.h>
@@ -527,11 +528,11 @@ DeprecatedString ChessWidget::get_fen() const
     return m_playback ? m_board_playback.to_fen() : m_board.to_fen();
 }
 
-void ChessWidget::import_pgn(Core::File& file)
+ErrorOr<void> ChessWidget::import_pgn(Core::Stream::File& file)
 {
     m_board = Chess::Board();
 
-    ByteBuffer bytes = file.read_all();
+    ByteBuffer bytes = TRY(file.read_until_eof());
     StringView content = bytes;
     auto lines = content.lines();
     StringView line;
@@ -620,29 +621,31 @@ void ChessWidget::import_pgn(Core::File& file)
     m_playback_move_number = m_board_playback.moves().size();
     m_playback = true;
     update();
+
+    return {};
 }
 
-void ChessWidget::export_pgn(Core::File& file) const
+ErrorOr<void> ChessWidget::export_pgn(Core::Stream::File& file) const
 {
     // Tag Pair Section
-    file.write("[Event \"Casual Game\"]\n"sv);
-    file.write("[Site \"SerenityOS Chess\"]\n"sv);
-    file.write(DeprecatedString::formatted("[Date \"{}\"]\n", Core::DateTime::now().to_deprecated_string("%Y.%m.%d"sv)));
-    file.write("[Round \"1\"]\n"sv);
+    TRY(file.write("[Event \"Casual Game\"]\n"sv.bytes()));
+    TRY(file.write("[Site \"SerenityOS Chess\"]\n"sv.bytes()));
+    TRY(file.write(DeprecatedString::formatted("[Date \"{}\"]\n", Core::DateTime::now().to_deprecated_string("%Y.%m.%d"sv)).bytes()));
+    TRY(file.write("[Round \"1\"]\n"sv.bytes()));
 
     DeprecatedString username(getlogin());
-    const DeprecatedString player1 = (!username.is_empty() ? username.view() : "?"sv);
-    const DeprecatedString player2 = (!m_engine.is_null() ? "SerenityOS ChessEngine"sv : "?"sv);
-    file.write(DeprecatedString::formatted("[White \"{}\"]\n", m_side == Chess::Color::White ? player1 : player2));
-    file.write(DeprecatedString::formatted("[Black \"{}\"]\n", m_side == Chess::Color::Black ? player1 : player2));
-
-    file.write(DeprecatedString::formatted("[Result \"{}\"]\n", Chess::Board::result_to_points_deprecated_string(m_board.game_result(), m_board.turn())));
-    file.write("[WhiteElo \"?\"]\n"sv);
-    file.write("[BlackElo \"?\"]\n"sv);
-    file.write("[Variant \"Standard\"]\n"sv);
-    file.write("[TimeControl \"-\"]\n"sv);
-    file.write("[Annotator \"SerenityOS Chess\"]\n"sv);
-    file.write("\n"sv);
+    auto const player1 = (!username.is_empty() ? username.view() : "?"sv.bytes());
+    auto const player2 = (!m_engine.is_null() ? "SerenityOS ChessEngine"sv.bytes() : "?"sv.bytes());
+    TRY(file.write(DeprecatedString::formatted("[White \"{}\"]\n", m_side == Chess::Color::White ? player1 : player2).bytes()));
+    TRY(file.write(DeprecatedString::formatted("[Black \"{}\"]\n", m_side == Chess::Color::Black ? player1 : player2).bytes()));
+
+    TRY(file.write(DeprecatedString::formatted("[Result \"{}\"]\n", Chess::Board::result_to_points_deprecated_string(m_board.game_result(), m_board.turn())).bytes()));
+    TRY(file.write("[WhiteElo \"?\"]\n"sv.bytes()));
+    TRY(file.write("[BlackElo \"?\"]\n"sv.bytes()));
+    TRY(file.write("[Variant \"Standard\"]\n"sv.bytes()));
+    TRY(file.write("[TimeControl \"-\"]\n"sv.bytes()));
+    TRY(file.write("[Annotator \"SerenityOS Chess\"]\n"sv.bytes()));
+    TRY(file.write("\n"sv.bytes()));
 
     // Movetext Section
     for (size_t i = 0, move_no = 1; i < m_board.moves().size(); i += 2, move_no++) {
@@ -650,17 +653,19 @@ void ChessWidget::export_pgn(Core::File& file) const
 
         if (i + 1 < m_board.moves().size()) {
             const DeprecatedString black = m_board.moves().at(i + 1).to_algebraic();
-            file.write(DeprecatedString::formatted("{}. {} {} ", move_no, white, black));
+            TRY(file.write(DeprecatedString::formatted("{}. {} {} ", move_no, white, black).bytes()));
         } else {
-            file.write(DeprecatedString::formatted("{}. {} ", move_no, white));
+            TRY(file.write(DeprecatedString::formatted("{}. {} ", move_no, white).bytes()));
         }
     }
 
-    file.write("{ "sv);
-    file.write(Chess::Board::result_to_deprecated_string(m_board.game_result(), m_board.turn()));
-    file.write(" } "sv);
-    file.write(Chess::Board::result_to_points_deprecated_string(m_board.game_result(), m_board.turn()));
-    file.write("\n"sv);
+    TRY(file.write("{ "sv.bytes()));
+    TRY(file.write(Chess::Board::result_to_deprecated_string(m_board.game_result(), m_board.turn()).bytes()));
+    TRY(file.write(" } "sv.bytes()));
+    TRY(file.write(Chess::Board::result_to_points_deprecated_string(m_board.game_result(), m_board.turn()).bytes()));
+    TRY(file.write("\n"sv.bytes()));
+
+    return {};
 }
 
 void ChessWidget::flip_board()
diff --git a/Userland/Games/Chess/ChessWidget.h b/Userland/Games/Chess/ChessWidget.h
index cc45e76ef9..e0c5eb7a27 100644
--- a/Userland/Games/Chess/ChessWidget.h
+++ b/Userland/Games/Chess/ChessWidget.h
@@ -48,8 +48,8 @@ public:
     void set_show_available_moves(bool e) { m_show_available_moves = e; }
 
     DeprecatedString get_fen() const;
-    void import_pgn(Core::File&);
-    void export_pgn(Core::File&) const;
+    ErrorOr<void> import_pgn(Core::Stream::File&);
+    ErrorOr<void> export_pgn(Core::Stream::File&) const;
 
     int resign();
     void flip_board();
diff --git a/Userland/Games/Chess/main.cpp b/Userland/Games/Chess/main.cpp
index 273e2ba679..cde3533599 100644
--- a/Userland/Games/Chess/main.cpp
+++ b/Userland/Games/Chess/main.cpp
@@ -67,20 +67,24 @@ ErrorOr<int> serenity_main(Main::Arguments arguments)
     TRY(game_menu->try_add_separator());
 
     TRY(game_menu->try_add_action(GUI::Action::create("&Import PGN...", { Mod_Ctrl, Key_O }, [&](auto&) {
-        auto result = FileSystemAccessClient::Client::the().try_open_file_deprecated(window);
+        auto result = FileSystemAccessClient::Client::the().open_file(window);
         if (result.is_error())
             return;
 
-        widget->import_pgn(result.value());
-        dbgln("Imported PGN file from {}", result.value()->filename());
+        if (auto maybe_error = widget->import_pgn(*result.value().release_stream()); maybe_error.is_error())
+            dbgln("Failed to import PGN: {}", maybe_error.release_error());
+        else
+            dbgln("Imported PGN file from {}", result.value().filename());
     })));
     TRY(game_menu->try_add_action(GUI::Action::create("&Export PGN...", { Mod_Ctrl, Key_S }, [&](auto&) {
-        auto result = FileSystemAccessClient::Client::the().try_save_file_deprecated(window, "Untitled", "pgn");
+        auto result = FileSystemAccessClient::Client::the().save_file(window, "Untitled", "pgn");
         if (result.is_error())
             return;
 
-        widget->export_pgn(result.value());
-        dbgln("Exported PGN file to {}", result.value()->filename());
+        if (auto maybe_error = widget->export_pgn(*result.value().release_stream()); maybe_error.is_error())
+            dbgln("Failed to export PGN: {}", maybe_error.release_error());
+        else
+            dbgln("Exported PGN file to {}", result.value().filename());
     })));
     TRY(game_menu->try_add_action(GUI::Action::create("&Copy FEN", { Mod_Ctrl, Key_C }, [&](auto&) {
         GUI::Clipboard::the().set_data(widget->get_fen().bytes());