Implement LuaScript::_get_script_signal_list

Author: gilzoide

src/script-language/LuaScript.cpp

@@ -169,8 +169,11 @@ bool LuaScript::_has_script_signal(const StringName &p_signal) const {
 }
 
 TypedArray<Dictionary> LuaScript::_get_script_signal_list() const {
-	// TODO
-	return {};
+	TypedArray<Dictionary> signals;
+	for (auto [name, signal] : metadata.signals) {
+		signals.append(signal.to_dictionary());
+	}
+	return signals;
 }
 
 bool LuaScript::_has_property_default_value(const StringName &p_property) const {

src/script-language/LuaScriptLanguage.cpp

@@ -150,15 +150,17 @@ Dictionary LuaScriptLanguage::_validate(const String &script, const String &path
 	Dictionary result;
 	Variant f = lua_state->load_string(script, path);
 	if (LuaError *error = Object::cast_to<LuaError>(f)) {
-		auto line_re = RegEx::create_from_string(R":(\d+):");
-		auto match = line_re->search(error->get_message());
-		Dictionary error_dict;
-		error_dict["path"] = path;
-		error_dict["line"] = match->get_string().to_int();
-		error_dict["column"] = 1;
-		error_dict["message"] = error->get_message();
+		if (validate_errors) {
+			auto line_re = RegEx::create_from_string(R":(\d+):");
+			auto match = line_re->search(error->get_message());
+			Dictionary error_dict;
+			error_dict["path"] = path;
+			error_dict["line"] = match->get_string().to_int();
+			error_dict["column"] = 1;
+			error_dict["message"] = error->get_message();
+			result["errors"] = Array::make(error_dict);
+		}
 		result["valid"] = false;
-		result["errors"] = Array::make(error_dict);
 	}
 	else {
 		result["valid"] = true;

src/script-language/LuaScriptMetadata.cpp

@@ -69,6 +69,7 @@ void LuaScriptMetadata::setup(const sol::table& t) {
 			is_tool = to_variant(value).booleanize();
 		}
 		else if (auto signal = value.as<sol::optional<LuaScriptSignal>>()) {
+			signal->name = name;
 			signals.insert(name, *signal);
 		}
 		else if (auto property = value.as<sol::optional<LuaScriptProperty>>()) {

src/script-language/LuaScriptSignal.cpp

@@ -27,6 +27,19 @@
 
 namespace luagdextension {
 
+MethodInfo LuaScriptSignal::to_method_info() const {
+	MethodInfo mi;
+	mi.name = name;
+	for (int i = 0; i < arguments.size(); i++) {
+		mi.arguments.push_back(PropertyInfo(Variant::Type::NIL, arguments[i]));
+	}
+	return mi;
+}
+
+Dictionary LuaScriptSignal::to_dictionary() const {
+	return to_method_info();
+}
+
 static LuaScriptSignal lua_signal(sol::variadic_args arguments) {
 	LuaScriptSignal signal = {
 		.arguments = VariantArguments(arguments).get_array(),

src/script-language/LuaScriptSignal.hpp

@@ -22,6 +22,7 @@
 #ifndef __LUA_SCRIPT_SIGNAL_HPP__
 #define __LUA_SCRIPT_SIGNAL_HPP__
 
+#include <godot_cpp/core/object.hpp>
 #include <godot_cpp/variant/variant.hpp>
 
 typedef struct lua_State lua_State;
@@ -31,8 +32,12 @@ using namespace godot;
 namespace luagdextension {
 
 struct LuaScriptSignal {
+	StringName name;
 	PackedStringArray arguments;
 
+	MethodInfo to_method_info() const;
+	Dictionary to_dictionary() const;
+
 	static void register_lua(lua_State *L);
 };