Ensure global shader sampler parameters are initialized when loading

[Lateasusual]

During editor initialization, Global Shader Parameters with Texture references are skipped since the textures may be reloaded later. Previously this also skipped creating the parameter, which would cause shaders to fail to compile during initialization due to missing global uniforms, and spam the console with errors.

This just changes it to still create the parameters with invalid RID instead of skipping it entirely.

[clayjohn]

This makes sense! It looks like the compatibility renderer uses the same code. Can you make this change there as well?

godot/drivers/gles3/storage/material_storage.cpp

Lines 1968 to 1980 in d4f726f

	if (gvtype >= RS::GLOBAL_VAR_TYPE_SAMPLER2D) {
	//textire
	if (!p_load_textures) {
	continue;
	}
	String path = value;
	if (path.is_empty()) {
	value = RID();
	} else {
	Ref<Resource> resource = ResourceLoader::load(path);
	value = resource;
	}

[Lateasusual]

Ah, I forgot about that, thanks!

[clayjohn]

Looks great!

Tested locally and it also removes the !global_shader_uniforms.variables.has(p_name) error that you often get at startup when using textures in a global shader.

[akien-mga]

Thanks!