Merge pull request OpenVicProject#125 from OpenVicProject/import-fix

Author: Spartan322

extension/src/LoadGameCompatibility.cpp

@@ -88,9 +88,8 @@ Error GameSingleton::_load_terrain_variants_compatibility_mode(String const& ter
 	}
 
 	// Load the terrain texture sheet and prepare to slice it up
-	Ref<Image> terrain_sheet;
-	terrain_sheet.instantiate();
-	if (terrain_sheet->load(terrain_texturesheet_path) != OK) {
+	Ref<Image> terrain_sheet = load_godot_image(terrain_texturesheet_path);
+	if (terrain_sheet.is_null()) {
 		UtilityFunctions::push_error("Failed to load terrain texture sheet: ", terrain_texturesheet_path);
 		return FAILED;
 	}

extension/src/LoadGameOpenVic.cpp

@@ -19,14 +19,15 @@ static Error _load_json_file(String const& file_description, String const& file_
 		return err == OK ? FAILED : err;
 	}
 	const String json_string = file->get_as_text();
-	JSON json;
-	err = json.parse(json_string);
+	Ref<JSON> json;
+	json.instantiate();
+	err = json->parse(json_string);
 	if (err != OK) {
 		UtilityFunctions::push_error("Failed to parse ", file_description, " file as JSON: ", file_path,
-			"\nError at line ", json.get_error_line(), ": ", json.get_error_message());
+			"\nError at line ", json->get_error_line(), ": ", json->get_error_message());
 		return err;
 	}
-	result = json.get_data();
+	result = json->get_data();
 	return err;
 }
 
@@ -175,12 +176,10 @@ Error GameSingleton::_parse_terrain_entry(String const& identifier, Variant cons
 		return FAILED;
 	}
 	const String terrain_path = terrain_texture_dir_path + identifier;
-	Ref<Image> terrain_image;
-	terrain_image.instantiate();
-	const Error err = terrain_image->load(terrain_path);
-	if (err != OK) {
+	const Ref<Image> terrain_image = load_godot_image(terrain_path);
+	if (terrain_image.is_null()) {
 		UtilityFunctions::push_error("Failed to load terrain image: ", terrain_path);
-		return err;
+		return FAILED;
 	}
 	return ERR(terrain_variants.add_item({ godot_to_std_string(identifier), colour, terrain_image }));
 }
@@ -229,18 +228,15 @@ Error GameSingleton::_load_map_images(String const& province_image_path, String
 	}
 
 	// Load images
