Error creating .vmt files

[TheBreadPerson]

Describe the bug
Creating .vmt files fails when the source texture is not already inside the configured materials folder.

Not sure if I missed something, but I had an error creating .vmt files from texutres. I'd get the following error each time unless the texture was in the set materials folder:

res://addons/godotvmf/godotvmt/vmt_material_conversion_context_menu.gd:89 - Cannot call method 'store_string' on a null value.

I looked at the code and saw the basetexture var seems to be relying on the texture to be actively in the VMFConfig materials folder:

var basetexture := path.replace(VMFConfig.materials.target_folder, '').substr(1).replace('.' + path.get_extension(), '') as String;

Not sure if this is intended, but I just changed this line to something that wasn't reliant on the texture's path:

var basetexture := path.get_file().get_basename() as String

This worked for me, please lmk if I did something wrong or if this is a geniune bug.

[H2xDev]

All materials should be configured inside the target folder ( Project Settings -> Godot VMF -> Materials -> Target Folder) because Hammer Editor and the importer itself relying on the strict folder structure defined by Source Engine. I think i need to add some error message in this case.

In your case you need to create all VMT/VTF files only inside the target folder.

[TheBreadPerson]

ah ok, thanks for clarifying

[TheBreadPerson]

actually I just found the issue as far as I'm aware.

var basetexture := path.replace(VMFConfig.materials.target_folder, '').substr(1).replace('.' + path.get_extension(), '') as String;

the substring call seems to cut off the first character for no reason, causing the same error from the issue. I removed it and it works perfectly for me.

var basetexture := path.replace(VMFConfig.materials.target_folder, '').replace('.' + path.get_extension(), '') as String;

[H2xDev]

Oh lol! You're right. I'll add it to the upcoming version. Thank you! ❤️