From def486305a1e1a3e2980e322003bcc9b83ce212c Mon Sep 17 00:00:00 2001
From: Aryan Baburajan <aryanbaburajan2007@gmail.com>
Date: Tue, 2 Apr 2024 14:41:26 +0530
Subject: [PATCH] SQLStudio: Port to GML Compilation

---
 Userland/DevTools/SQLStudio/CMakeLists.txt |  7 ++-----
 Userland/DevTools/SQLStudio/MainWidget.cpp | 13 +------------
 Userland/DevTools/SQLStudio/MainWidget.h   |  7 ++++---
 Userland/DevTools/SQLStudio/SQLStudio.gml  |  4 ++--
 Userland/DevTools/SQLStudio/main.cpp       |  2 +-
 5 files changed, 10 insertions(+), 23 deletions(-)

diff --git a/Userland/DevTools/SQLStudio/CMakeLists.txt b/Userland/DevTools/SQLStudio/CMakeLists.txt
index 94580ecf49..0e3c301e15 100644
--- a/Userland/DevTools/SQLStudio/CMakeLists.txt
+++ b/Userland/DevTools/SQLStudio/CMakeLists.txt
@@ -4,16 +4,13 @@ serenity_component(
     TARGETS SQLStudio
 )
 
-stringify_gml(SQLStudio.gml SQLStudioGML.h sql_studio_gml)
+compile_gml(SQLStudio.gml SQLStudioGML.cpp)
 
 set(SOURCES
     main.cpp
     MainWidget.cpp
     ScriptEditor.cpp
-)
-
-set(GENERATED_SOURCES
-    SQLStudioGML.h
+    SQLStudioGML.cpp
 )
 
 serenity_app(SQLStudio ICON app-sql-studio)
diff --git a/Userland/DevTools/SQLStudio/MainWidget.cpp b/Userland/DevTools/SQLStudio/MainWidget.cpp
index df129dcd4e..a757fe155b 100644
--- a/Userland/DevTools/SQLStudio/MainWidget.cpp
+++ b/Userland/DevTools/SQLStudio/MainWidget.cpp
@@ -6,7 +6,6 @@
  * SPDX-License-Identifier: BSD-2-Clause
  */
 
-#include <DevTools/SQLStudio/SQLStudioGML.h>
 #include <LibCore/DirIterator.h>
 #include <LibCore/StandardPaths.h>
 #include <LibDesktop/Launcher.h>
@@ -60,17 +59,7 @@ static Vector<ByteString> lookup_database_names()
     return database_names;
 }
 
-ErrorOr<NonnullRefPtr<MainWidget>> MainWidget::create()
-{
-    auto widget = TRY(try_make_ref_counted<MainWidget>());
-
-    TRY(widget->load_from_gml(sql_studio_gml));
-    TRY(widget->setup());
-
-    return widget;
-}
-
-ErrorOr<void> MainWidget::setup()
+ErrorOr<void> MainWidget::initialize()
 {
     m_new_action = GUI::Action::create("&New", { Mod_Ctrl, Key_N }, TRY(Gfx::Bitmap::load_from_file("/res/icons/16x16/new.png"sv)), [this](auto&) {
         open_new_script();
diff --git a/Userland/DevTools/SQLStudio/MainWidget.h b/Userland/DevTools/SQLStudio/MainWidget.h
index dfb9b4f951..1053189c4a 100644
--- a/Userland/DevTools/SQLStudio/MainWidget.h
+++ b/Userland/DevTools/SQLStudio/MainWidget.h
@@ -9,6 +9,8 @@
 
 #include <AK/LexicalPath.h>
 #include <AK/Vector.h>
+#include <LibGUI/ComboBox.h>
+#include <LibGUI/TableView.h>
 #include <LibGUI/Widget.h>
 #include <LibSQL/SQLClient.h>
 
@@ -21,7 +23,8 @@ class MainWidget : public GUI::Widget {
 
 public:
     virtual ~MainWidget() = default;
-    static ErrorOr<NonnullRefPtr<MainWidget>> create();
+    static ErrorOr<NonnullRefPtr<MainWidget>> try_create();
+    ErrorOr<void> initialize();
 
     ErrorOr<void> initialize_menu(GUI::Window*);
     void open_new_script();
@@ -30,8 +33,6 @@ public:
     bool request_close();
 
 private:
-    ErrorOr<void> setup();
-
     ScriptEditor* active_editor();
 
     void update_title();
diff --git a/Userland/DevTools/SQLStudio/SQLStudio.gml b/Userland/DevTools/SQLStudio/SQLStudio.gml
index c08bffe0d7..0cb51c8eb4 100644
--- a/Userland/DevTools/SQLStudio/SQLStudio.gml
+++ b/Userland/DevTools/SQLStudio/SQLStudio.gml
@@ -12,12 +12,12 @@
         @GUI::TabWidget {
             name: "script_tab_widget"
             reorder_allowed: true
-            show_close_buttons: true
+            close_button_enabled: true
         }
 
         @GUI::TabWidget {
             name: "action_tab_widget"
-            show_close_buttons: true
+            close_button_enabled: true
             visible: false
         }
     }
diff --git a/Userland/DevTools/SQLStudio/main.cpp b/Userland/DevTools/SQLStudio/main.cpp
index 541e34e90e..e984e2c814 100644
--- a/Userland/DevTools/SQLStudio/main.cpp
+++ b/Userland/DevTools/SQLStudio/main.cpp
@@ -31,7 +31,7 @@ ErrorOr<int> serenity_main(Main::Arguments arguments)
     window->set_icon(app_icon.bitmap_for_size(16));
     window->set_title("SQL Studio");
 
-    auto main_widget = TRY(MainWidget::create());
+    auto main_widget = TRY(MainWidget::try_create());
     window->set_main_widget(main_widget);
     TRY(main_widget->initialize_menu(window));