-	Ref<Image> province_image, terrain_image;
-	province_image.instantiate();
-	terrain_image.instantiate();
-	Error err = province_image->load(province_image_path);
-	if (err != OK) {
+	Ref<Image> province_image = load_godot_image(province_image_path);
+	if (province_image.is_null()) {
 		UtilityFunctions::push_error("Failed to load province image: ", province_image_path);
-		return err;
+		return FAILED;
 	}
-	err = terrain_image->load(terrain_image_path);
-	if (err != OK) {
+	Ref<Image> terrain_image = load_godot_image(terrain_image_path);
+	if (terrain_image.is_null()) {
 		UtilityFunctions::push_error("Failed to load terrain image: ", terrain_image_path);
-		return err;
+		return FAILED;
 	}
 
 	if (flip_vertical) {
@@ -249,6 +245,7 @@ Error GameSingleton::_load_map_images(String const& province_image_path, String
 	}
 
 	// Validate dimensions and format
+	Error err = OK;
 	const Vector2i province_dims = province_image->get_size(), terrain_dims = terrain_image->get_size();
 	if (province_dims.x < 1 || province_dims.y < 1) {
 		UtilityFunctions::push_error("Invalid dimensions (", province_dims.x, "x", province_dims.y, ") for province image: ", province_image_path);
@@ -378,16 +375,14 @@ Error GameSingleton::_load_goods(String const& defines_path, String const& icons
 		});
 	game_manager.good_manager.lock_goods();
 	for (Good const& good : game_manager.good_manager.get_goods()) {
-		Ref<Image> image;
-		image.instantiate();
 		const String path = icons_dir_path + String { "/" } + std_to_godot_string(good.get_identifier()) + ".png";
-		const Error good_err = image->load(path);
-		if (good_err || image.is_null()) {
+		const Ref<Image> image = load_godot_image(path);
+		if (image.is_null()) {
 			UtilityFunctions::push_error("Failed to load good icon image: ", path);
 			err = FAILED;
 			continue;
 		}
-		Ref<Texture> tex = ImageTexture::create_from_image(image);
+		const Ref<Texture> tex = ImageTexture::create_from_image(image);
 		if (tex.is_null()) {
 			UtilityFunctions::push_error("Failed to generate good icon texture: ", path);
 			err = FAILED;

extension/src/Utilities.cpp

@@ -0,0 +1,16 @@
+#include "Utilities.hpp"
+
+#include <godot_cpp/classes/resource_loader.hpp>
+#include <godot_cpp/variant/utility_functions.hpp>
+
+using namespace godot;
+using namespace OpenVic;
+
+Ref<Image> OpenVic::load_godot_image(String const& path) {
+	if (path.begins_with("res://")) {
+		ResourceLoader* loader = ResourceLoader::get_singleton();
+		return loader ? loader->load(path) : nullptr;
+	} else {
+		return Image::load_from_file(path);
+	}
+}

extension/src/Utilities.hpp

@@ -1,6 +1,6 @@
 #pragma once
 
-#include <godot_cpp/variant/string.hpp>
+#include <godot_cpp/classes/image.hpp>
 
 #include "openvic/Types.hpp"
 
@@ -15,4 +15,6 @@ namespace OpenVic {
 	inline godot::String std_to_godot_string(std::string const& str) {
 		return str.c_str();
 	}
+
+	godot::Ref<godot::Image> load_godot_image(godot::String const& path);
 }

game/art/economy/goods/good_aeroplanes.png.import

@@ -1,3 +1,14 @@
 [remap]
 
-importer="keep"
+importer="image"
+type="Image"
+uid="uid://b1kuq7k8prdc5"
+path="res://.godot/imported/good_aeroplanes.png-7a57a10fe765bbdfb738ac1b788a0a02.image"
+
+[deps]
+
+source_file="res://art/economy/goods/good_aeroplanes.png"
+dest_files=["res://.godot/imported/good_aeroplanes.png-7a57a10fe765bbdfb738ac1b788a0a02.image"]
+
+[params]
+

game/art/economy/goods/good_ammunition.png.import

@@ -1,3 +1,14 @@
 [remap]
 
-importer="keep"
+importer="image"
+type="Image"
+uid="uid://mdsbfb1kx2yr"
+path="res://.godot/imported/good_ammunition.png-ab17c8b5b6f699fce3dd4d8842555fcc.image"
+
+[deps]
+
+source_file="res://art/economy/goods/good_ammunition.png"
+dest_files=["res://.godot/imported/good_ammunition.png-ab17c8b5b6f699fce3dd4d8842555fcc.image"]
+
+[params]
+

game/art/economy/goods/good_artillery.png.import

@@ -1,3 +1,14 @@
 [remap]
 
-importer="keep"
+importer="image"
+type="Image"
+uid="uid://bao8gdl5pjr3l"
+path="res://.godot/imported/good_artillery.png-ccb7bdbd694b82b7887edc5e918f891c.image"
+
+[deps]
+
+source_file="res://art/economy/goods/good_artillery.png"
+dest_files=["res://.godot/imported/good_artillery.png-ccb7bdbd694b82b7887edc5e918f891c.image"]
+
+[params]
+

game/art/economy/goods/good_automobiles.png.import

@@ -1,3 +1,14 @@
 [remap]
 
-importer="keep"
+importer="image"
+type="Image"
+uid="uid://cbgodumv1urxc"
+path="res://.godot/imported/good_automobiles.png-e7135793fbe9163a9843b57ef1cc9b26.image"
+
+[deps]
+
+source_file="res://art/economy/goods/good_automobiles.png"
+dest_files=["res://.godot/imported/good_automobiles.png-e7135793fbe9163a9843b57ef1cc9b26.image"]
+
+[params]
+

game/art/economy/goods/good_canned_food.png.import

@@ -1,3 +1,14 @@
 [remap]
 
-importer="keep"
+importer="image"
+type="Image"
+uid="uid://1uru8eufge1w"
+path="res://.godot/imported/good_canned_food.png-c83bafca536c627868b367290d5be590.image"
+
+[deps]
+
+source_file="res://art/economy/goods/good_canned_food.png"
+dest_files=["res://.godot/imported/good_canned_food.png-c83bafca536c627868b367290d5be590.image"]
+
+[params]
+

game/art/economy/goods/good_cattle.png.import

@@ -1,3 +1,14 @@
 [remap]
 
-importer="keep"
+importer="image"
+type="Image"
+uid="uid://gkjpiul6248l"
+path="res://.godot/imported/good_cattle.png-089bb1aba8f91888c577c6cb83992c61.image"
+
+[deps]
+
+source_file="res://art/economy/goods/good_cattle.png"
+dest_files=["res://.godot/imported/good_cattle.png-089bb1aba8f91888c577c6cb83992c61.image"]
+
+[params]
+